J'ai assisté à un événement sur la conception de logiciels il y a quelques semaines et l'un des commentaires formulés était: "Je suis sûr que nous reconnaissons tous le code défectueux quand nous le voyons" et tout le monde a acquiescé avec honte sans autre discussion.
Ce genre de chose m'inquiète toujours car il y a ce truisme que tout le monde pense être un conducteur supérieur à la moyenne. Bien que je pense pouvoir reconnaître le mauvais code, j'aimerais en savoir plus sur ce que les autres utilisateurs considèrent comme des odeurs de code, car il est rarement discuté en détail sur les blogs des internautes et seulement dans une poignée de livres. En particulier, je pense qu'il serait intéressant d'entendre parler de tout ce qui ressemble à une odeur de code dans une langue mais pas une autre.
Je vais commencer par un facile:
Code dans le contrôle de source qui contient une forte proportion de code commenté - pourquoi est-il là? Était-ce censé être supprimé? est-ce un travail à moitié fini? peut-être n'aurait-il pas dû être commenté et n'a été fait que lorsque quelqu'un testait quelque chose? Personnellement, je trouve ce genre de chose vraiment ennuyeux, même s’il s’agit d’une simple ligne ici et là, mais il est totalement inacceptable de voir de gros blocs entremêlés du reste du code. Cela indique aussi généralement que le reste du code sera probablement de qualité douteuse.