Le fait de demander deux fois la même chose donne-t-il de mauvais résultats ?

Question

J'ai une fonction :

function getMilk() {
    if($condition == true)
        return "Milk for you, madam";
    return false;
}

Exemple 1 :

if(getMilk()) 
    echo getMilk();

Le premier exemple fait-il courir PHP deux fois après le lait ?

Exemple 2 :

echo getMilk(); // don't check for milk first, just get it if they have it

Si j'étais PHP, j'opterais plutôt pour le deuxième exemple. Ainsi, je n'aurais pas à courir jusqu'au magasin pour vérifier s'il y a du lait, puis à courir une fois de plus pour l'obtenir.

L'exemple 2 serait-il plus rapide/meilleur, ou cela n'a-t-il pas d'importance ?

Answer 1

Oui, vous appelez la fonction deux fois. Pour éviter cela (car l'appel de la fonction peut être coûteux), procédez de l'une des manières suivantes :

$getMilk = getMilk();
if($getMilk) echo $getMilk;

Vous pouvez le réduire à un format d'une ligne (mais illisible) :

if ($getMilk = getMilk()) echo $getMilk;

Vous pouvez également utiliser un if ternaire, avec une solution de continuité :

echo getMilk()?:""; //Will echo the result of getMilk() if there is any or nothing.

Answer 2

Oui, PHP appellera les fonctions exactement aussi souvent que vous le lui demanderez. La fonction sera exécutée deux fois. Vous ne devriez pas faire cela, car vous ne connaissez pas intrinsèquement le coût d'une fonction particulière, et/ou cela peut changer dans le futur si vous réécrivez les détails de l'implémentation de la fonction. En particulier, les fonctions qui ont des effets secondaires (elles modifient quelque chose, par exemple elles écrivent dans un fichier) ne doivent pas être appelées plus souvent que nécessaire.

Au lieu de cela, vous stockez le résultat de la fonction dans une variable :

$milk = getMilk();
if ($milk) {
    echo $milk;
}

Il est même possible d'y ajouter une ligne :

if ($milk = getMilk()) {
    echo $milk;
}

Answer 3

La question est plus théorique qu'autre chose puisque la différence de performance serait négligeable.

Cela dit, dans votre cas, la deuxième approche est la bonne puisque false ne se traduit par aucun résultat. Ainsi, les

echo getMilk();

soit l'écho de la chaîne, soit rien du tout.