Le nouvel attribut natif #[\NoDiscard] indique que la valeur de retour d’une fonction ou d’une méthode ne doit pas être ignorée. Autrement dit, si une fonction est marquée avec #[\NoDiscard] mais qu’elle est appelée sans utiliser son résultat, alors un warning sera lancé à l’exécution. C’est un mécanisme de sécurité logique et de qualité de code.

1. php 8.5
#[NoDiscard]
Attribute
( les nouveautés... )

2. Le nouvel attribut natif #[NoDiscard] indique que
la valeur de retour d’une fonction ou d’une
méthode ne doit pas être ignorée. Autrement dit,
si une fonction est marquée avec #[NoDiscard]
mais qu’elle est appelée sans utiliser son résultat,
alors un warning sera lancé à l’exécution.
C’est un mécanisme de sécurité logique
et de qualité de code.

3. Bug silencieux ?
L’appel à slugify() est bien réalisé
sans effet de bord et
le remplacement est effectué,
mais le résultat est perdu.
Au mieux, on peut imaginer que c’est
du code inutile ou mort,
mais il est plus probable que
ce soit un oubli, et donc un bug…

4. Un warning en PHP 8.5
L’appel provoque un warning
si on ajoute #[NoDiscard] à la fonction,
ce qui permet de détecter l’oublie.
OK si utilisé

5. La plupart des éditeurs mettent déjà en
œuvre ce mécanisme.
Il sera désormais inclus dans PHP.
Le fait de marquer la fonction est une
indication importante dans le code car elle
attire l’attention du lecteur/contributeur.

6. Pour forcer ?
Il suffit de caster en (void) pour ignorer.
Mais ce n’est pas spécialement élégant.
Bonne idée, de forcer ?
pas certain...

7. Références
RFC
👉 https://wiki.php.net/rfc/marking_return_value_as_important

8. Qui suis-je ?
Thierry Leriche
Architecte - développeur - concepteur
rédacteur - professeur
Linked’in
👉 https://www.linkedin.com/in/thierryler