Je pense qu’il y a d’autres points discutables : le nom de la fonction et son commentaire (on ne parle de BDD nulle part), la requête n’a rien à faire là (et son retour n’est pas testé).
Aussi, pour ce genre de test, je préfère passer par un tableau (une propriété en protected dans la classe) (par contre, je développe en frenglish )

D’une manière générale, je ne parlais pas forcément de mauvaises pratiques, mais juste de « points discutables », qui méritent d’être formalisés dans une entreprise, si ce n’est au niveau d’une communauté entière.

Pour le return, cependant, je suis moins mitigé que toi ! Je me base sur les dires du manuel : http://php.net/manual/fr/function.return.php

À bientôt !

3,4 => Ah oui, le nombre de jeunes programmeurs qui ne connaissent ni Doxygen ni PHPDoc !…

6 => Mouais, sujet à discussion. Habituellement, on considère que la documentation au-dessus de la fonction doit expliquer ce que fait cette fonction (et non comment elle le fait). Par contre, expliquer à l’intérieur comment l’algorithme a été pensé, c’est une bonne chose. Je suis d’accord que dans ton exemple les commentaires sont inutiles ; mais là encore, c’est difficile de dire que les commentaires dans le code sont une mauvaise pratique.

11 => Ou bien on pourrait retourner NULL. Ou lever une exception…

12 => Un ORM mal utilisé (surtout par de jeunes développeurs) est bien pire que l’utilisation demysql_query directe. Bon, le minimum serait de passer par une librairie d’abstraction. Mais dans le cadre de ton test, avec juste un bout de code sorti de tout contexte, c’est difficile de leur en vouloir.

16 => L’argument n’est pas valable. « if » est un mot-clé du langage, et il a besoin de parenthèses. Là on est dans le domaine de la norme de codage, ça revient à discuter des goûts et des couleurs.

17 => Argument très discutable. C’est sûr que pour les développeurs étourdis, ça évite d’avoir des espaces qui s’ajoutent sur la sortie standard. Mais ça reste globalement malpropre, et pourrait poser des problèmes avec certains compilateurs PHP.

18 => Vu les problèmes de logique (intentionnels, je sais) de cette fonction, elle mériterait surtout d’être effacée !

Je pense qu’il y a d’autres points discutables : le nom de la fonction et son commentaire (on ne parle de BDD nulle part), la requête n’a rien à faire là (et son retour n’est pas testé).
Aussi, pour ce genre de test, je préfère passer par un tableau (une propriété en protected dans la classe) (par contre, je développe en frenglish )