use right ifs