Comment gérez-vous les situations où vous n'êtes pas d'accord avec l'évaluation de la revue de code?

Lorsque des désaccords surviennent lors de l’évaluation d’une revue de code, j’applique l’approche suivante :

1. Étudie attentivement le commentaire du relecteur : Je tente de comprendre la raison principale de sa remarque et la proposition d’amélioration. Il se peut que j’aie manqué quelque chose ou que son point de vue soit basé sur une autre expérience.

2. Analyse ma position : Je revois ma décision initiale et argumente pourquoi j’ai choisi cette approche. Il est important de formuler clairement mes arguments.

3. Discute directement la remarque : Au lieu d’être passivement en désaccord, j’initie une discussion avec le relecteur. Cela peut être une communication synchrone via un messager, une réunion ou un commentaire dans le système de revue de code. Mon objectif n’est pas de « gagner » la discussion, mais d’arriver à une solution optimale.

4. Propose des solutions alternatives : Si ma solution ne convient pas, je suis prêt à considérer d’autres options proposées par le relecteur ou à élaborer conjointement une nouvelle approche prenant en compte les deux points de vue.

5. Si nécessaire, je fais appel à un tiers : Si après discussion aucun consensus n’est atteint, je peux consulter un autre membre de l’équipe ou le chef d’équipe pour obtenir un avis indépendant.

Exemple de discussion dans le système de revue de code :

// Mon commentaire sur la remarque
// Bonjour ! Merci pour la revue. Je comprends votre point de vue concernant l’utilisation de this.findViewById.
// J’ai choisi cette approche pour des raisons de performance, car la hiérarchie des vues est peu profonde ici.
// Pouvez-vous expliquer pourquoi, dans ce cas précis, vous suggérez d’utiliser ViewBinding ?

Exemple de discussion si le relecteur propose une optimisation qui semble excessive :

Je présenterais les données sous forme de tableau, illustrant l’impact de l’optimisation proposée sur la performance dans ce cas précis par rapport au temps consacré à sa mise en œuvre.

Optimisation	Amélioration potentielle des performances	Temps de mise en œuvre
Solution initiale	Base	0
Optimisation proposée	Minime (environ 1-2 ms)	1 heure

En fin de compte, mon objectif principal est d’améliorer la qualité du code et de le rendre meilleur, et non simplement de défendre mon point de vue. Je suis ouvert aux critiques constructives et prêt à modifier ma décision si des arguments convaincants sont présentés.