L'exemple suivant utilise une variable intermédiaire $shouldNotSave sur le retour de la fonction preg_match.
$regex = $this->makeRegex();
$shouldNotSave = preg_match($regex, $content);
if ($shouldNotSave) {
return $response;
}
Cette variable $shouldNotSave peut paraître superflue et des outils de refactoring tel que rectorphp proposera de modifier ce code de la manière suivante :
$regex = $this->makeRegex();
if (preg_match($regex, $content)) {
return $response;
}
Ce code, bien que plus condensé, perdra cependant une information importante sur son fonctionnement.
La variable $shouldNotSave donnait du sens au retour du preg_math permettant de mieux comprendre le fonctionnement et l'intention du code, désormais, cette intention a disparu et le fonctionnement de la méthode est plus cryptique qu'auparavant.
Écrivez du code pour qu'il soit lu et compris et non pour être 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.