J'utilise CodeIgniter et j'ai un modèle qui récupère toutes les recettes où vous pouvez définir le décalage et la limite. Devrais-je étendre cette fonction pour récupérer une seule recette? Est-ce une bonne pratique?Dois-je implémenter des fonctions polyvalentes?
Votre objectif devrait être de minimiser la duplication de code tout en maximisant la compréhensibilité. Ces deux sont souvent en désaccord. Vous pouvez vous retrouver avec une duplication de code minimale mais avoir 12 paramètres optionnels pour une fonction. Donc, quelques conseils généraux:
Avec votre cas, je pense que vous voudriez finir avec quelque chose comme:
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
}
En général, pour plus de clarté, une fonction doit faire une seule tâche. Cependant, "obtenir N lignes" est une tâche unique - même lorsque N == 1; donc je dirais que cette affaire est qualifiée, c'est-à-dire que la fonction n'est PAS vraiment "multi-usage"! -)
La réponse à cette question est simple.
Rappelez-vous simplement que Worse is better
En fait, je l'ai combiné get_recipe et get_recipes. Si l'identifiant est défini et que les autres paramètres sont NULL, il ne renvoie qu'un seul champ, etc. – alimango
Je pense que c'est potentiellement un peu déroutant, mais c'est difficile à dire sans détails. En règle générale, je préfère un schéma comme j'ai fait qui a clairement nommé les fonctions (bien que ces trois pourraient être mieux nommés) qui utilisent la décomposition fonctionnelle pour réduire la duplication. – cletus
Je suis d'accord avec Cletus en séparant le 'get_recipe' de' get_recipes'. Il sera plus facile pour celui qui tiendra votre code de comprendre votre code. – Randell