Как лучше писать if условия?
Лучше писать так:return (foo == bar && foo >= 1) ? true : false;
Вы должны завершить работу функции так рано, как это возможно. Дальше может быть создание больших и сложных объектов, а потом фигак и параметр null, а вы уже насоздавали всего ненужного.