Должен ли я выполнять многоцелевые функции?

Question

Я использую CodeIgniter, и у меня есть модель, которая извлекает, скажем, все рецепты, где вы можете установить смещение и предел. Должен ли я распространять эту функцию для получения одного рецепта? Это хорошая практика?

Answer 1

Ваша цель должна заключаться в том, чтобы свести к минимуму дублирование кода и максимизировать понятность. Эти два часто расходятся. Вы можете в конечном итоге с минимальным дублированием кода, но иметь 12 необязательных параметров для функции. Итак, некоторые общие советы:

• Рассмотрите возможность упаковки общей функциональности в третью функцию, а затем две функции могут оба вызвать это по мере необходимости;
• Использование объекта или массива аргументов, если функция имеет более 3-4 параметров;
• Уничтожение кода должно быть сведено к минимуму, а не полностью устранено. Иногда самое четкое решение предполагает некоторое дублирование кода;
• Назначение функции или объекта должно быть четким. Если все, что он делает, полностью изменяется в зависимости от параметра, то вы, вероятно, будете путать людей.

С вашего конкретного случая, я полагаю, вы хотите, чтобы в конечном итоге что-то вроде:

function get_recipes($offset, $limit) { 
    // execute query and get resource 
    $ret = array(); 
    while ($row = mysql_fetch_assoc($rs)) { 
    $ret[] = build_recipe($row); 
    } 
    return $ret; 
} 

function get_recipe($id) { 
    // execute query and get row object 
    return build_recipe($row); 
} 

function build_recipe($row) { 
    // construct a recipe object from the row 
} 

Answer 2

В общем, для ясности, функция должна выполнять одну задачу. Однако «получение N строк» ​​- единственная задача - даже когда N == 1; поэтому я бы сказал, что этот случай квалифицируется, т. е. функция НЕ действительно «многоцелевая»!)

Answer 3

Ответ на этот вопрос прост.

Просто помните, что Worse is better