Следует избегать if .. else .. конструкций?

Question

[Никита Андреевич]

Всем привет.
Один знакомый программист сказал что лучше избегать в своем коде логических конструкций таких как if.. else и т д .
А я же наоборот пишу их везде где только можно. Например если можно написать не 2 функции, а одну - но будет меняться только входное значение, то я проверю это значение через if else.
Тaк все же следует избегать или знакомый ошибается ?

Comments

[Антон Литвиненко]

Конечно, стоит избегать базовых конструкций, которые есть в любом языке программирования. Зачем они вообще. А циклов так вообще...

[Виталий Першин]

Не помешал бы более конкретный пример.

[Сергей delphinpro]

Например если можно написать не 2 функции, а одну - но будет меняться только входное значение, то я проверю это значение через if else.

В программировании есть принцип единственной ответственности.
Если ваша функция, в зависимости от параметра выполняет совершенно разные действия, то эту функцию лучше разделить на две.

Вот примитивный пример:

function getUser($userId = null){
  if (is_null($userId)) {
    // …логика создания нового юзера
  } else {
   // ... логика возврата из базы существующего юзера
  }
}

Такую функцию следует разделить на две:

function getUserById($id) {}
function createUser() {}

Answer 1

[d-sem]

По ситуации и вкусу.

Если вся логика одна строка, то уместней тернарный оператор.
Если в if будет return или exception, то else не нужен.

Если считаете нужным использовать else, то используйте.

Вот что точно следует избегать, так это множество вложенных if...else.