Envoyé par
Luc Hermitte
Le vrai soucis, c'est les taux de commentaires imposées -- surtout si le calcul de la métrique ignore les commentaires d'interface. Ils poussent vite à rajouter des redondances un peu, voire excessivement, inutiles. D'autant qu'elles vont très vite devenir obsolètes au fil des refactoring (*). Le pire des cas, c'est encore quand le commentaire explique un truc, que le code fait autre chose, et qu'au final la fonction est buggué. Là, difficile de savoir si la correction consiste à corriger pour faire ce que le commentaire explique, ou s'il faut chercher une nouvelle solution. Et encore, il faut commencer par comprendre ce qui est exactement fait dans la fonction, en espérant que l'on ne se base pas sur des commentaires obsolètes pour se faire une idée.
(*) Je ne doute pas que plusieurs d'entre-vous vont hurler en disant que le problème est là: il fallait garder les commentaires à jour. Mais soyons honnête un instant! Vous n'en voyez jamais des commentaires obsolètes ? Plus un code sur lequel j'interviens est vieux, plus la chance pour que ses commentaires soient obsolètes est importante. Et l'expérience m'a montré que la probabilité d'en voir est très loin d'être nulle. Sans parler qu'au fur et à mesure que l'on corrige un code pour lui faire passer les tests, on oublie vite de patcher les commentaires tant que la correction n'est pas validée. Avant que mes collègues se focalisent sur des commentaires, je préfère de loin qu'ils se focalisent à respecter le SRP (et d'autres aspects bien plus importants)