web-dev-qa-db-fra.com

Conseils sur le patron de persuader que la revue de code est une bonne chose

Disons que l'on travaille dans une entreprise hypothétique qui a plusieurs développeurs qui travaillaient rarement ensemble sur des projets et que le patron ne croyait pas que les critiques de code valent le temps et le coût.

Quels sont divers arguments pouvant être présentés dans ce scénario qui représenteront le bénéfice de l'examen du code? En outre, quels sont les arguments potentiels contre l'examen du code ici et comment peuvent-ils être contré?

20
Kevin D

Si vous devez vous justifier de manière basique, vous avez un problème plus important.

Vous êtes l'expert, votre équipe doit décider des pratiques que vous utilisez. Peut-être devriez-vous commencer à convaincre votre patron de ce principe très important.

Votre patron est censé décider QU'EST-CE QUE Pour faire et plus important POURQUOI le faire. Vous devriez prendre soin de la COMMENT Construire

(cela ne signifie pas que vous ne pouvez pas suggérer quoi et pourquoi faire des choses dans votre entreprise bien sûr). Un grand patron devrait encourager ses employés à participer à la stratégie d'entreprise)

Cependant, voici comment je vois des critiques de code par pairs:

Étant donné que la programmation est un travail intellectuel très intensif, une personne ne peut pas s'assurer que tout est parfait. Par conséquent, l'examen du code garantit que:

  • les vulnérabilités ou les bugs se trouvent avant l'expédition de l'application
  • l'éducation mutuelle constante entre les développeurs (presque gratuitement) est atteinte
  • code Respect Standard pour faciliter la maintenance de l'application
  • code correspondent aux exigences

Tout le monde en prend des avantages directs:

  • le développeur qui augmente ses connaissances et peut passer la sienne à ses coéquipiers
  • le client/utilisateur qui a moins de bugs et dépense moins en maintenance
  • le patron qui a plus de clients/utilisateurs heureux et dépensez moins en formations
25
user2567

L'examen du code peut obtenir plusieurs développeurs familiers avec le même code. C'est une bonne chose. Et si l'auteur d'origine décide de cesser de fumer ou d'aggraver, quelque chose de mal lui arrive. Si des critiques de code sont effectuées régulièrement, d'autres peuvent prendre rapidement en charge.

Les pairs sont peut-être capables de repérer les bugs potentiels ou les problèmes de performance pendant l'examen du code. Cela réduit l'assurance qualité et le développement. Cela peut compenser les coûts supplémentaires impliqués dans les critiques de code.

CODE Avis Promouvoir le partage des connaissances. Les pairs peuvent raconter de meilleures façons ou des façons d'alternance de faire des choses. J'ai moi-même appris beaucoup de mes pairs à travers des critiques de code.

Code Reviews Aidez renforcer les directives de codage suivies de l'équipe. Si l'équipe n'en a pas, cela doit être corrigé. Le code est destiné à être écrit une fois et lu plusieurs fois. Les directives de codage sont une étape vers le code lisible. Le code est destiné à être lisible par des pairs. Quel meilleur moyen que d'avoir des critiques de code pour assurer la lisibilité?

7
aufather

De Steve McConnel's Construx l'analyse de rentabilisation pour de meilleures pratiques logicielles et fruit de logiciel de développement de logiciels de développement (LHF) adresse cette adresse. De ces derniers "LHF qui ne sera pas résisté par les inspections de la direction de la haute direction".

0
Brian Carlton

Lorsque vous le présentez, concentrez-vous sur la plus grande image.

Énumérez les avantages (meilleur code, moins de bugs, moins réécrit, etc.) et mentionnez un examen de code comme n des techniques que vous recommanderiez.

Je ferais cela partie d'une image plus grande de faire de l'artisanat de logiciels

  • code Reviews
  • tests
  • rétrospectives
  • le partage des connaissances
  • éducation
  • les critiques de livres
  • conférences sur le déjeuner

Soyez prêt à faire beaucoup de travail vous-même dans la promotion de ces principes.
[.
[.____] Vous devriez construire le cas au fil du temps avec calme et systématiquement. Lorsque vous êtes le plus ennuyé par des insectes qui auraient été corrigés par de meilleures techniques, c'est souvent le pire moment à faire votre cas car vous êtes plus susceptible d'être trop émotionnel et moins rationnel. Cela pourrait sembler un peu contre-intuituve, mais c'est ce que j'ai appris plus de 30 ans de programmation. Évidemment ymmv.

0
Michael Durrant