Pourquoi est-il mal de commenter le code, puis de le supprimer progressivement pour garder une trace de ce que j'ai déjà fait et de ce qui reste à faire?


21

Chaque fois que je découvre qu'une grande partie de mon code doit être modifiée, soit parce qu'elle est incorrecte, soit parce qu'elle doit être adaptée aux changements architecturaux majeurs nécessités par d'autres raisons, voici ce que je fais généralement:

  1. Je commente tout le code que je soupçonne de devoir changer. Je traite le code commenté comme une sorte de ma liste TODO.
  2. Je passe progressivement en revue le code commenté et décommente les parties de ce code, ou les copie-colle ailleurs et les modifie si nécessaire, ou réécris des parties de ce code à partir de zéro, en regardant le code commenté pour référence. Chaque fois que je pense avoir terminé avec une partie du code commenté, je le supprime.
  3. Je continue jusqu'à ce que je ne voie plus de code commenté.

Je dois noter que je le fais en grande partie sur le projet personnel que je développe seul.

Cependant, on m'a dit que je devais arrêter de faire ça. On m'a dit qu'à la place, je devrais commencer à utiliser git, en faisant référence aux anciens commits pour voir l'ancien code, au lieu de laisser du code commenté. On m'a dit:

Commenter le code est une mauvaise habitude qui devrait être effacée. Vous manquez d'expérience donc vous ne comprenez pas cela. Si, dans quelques années, vous voyez le code d'une autre personne qui aime commenter le code, vous commencerez vous-même à jurer sur cette personne. Chaque fois que je vois du code commenté, je le supprime dans son intégralité, sans même le regarder, car généralement un tel code n'a aucune valeur. Vous ne verrez certainement pas les inconvénients de commenter le code dans de petits projets d'une seule personne; mais si vous trouvez un travail et que vous gardez cette habitude là, ce sera dommage.

Puis-je demander quels sont ces inconvénients de ce que je fais que je ne vois pas maintenant?

Je dois dire que je n'ai pas vraiment envie de n'utiliser que git pour voir le code passé. Comme je l'ai dit, je traite les commentaires du code comme une sorte de liste de tâches; alors que git me montrera à quoi ressemblait le code, il ne me montrera pas clairement quelles parties du code doivent encore être examinées et lesquelles sont déjà effectuées. Je crains de manquer une partie du code et d'introduire des bogues.

Pour être complet, je pense que je devrais ajouter que la personne que je cite est un développeur expérimenté et un fan du "Clean Code" d'oncle Bob - et oncle Bob a critiqué commenter le code durement dans son livre.


Soumettez-vous le code commenté au contrôle de version?
Arrêtez de nuire à Monica

1
@Goyo Je n'utilisais pas du tout le contrôle de version. On m'a dit que je devrais certainement commencer à utiliser le contrôle de version (même s'il s'agit d'un projet personnel) et que, entre autres, le contrôle de version me permettra d'arrêter de commenter le code (ce que je devrais).
gaazkam

4
Si le code commenté n'est pas visible dans la branche principale après la fusion (et vous pouvez le faire) qui sera blessé?. Si vous ressentez le besoin de vous engager avant de vous débarrasser du code commenté qui laisse entendre que vous pourriez prendre des mesures trop importantes, mais c'est une autre affaire.
Arrêtez de nuire à Monica

4
Il est vrai que si vous codez du code, vous pouvez facilement le défaire en le décommentant, mais si vous avez modifié certains trucs dans pas mal de fichiers et que vous devez revenir en arrière, vous êtes plutôt foutu sans contrôle de version. Donc "commencer à utiliser le contrôle de code source" devrait être bien au-dessus de votre priorité que "ne pas commenter le code" :)
Walfrat

2
Attendez, vous avez écrit que vous avez une base de code suffisamment grande pour que des portions de celle-ci "doivent parfois être adaptées aux changements architecturaux majeurs" - et vous n'UTILISEZ PAS ACTUELLEMENT LE CONTRÔLE DE VERSION? WTF - sérieusement? Vous plaisantez, n'est-ce pas? Si c'est vraiment vrai, vous avez des problèmes plus importants que la question de savoir si votre façon de travailler avec du code commenté est correcte ou non.
Doc Brown

Réponses:


29

Si vous supprimez finalement tout le code commenté, je ne vois aucun problème réel avec cela. Laisser du code commenté dans votre base de code est une mauvaise pratique, mais ce n'est pas ce que vous faites si vous travaillez sur tout cela et l'éliminez. Je soupçonne que la personne à qui vous parlez ne comprend pas le processus que vous utilisez ou est dogmatique.

En réalité, le code commenté est inoffensif. Le problème est que c'est désordonné et qu'il est difficile à lire. Il y a des péchés bien pires mais c'est une chose très simple à éliminer. En tant que personne qui a commenté le code, vous êtes le mieux placé pour déterminer qu'il peut être complètement supprimé.

De nombreux IDE et éditeurs de code comprennent une sorte de syntaxe «TODO» dans les commentaires. Il s'agit d'une autre façon de marquer ce qui doit être changé. Vous voudrez peut-être considérer cela car il donne un peu plus d'informations sur ce que vous pensiez lorsque vous l'avez marqué.

À la fin de la journée, faites les choses de la manière qui vous donnera le meilleur résultat. Même s'il s'agissait d'un projet d'équipe, tant que vous supprimez tout le code commenté, vous ne chargez personne d'autre.


Je ne me souviens pas où je l'ai entendu, mais il y a un argument selon lequel en général le code commenté n'est pas inoffensif car il n'est jamais refactorisé. Notez que cela présuppose que vous finirez par annuler le commentaire et réutiliser ce bloc de code.
Peter M

@PeterM Le point ici est que ça va tant que vous vous en débarrassez. Vous ne devez pas laisser de code commenté dans votre base de code. Quelque chose que je fais souvent lors de la refactorisation est de commenter les variables pour voir combien d'erreurs cela crée pour m'aider à comprendre combien de travail cela va être. Selon ce que j'ai l'intention de faire, je pourrais le laisser comme ça jusqu'à ce que j'aie résolu tous ces problèmes, puis finaliser la modification en supprimant le code commenté.
JimmyJames

J'ai plusieurs bases de code sur lesquelles je travaille qui sont jonchées de commentaires TODO . Honnêtement, ce n'est pas si mal parce que c'est généralement 1-2 lignes. Ce que j'aime dans les commentaires TODO, c'est que mon IDE a un onglet "TODO" près du terminal qui se remplit automatiquement avec cette liste de commentaires, avec un aperçu du commentaire et le numéro de fichier / ligne. Le point est, il est utile quand dans une entreprise en particulier , ils ne gasp utiliser les questions même si elles utilisent Git / GitHub. Ouais, que pouvez-vous faire, essayez de convaincre la direction d'utiliser Git Issues au lieu de Google Sheets? Ouais, essayé et échoué. Tant pis. TODO commente c'est!
Chris Cirefice

6

Puis-je demander quels sont ces inconvénients de ce que je fais que je ne vois pas maintenant?

Sans doute, si vous travaillez seul et n'utilisez pas le contrôle de version et que vous pensez que vous pouvez le faire de cette façon.

En fait, sans contrôle de version, peu importe ce que vous faites à tout moment dans le "temps" car le code est toujours quel que soit l'état dans lequel le fichier actuel est "enregistré" comme dans le système d'exploitation.

Si vous utilisez le contrôle de version, et que vous avez une charge de commentaires en tant que votre "liste de tâches", et que vous en corrigez et en supprimez le commentaire, puis répétez, puis répétez etc ... vous avez alors le code "work in progress" et les commentaires enregistrés dans votre historique de révision. Ce ne sera pas un problème si vous n'avez jamais besoin de revenir à un autre commit ou même à un "choix de cerise" plus tard (c'est par exemple là que vous prenez des commits spécifiques et les tirez dans une autre branche à utiliser). Mais sinon, cela pourrait être un problème.

On peut dire que cela peut être comparé aux "clichés" du logiciel du disque dur, comme Windows (la chose Restore). S'il prend un instantané contenant un virus, vous tuez le virus, mais vous devez ensuite revenir en arrière, vous pouvez revenir à un point où le virus est à nouveau présent.

Cette approche est également susceptible d'être un problème lorsque vous utilisez le contrôle de version et travaillez avec d'autres développeurs, car ils doivent alors voir votre liste de tâches qui ne leur est d'aucune utilité. En fait, c'est juste de l'encombrement qu'ils doivent ignorer et contourner. Dans nos équipes, nous supprimons toujours tous les commentaires, tels que l'ancien code ou les "notes". À moins qu'ils ne soient utiles - cependant, cela est très rare car nous avons une documentation pour "comment cela fonctionne", et un logiciel pour suivre ce qui doit être fait (alias todo).

De plus, lorsque vous travaillez sur un projet plus important, vous avez tendance à collaborer, à vous engager et à pousser souvent, il est donc possible que leur branche sur laquelle ils travaillent ait votre liste TODO s'ils ont fusionné votre branche avec la leur. Alors votre liste TODO est l'affaire de tous: D

En résumé, si vous ne travaillez pas seul et surtout lorsque vous utilisez le contrôle de version, il encombre l'historique et peut être encombré par d'autres développeurs.

Et, c'est une chose personnelle à certains égards, mais utiliser une base de code comme "liste de tâches" n'est pas vraiment idéal. Un jour, vous pourriez laisser quelque chose par accident, ou oublier de le commenter ou de le commenter par erreur.


Comme pour de nombreuses approches de l'architecture, du codage et de la façon dont vous-même ou votre équipe travaillez, chaque scénario peut exiger quelque chose de différent. Considérez donc les inconvénients mentionnés et les avantages de l'utilisation du contrôle de version, et décidez si cela fonctionne pour vous .


C'est pourquoi vous travaillez sur des branches d'entités et utilisez des fusions écrasées. Le code de travail en cours ne devrait jamais être vu par un autre développeur, donc peu importe la méthode utilisée pour le développer.
Jules

4

Il semble que votre critique soit un peu dogmatique. Je ne suis pas sûr de jurer sur quelqu'un d'avoir commenté le code est un PC ;-), ou même utile ;-)

Mais plus sérieusement, je pense que votre critique a raison, que vous devriez sérieusement envisager d'utiliser git (ou un autre système de contrôle de source, mais git est un choix judicieux).

Et cela peut alléger certains de vos besoins de commenter le code.

Mais avoir une liste TODO dans le code (qu'il s'agisse de listes à puces ou d'un ancien code) - est tout à fait raisonnable à mon avis. Mais vous voudrez peut-être reconsidérer un peu la façon dont vous le faites. Pour une chose - je suggère de penser à quelqu'un d'autre lisant votre code. Que quelqu'un d'autre puisse être vous, un an après avoir abandonné et rejoint un projet. Ou ce pourrait être quelqu'un d'autre entièrement. JUSTE rencontrer du code commenté est un peu déroutant. Peut-être quelque chose comme:

/*
 * Need this sort of functionality added back before too long:
 * .... OLD CODE HERE
 */

Personnellement, je penche davantage vers quelque chose comme ça:

 * TODO:
 *      @todo   Possible get rid of intermediate LRUCache_ object.
 *
 *      @todo   Find some reasonable/simple way to get
 *              LRUCache<PHRShortcutSpec, PHRShortcutSpec, PHRShortcutSpecNoAuthCacheTraits_>   sRecentlyUsedCache (kMaxEltsInReceltlyUsedCache_);
 *              Working with ONE T argument
 *              Add(elt2cache).
 ...

et je me sens libre d'ajouter des "extraits de code" de l'ancien code comme utile.

Utiliser git est difficile (malheureusement). Il vous faudra un certain temps pour apprendre, et il peut sembler que cela ne fait pas partie de ce que vous essayez d'accomplir. Mais si vous voulez programmer de manière utile, vous devrez apprendre à le faire en tant que membre d'une équipe et à communiquer avec une équipe, et git est exactement la façon dont cela se fait de nos jours. Et une fois que vous aurez commencé à l'utiliser, vous trouverez que c'est un outil / béquille TRÈS utile, facilitant le développement de votre logiciel.

Bonne chance!


2

Il existe de nombreuses raisons de commenter le code: -

  • Ce n'est pas encore correct, et vous ne le commenterez pas quand il sera prêt.
  • Vous le commentez temporairement pour modifier le comportement lors du débogage.
  • Vous n'êtes pas sûr si le code est nécessaire, mais vous ne voulez pas le supprimer avant d'en avoir testé davantage.
  • Le code est nécessaire pour certaines versions du logiciel, mais pas pour celle-ci.
  • Le code est obsolète, mais il a fallu des siècles pour l'écrire et vous y êtes émotionnellement attaché. D'ailleurs, cela pourrait être utile un jour.

Le problème survient lorsque vous mettez le code au lit, puis y revenez quelques années plus tard pour effectuer une maintenance. Vous trouverez la base de code jonchée de code commenté. Il ne sera plus clair pourquoi tout cela est là, et c'est maintenant juste encombré.

Si vous utilisez un outil de contrôle de version à moitié décent, vous pouvez supprimer audacieusement tout code dont vous n'avez plus besoin, sachant que le système de contrôle de version le conserve encore. Un fichier différent entre les versions révélera ce qui a été supprimé. Une fois que vous avez implémenté le contrôle de version, le seul besoin de commenter est pour les choses temporaires. Si vous trouvez du code commenté dans des fichiers sur lesquels vous ne travaillez pas, vous pouvez simplement le supprimer.


1
Ce sont exactement les raisons pour lesquelles vous devriez plutôt utiliser un SCM (et des cranches en son sein)
Timothy Truckle

2

Je ne vais pas répéter pourquoi vous devriez utiliser le contrôle de code source même pour des projets à une personne, car il existe de nombreuses autres ressources qui vous diront de le faire. Mais un inconvénient majeur de votre approche actuelle est que si vous commentez le code, vous le cachez à votre IDE (si vous n'utilisez pas un IDE, vous devriez probablement le considérer).

Par exemple, si vous voulez renommer une méthode ou une classe, ou changer le nombre de paramètres qu'une méthode prend, votre IDE devrait avoir une option de refactorisation pour ce faire qui trouvera toutes les références appropriées et les mettra à jour en conséquence - mais il a probablement gagné '' t rechercher dans les commentaires.

Au lieu d'essayer de deviner où vous devez apporter des modifications, faites-les simplement et laissez votre IDE vous dire où vos modifications ont provoqué la rupture. Vous pouvez ensuite commenter le code de manière plus chirurgicale et, espérons-le, pendant une période plus courte avant de réparer ce qui s'est cassé.


1

C'est mauvais et tu devrais arrêter .

La raison se résume à essayer de faire une grande quantité de refactoring en une seule fois.

Si vous commentez de grandes sections de code, corrigez un peu et archivez, vous avez alors archivé du code non fonctionnel. et un tas de trucs commentés que les autres considéreront comme vieux et peuvent être ignorés

Si vous ne vous enregistrez pas souvent, vous accumulez des conflits de fusion et n'enregistrez pas les progrès pas à pas.

Vous devez changer votre pratique de travail afin que si vous devez vous arrêter à mi-chemin, tout fonctionne toujours.

Faites quelques pas et enregistrez-vous après chaque:

  • extraire une interface
  • écrire un test
  • refactoriser une fonction

Ne marquez pas de gros morceaux de code comme étant les vôtres en les commentant et en les emportant pour travailler par vous-même jusqu'à ce qu'ils soient terminés ou que vous échouiez.

Si vous devez garder une trace de ce qui doit être fait, utilisez un tableau de tâches comme Scrum ou Trello

En utilisant notre site, vous reconnaissez avoir lu et compris notre politique liée aux cookies et notre politique de confidentialité.
Licensed under cc by-sa 3.0 with attribution required.