Variable intermédiaire

L’exemple suivant utilise une variable intermédiaire $shouldNotSave pour stocker le résultat de la fonction preg_match :

$regex = $this->makeRegex();

$shouldNotSave = preg_match($regex, $content);

if ($shouldNotSave) {
    return $response;
}

Cette variable peut sembler superflue, et des outils de refactoring comme RectorPHP pourraient proposer de simplifier le code ainsi :

$regex = $this->makeRegex();

if (preg_match($regex, $content)) {
    return $response;
}

Le code est plus court, mais il perd une information importante : $shouldNotSave donnait du sens au retour de preg_match et expliquait l’intention derrière la condition. Sans elle, le fonctionnement de la méthode devient plus cryptique.

Écrivez du code pour qu’il soit lu et compris, pas pour qu’il soit le plus court possible.

L'utilisation d'une variable intermédiaire améliorera l'expressivité de votre code, un nom de variable adéquate donnera du contexte et rendra votre code à la fois plus lisible et compréhensible.