Code Review, fait référence à la révision du code ; scénario : une nouvelle personne ou un ingénieur junior apporte des modifications, mais doit être confirmée par un programmeur expérimenté avant de les ajouter à la branche principale ou à la branche de fonctionnalités, il est impossible de les laisser faire ; un programmeur expérimenté s'assoit à côté de vous. Regardez le code, c'est une telle perte de temps
Par exemple : Le projet A a deux branches, la branche master et la branche dev ; maintenant l'entreprise se développe, et tout le monde doit fusionner la branche avec la branche dev une fois la tâche terminée
Zhang San a créé la branche
dans son environnement de développement local basé sur la branche dev ; il a fallu trois jours pour remplir une fonction zhang_san ;
L'ancien programmeur sentait que Zhang San était un novice et craignait de mettre ses modifications directement dans la branche
; il a demandé à Zhang San de soumettre une dev candidature et lui a demandé de Merge;Code Review
Ainsi, Zhang San a mentionné sa branche
sur le serveur distant, a opéré une "nouvelle demande de fusion", a lancé une requête zhang_san, a fusionné Merge avec la branche zhang_san et a désigné le réviseur comme étant M. Programmeur ;dev
L'ancien programmeur a trouvé du temps libre et a vérifié cela
Il a senti qu'il y avait quelque chose qui n'allait pas, alors il a fait quelques suggestions (les suggestions seront automatiquement envoyées à l'adresse e-mail correspondante de la personne concernée
Merge RequestZhang San a ensuite vérifié les suggestions faites par l'ancien programmeur via son courrier électronique, les a modifiées localement
et
les a transférées vers la branche zhang_san distante git push ;
zhang_sanÀ ce moment-là, la « Demande de fusion » sera mise à jour avec la dernière
; le programmeur vétéran a de nouveau examiné les changements généraux et a trouvé que c'était bon, très bon. Après avoir cliqué pour confirmer la demande, il a réussi. fusionné les modifications sur la branche
Les modifications ont été fusionnées dans la branche Commit [Ceci termine la révision du code] zhang_san ;
devZhang San est très heureux de savoir par e-mail que sa branche a été fusionnée avec la branche de développement
; il peut supprimer la branche
locale et la branche dev distante pour garder la branche propre ; continuez également à zhang_san Brancher pour de nouveaux développements, soumettez de nouveaux zhang_san et menez une nouvelle série de zhang_san. Merge_Request
Révision du code, généralement les grandes entreprises exigent une révision du code. S'il y a un problème avec le code ou s'il ne répond pas aux spécifications, modifiez-le correctement avant de vous connecter.
![[Web front-end] Démarrage rapide de Node.js](https://img.php.cn/upload/course/000/000/067/662b5d34ba7c0227.png)
Code Review, fait référence à la révision du code ; scénario : une nouvelle personne ou un ingénieur junior apporte des modifications, mais doit être confirmée par un programmeur expérimenté avant de les ajouter à la branche principale ou à la branche de fonctionnalités, il est impossible de les laisser faire ; un programmeur expérimenté s'assoit à côté de vous. Regardez le code, c'est une telle perte de tempsPar exemple :
Zhang San a créé la brancheLe projet A a deux branches, la branche
masteret la branchedev; maintenant l'entreprise se développe, et tout le monde doit fusionner la branche avec la branchedevune fois la tâche terminéedans son environnement de développement local basé sur la branche
L'ancien programmeur sentait que Zhang San était un novice et craignait de mettre ses modifications directement dans la branchedev; il a fallu trois jours pour remplir une fonctionzhang_san;; il a demandé à Zhang San de soumettre une
Ainsi, Zhang San a mentionné sa branchedevcandidature et lui a demandé deMerge;Code Reviewsur le serveur distant, a opéré une "nouvelle demande de fusion", a lancé une requête
L'ancien programmeur a trouvé du temps libre et a vérifié celazhang_san, a fusionnéMergeavec la branchezhang_sanet a désigné le réviseur comme étant M. Programmeur ;devIl a senti qu'il y avait quelque chose qui n'allait pas, alors il a fait quelques suggestions (les suggestions seront automatiquement envoyées à l'adresse e-mail correspondante de la personne concernée
etMerge RequestZhang San a ensuite vérifié les suggestions faites par l'ancien programmeur via son courrier électronique, les a modifiées localementles a transférées vers la branche
; le programmeur vétéran a de nouveau examiné les changements généraux et a trouvé que c'était bon, très bon. Après avoir cliqué pour confirmer la demande, il a réussi. fusionné les modifications sur la branchezhang_sandistantegit push;zhang_sanÀ ce moment-là, la « Demande de fusion » sera mise à jour avec la dernièreLes modifications ont été fusionnées dans la branche
; il peut supprimer la brancheCommit[Ceci termine la révision du code]zhang_san;devZhang San est très heureux de savoir par e-mail que sa branche a été fusionnée avec la branche de développementlocale et la branche
devdistante pour garder la branche propre ; continuez également àzhang_sanBrancher pour de nouveaux développements, soumettez de nouveauxzhang_sanet menez une nouvelle série dezhang_san.Merge_RequestLa révision de code est une révision de code, vous pouvez effectuer une recherche sur Baidu, il existe de nombreux articles
D'autres liront le code que vous écrivez, et vous lirez également le code des autres
Après examen, reconnectez-vous
Révision du code, généralement les grandes entreprises exigent une révision du code. S'il y a un problème avec le code ou s'il ne répond pas aux spécifications, modifiez-le correctement avant de vous connecter.
Des milliers de vies perdues : les pièges de l'ancien code