Как сократить и улучшить представленный код

Question

Как сократить и улучшить представленный код

Давид Мейстер Мастер (2089), на голосовании 1 год назад

 if (!empty($_POST['factor']) && (!empty($_POST['location']))) { 
  if (in_array($_POST['factor'], array(1, 2, 3, 4, 5))) { 
    for ($i = 0; $i < $factor; $i++) { 
      if ($factor === 1) { 
         
      } else { 
 
      } 
    } 
  } 
}

Answer 1

Одно из возможных улучшений кода может быть следующим:

Проверку условия if (in_array($_POST['factor'], array(1, 2, 3, 4, 5))) можно упростить с помощью конструкции switch.
Использование цикла for не обязательно, так как блок кода, который нужно выполнить, не зависит от переменной $i. Вместо цикла можно использовать конструкцию if...else.
Можно улучшить читабельность кода, разбив его на несколько функций с понятными именами.
Вот как это может выглядеть:

Или же: