Comment donner des commentaires après le processus de révision du code


10

Je suis en train de passer en revue le code des développeurs juniors qui viennent de rejoindre mon équipe, je me demande comment dois-je fournir le résultat de cette revue:

  1. Dois-je réparer le code moi-même?

  2. Dois-je leur faire part de leurs commentaires sur le processus de révision et les laisser effectuer les corrections conformément à mes instructions? Et si oui, comment puis-je donner cette rétroaction, dois-je remplir un certain modèle de document et le lui envoyer, ou il existe un logiciel qui m'aidera à marquer les choses avec des problèmes dans les fichiers de code où ils pourront plus tard le vérifier? (J'utilise Visual Studio).

Après avoir passé en revue le code et les correctifs, puis un certain temps s'est écoulé et certaines parties du code que j'ai examiné dans le passé ont changé, comment dois-je faire le processus de réexamen? Dois-je revérifier tout le code à nouveau? Ou dois-je simplement vérifier les pièces qui ont changé? Et si oui, comment puis-je garder une trace des pièces qui ont été modifiées afin d'éviter la double révision du code?


Réponses:


14

Réponse courte

Dois-je réparer le code moi-même?

Non.

Dois-je leur faire part de leurs commentaires sur le processus de révision et les laisser effectuer les corrections conformément à mes instructions?

Oui. Selon vos suggestions , pas les instructions . Les instructions semblent trop autoritaires.

Et si oui, comment puis-je donner cette rétroaction, dois-je remplir certains documents de modèle et les envoyer à eux, ou il existe un logiciel qui m'aidera à marquer les choses avec des problèmes dans les fichiers de code où ils pourront plus tard le vérifier? (J'utilise Visual Studio).

Utilisez un outil pour donner votre avis. Vous pouvez utiliser Visual Studio.

Longue réponse

J'utilisais Visual Studio mais je devais constamment demander à d'autres développeurs: "Hé, pouvez-vous m'envoyer votre travail pour que je puisse le revoir?" Je n'aimais pas ça et ça n'a jamais vraiment bien marché. Maintenant, j'utilise l' assistant d'examen car je peux commencer un examen en consultant les enregistrements. Je n'ai pas besoin de me fier à un autre développeur pour m'envoyer du travail à réviser. Cela m'aide également à marquer les éléments comme des défauts, ou simplement à marquer les éléments à regarder par l'auteur. Cela fonctionne pour notre équipe car une fois que nous commençons une révision, elle reste là sur le tableau de révision et ne se perd pas dans la traduction. Ceci est intégré à Visual Studio. Comme je l'ai mentionné, Visual Studio a également son processus de révision natif mais je trouve qu'il a des limites et le processus n'est pas naturel; par conséquent, j'utilise Review Assistant.

Cet outil facilite également les allers-retours, les discussions, etc.

Le processus est plus ou moins le suivant:

Je passe en revue quelque chose, puis je l'envoie à l'auteur (dans votre cas junior dev). Ils apportent des modifications, puis ils le renvoient. J'examine les changements et je fais des commentaires. Si je suis bon avec les changements, je ferme l'examen. Sinon, ça va et vient. Parfois, s'il y a trop de va-et-vient, je vais simplement me diriger vers leur bureau et utiliser un tableau blanc - cela accélère vraiment le processus.

Les révisions de code sont un domaine sensible, alors soyez très prudent avec le choix du libellé. Je ne dis jamais à personne

Mauvais choix de formulation

J'ai examiné votre code et vous devez modifier certains éléments.

Je dis plutôt ceci:

Meilleur choix de formulation

J'ai regardé votre code et j'ai besoin d'aide. Pouvez-vous revoir les articles que je vous ai envoyés et voir si vous pouvez clarifier certaines de mes questions?

Cela fait penser à l'auteur:

  1. J'ai besoin d'aide pour qu'ils ne passent pas en mode défensif.
  2. On dirait qu'ils sont le critique, pas moi. Techniquement parlant, puisque je leur demande de revoir et de changer certaines choses, ils sont en quelque sorte comme le critique.

Ces choix de mots simples m'ont énormément aidé.

Je ne sous-estime jamais les développeurs juniors. J'ai travaillé avec des développeurs seniors (plus de 10 ans d'expérience) et le code était pire qu'un étudiant coop junior. Ce n'est donc pas si important parce qu'ils sont seniors ou juniors; leur travail est vraiment ce qui parle plus fort que des années d'expérience.

Souvent, pour encourager les développeurs juniors et les faire participer aux revues, je leur enverrai quelque chose à réviser pour moi: quelque chose qu'ils peuvent comprendre, quelque chose qu'ils trouveront difficile mais pas totalement au-delà d'eux. Je peux le dire comme ci-dessous:

Pouvez-vous s'il vous plaît examiner un code pour moi parce que je ne peux pas le comprendre.

Cela m'aide beaucoup à nouveau. Cela aide parce que cela montre clairement que je ne suis pas le seul à réviser, mais ils font également des révisions et ils font également partie du processus. Cela montre que l'idée est de produire un bon code propre et de demander de l'aide si nécessaire. Le processus d'examen est une culture, nous devons donc vraiment y travailler.

Maintenant, certaines personnes peuvent craindre que si elles font ce qui précède, les développeurs juniors perdront leur respect parce qu'ils ont juste fait quelque chose que vous ne pouviez pas faire. Mais c'est loin de la vérité: demander de l'aide montre de l'humilité. De plus, il existe de nombreuses situations dans lesquelles vous pouvez briller. Enfin, si telle est votre peur, vous avez des problèmes d'estime de soi. Enfin, peut-être que je ne le sais vraiment pas: je veux dire que certains de ces développeurs ont des algorithmes frais dans leur tête parce qu'ils les ont étudiés il y a un mois.

Quoi qu'il en soit, retour aux juniors et critiques. Vous devriez voir le regard sur leurs visages quand ils le découvriront et m'enverront une réponse. Je peux alors leur dire: "OK, laissez-moi faire les changements et si vous en êtes satisfait, veuillez fermer le problème."

Ils se sentent super d'avoir le pouvoir de regarder mon travail et de dire: "Oui, vos changements sont bons. J'ai clos le problème."

Je ne corrige jamais le code moi-même car:

  1. L'auteur n'en tirera aucune leçon.
  2. C'est comme si je disais: "Écarte-toi, laisse-moi te montrer comment c'est fait. Mon code est meilleur que le tien."
  3. Pourquoi aurais-je? C'est plus de travail pour moi.

Mais je proposerai des idées et des extraits de code dans mes commentaires pour aider l'auteur. Veuillez noter que parfois mes critiques demandent simplement à l'auteur que je ne comprends pas leur code; il n'y a peut-être rien de mal avec leur code. Ils peuvent avoir besoin de changer les noms des variables, d'ajouter des commentaires, etc. Ainsi, je ne saurai même pas quoi changer dans ce cas; seulement ils le feront.

Si vous continuez à faire des révisions, vous aurez tôt ou tard une très bonne idée du niveau de connaissance de chaque développeur de l'équipe. Le savoir est vraiment utile et vous devez en profiter et le libérer. Voici comment: si je passe en revue du code et que je vois des domaines d'amélioration évidents et que je sais qu'un autre développeur peut également les attraper, je les ferai le réviser à la place. Quelque chose comme "Hé, je vois quelques domaines qui peuvent être améliorés. Pouvez-vous s'il vous plaît l'examiner plus en détail et envoyer vos commentaires à l'auteur?" Cela fonctionne aussi très bien car maintenant j'ai 2 autres développeurs qui travaillent ensemble.

Si je passe en revue certains travaux et que je remarque quelque chose dont toute l'équipe peut bénéficier, je vais créer un scénario hypothétique et expliquer le problème lors d'une réunion. Je vais commencer par expliquer le scénario et demander à tout le monde s'ils peuvent trouver le problème ou voir un problème, les impliquer. Demandez à tout le monde de poser des questions. Enfin, présentez une meilleure approche. Si quelqu'un d'autre a une meilleure approche, je les remercie et je reconnais devant l'équipe que leur approche est meilleure. Cela montre que je ne suis pas un type de personnalité «à ma façon ou sur l'autoroute». De plus, je n'ouvre JAMAIS le travail de quelqu'un et je commence à souligner les problèmes lors d'une réunion, l'auteur ne l'appréciera pas - peu importe à quel point je suis gentil et inoffensif.

Quand je fais des revues, je ne cherche pas seulement un bon code propre mais aussi ce que fait le code. Si l'exigence commerciale est la suivante: si un employé travaille dans l'entreprise depuis plus de 10 ans, accordez-lui une augmentation de 5%. Sinon, 2,5% . La première chose que je vérifie est de savoir si c'est effectivement le cas. Ensuite, je vérifie s'il le fait de manière propre, cohérente et performante.

Si je fais un examen, je m'assure de faire un suivi ou personne ne prendra les examens au sérieux.

Il y a des années, je travaillais avec quelqu'un qui ferait une revue et ferait un cc au responsable du développement et au responsable de l'assurance qualité, mais les deux managers venaient d'un milieu commercial ou avaient peu d'expérience en développement. Il n'a fait cela que pour les impressionner. Personne n'a aimé et c'est à ce moment-là que je me suis dit que je ne ferais jamais cette erreur.

L'autre chose qu'il avait l'habitude de faire était de choisir le style de programmation et était convaincu que son style était le meilleur ("Mon kungfu est bien meilleur que votre style singe ..."). C'était une autre leçon pour moi: il y a toujours plus d'une façon de résoudre un problème.

Répondez à certaines de vos questions numérotées

1- Dois-je réparer le code moi-même?

Non, veuillez consulter les raisons que j'ai mentionnées ci-dessus.

2- Dois-je leur donner des commentaires sur le processus de révision et les laisser faire les corrections selon mes instructions?

Oui, essayez d'utiliser des phrases et un ton amical mais qui nécessitent encore de l'attention. Soyez aussi clair que possible. Indiquez quel est le problème avec le code, comment l'améliorer. Ne demandez pas simplement de le changer. Mais donnez des raisons.

comment puis-je donner cette rétroaction, dois-je remplir certains documents de modèle et les envoyer à eux, ou il existe un logiciel qui m'aidera à marquer les choses avec des problèmes dans les fichiers de code où ils pourront plus tard le vérifier? (J'utilise Visual Studio).

Comme je l'ai dit, vous pouvez utiliser l'outil que j'utilise ou un autre outil. N'utilisez pas de documents électroniques ou Word car ils se perdent et il est difficile de les suivre.

Après avoir passé en revue le code et les correctifs, puis un certain temps s'est écoulé et certaines parties du code que j'ai examiné dans le passé ont changé, comment dois-je procéder pour réexaminer le code? dois-je revérifier tout le code?

La plupart du temps, je vérifie le delta (modifications uniquement). Mais vous devez avoir une vue d'ensemble pour vous assurer que rien n'est cassé et qu'il suit l'architecture.

Dernières pensées

Personnellement, je pense que le mot «révision de code» est un mauvais choix et je ne sais pas comment il a commencé. Ils auraient pu choisir un mot bien meilleur et moins autoritaire.


Cela ne fonctionnerait que pour de petites choses comme les noms de variables. Cependant, si l'architecture est incorrecte, aucun outil ne peut vous aider. Il vous suffit de jeter le tout et de le réécrire.
t3chb0t

@ t3chb0t Pourquoi de petites choses? Par making sense architecturally, je voulais dire que le code de la couche d'accès aux données se trouve dans la couche d'accès aux données, la validation de l'interface utilisateur est dans l'interface utilisateur, etc. En d'autres termes, s'assurer que les préoccupations ne se propagent pas dans d'autres domaines. Il existe des outils pour aider à conserver l'architecture; en fait VS le fait maintenant hors de la boîte maintenant . Nous l'utilisons également.
CodingYoshi

En ce qui concerne les outils, je pense qu'un outil parfaitement valable à utiliser est la forme de logiciel de billetterie / de suivi du travail que vous utilisez probablement déjà pour suivre ce qui doit être fait. Par exemple, si vous utilisiez Github, vos modifications pourraient toutes être associées à un problème, puis l'examen serait effectué dans ce fil de discussion. Jira et Trac sont deux autres outils de ce type. Cela conserve un emplacement centralisé pour toutes les informations liées au travail et il est clairement visible avant la fermeture du ticket.
Kat

@Kat Je ne sais pas si un outil de billetterie est le bon outil à utiliser ici. Les outils de révision de code montrent la différence entre les changements et les problèmes sont différents des problèmes sur un système de billetterie; ils signifient des choses différentes. Mais je peux me tromper.
CodingYoshi

3

Cela dépend beaucoup de la façon dont vous comprenez les revues de code dans votre entreprise. Il y a des entreprises où la révision du code est un processus hautement formalisé qui a rarement lieu mais qui est un gros problème. À d'autres, la révision du code est une partie obligatoire de chaque tâche qui est implémentée, et est une chose très banale et rapide avec peu de formalisme. Personnellement, j'opte pour cette dernière approche, mais cela peut ou non être votre décision si vous pouvez l'utiliser ou non.

J'ai écrit un article de blog intitulé Les revues de code valent-elles votre temps? pour résumer le processus utilisé par mon équipe. Les points à retenir, en ce qui concerne votre question, seraient les suivants:

  1. Laissez les développeurs corriger le code. Cela leur permettra de mieux comprendre vos commentaires (ou de réaliser qu'ils ne les comprennent pas pleinement et ne demandent pas), et effectuer une tâche est une meilleure façon d'apprendre que de simplement lire à ce sujet.
  2. Le logiciel pour les revues de code est la voie à suivre. Il existe de nombreuses options disponibles, à la fois open-source et propriétaires. La plupart fonctionne avec git. Mon équipe utilise BitBucket (anciennement connu sous le nom de Stash), il y a aussi GitLab et GitHub, Gerrit (dont je ne suis personnellement pas fan) et un tas d'autres. La plupart de ces applications sont basées sur le Web, peu importe l'IDE que vous utilisez, mais il existe également des plugins pour de nombreux IDE, donc je suis sûr qu'il y en a aussi pour Visual Studio. Un logiciel comme celui-ci vous permet de revoir le code avant qu'il ne soit fusionné dans la branche principale (généralement via des demandes de tirage) et il montre les parties modifiées et certaines lignes de contexte autour de chaque changement. Cela rend la révision du code fluide et sans tracas.

En ce qui concerne le cycle Review-Fix-Check-The-Fix, ce que vous choisissez doit dépendre de la maturité des développeurs et de la gravité des problèmes que vous avez repérés. Une fois que les équipes font des révisions de code un processus quotidien, vous pouvez vous attendre à ce que des changements triviaux tels que renommer une classe, soient simplement appliqués et il n'est probablement pas nécessaire de les revérifier. Si l'équipe n'est pas encore vendue sur les revues de code ou que les gens sont vraiment inexpérimentés, vous voudrez peut-être vérifier ces choses malgré tout. D'un autre côté, si votre avis a repéré un problème de concurrence complexe que les développeurs juniors peuvent ne pas comprendre pleinement même après que vous le leur ayez signalé, vous feriez mieux de vérifier le correctif et de ne pas donner votre approbation au changement avant d'être sûr qu'il a vraiment été corrigé.

Les outils peuvent vous y aider, car si vous travaillez avec des demandes d'extraction, vous pouvez facilement configurer le logiciel pour ne pas autoriser la fusion d'un changement jusqu'à ce qu'il ait l'approbation d'un nombre prédéterminé de développeurs. Habituellement, vous pouvez afficher les modifications dans les validations individuelles d'une modification, ce qui vous permet de consulter facilement uniquement les modifications ajoutées depuis votre dernière série de commentaires.


1

Je vote pour la deuxième option. Les juniors peuvent avoir une meilleure "courbe de retard" lorsqu'ils effectuent eux-mêmes les changements.

Aussi: ne soyez pas le seul à revoir le code.
Permettez à certains des membres expérimentés de votre équipe de consulter également le code et de planifier une réunion d'examen régulière au cours de laquelle les examinateurs présentent leurs conclusions (qu'ils ont faites avant la réunion!) À l'auteur. Cela augmentera la motivation des juniors et des membres de l'équipe d'expérience.


4
Bon point. De plus, les juniors devraient "revoir" le code de l'OP et le code de l'autre.
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.