git - code review是什麼？

Question

新手剛入門，能否簡單講一下code review是什麼？怎麼進行code review？
謝謝

Answer 1

Code Review, 指代碼審閱；場景：新人或初級工程師，做了一些改動，但是需要在老程式設計師的確認後再添加至主分支或特性分支；不可能讓老程式設計師坐你旁邊看程式碼嗎，這樣太浪費時間了；

舉例：
A計畫有兩個分支master分支和dev分支；现在公司在开发，所有人完成任务，都应该把分支合并至dev分支；

張三在自己的本地開發環境，基於dev分支，创建zhang_san分支；花了三天，完成一個功能；

老程式設計師覺得張三是新手，不放心就這麼將他的改動直接放進dev分支；就讓張三提個Merge申請，讓他dev分支；就让张三提个Merge申请，让他Code Review ;

於是張三就將自己的zhang_san分支提到远程服务器上，操作“New Merge Request”，发起一个Merge请求，将zhang_san合并至dev分支，指定審閱人是老程式設計師；

老程式設計師找了個空閒時間，查看這個Merge Request,覺得有點問題, 於是就提了一些建議（提的建議會自動發送到相關人的關聯郵箱）；

張三後來透過郵箱查看到老程式設計師提的建議，在自己本地zhang_san进行了修改，并git push至远程zhang_san分支；

這時的「Merge Request」會更新最新的Commit；老程序员再看这次总的改动，发现不错，挺好的，点击确认请求后，就成功将zhang_san分支上的改动合并至dev分支啦；【到這就完成了Code Review】

張三通過郵件，知道自己的分支已經被合併進開發分支dev很開心；可以刪除這個本地zhang_san分支和遠端zhang_san分支，保持分支乾淨；也可以繼續在zhang_san分支做新的開發，提交新的Merge_Request,進行新一輪的dev很开心；可以删除这个本地zhang_san分支和远程zhang_san分支，保持分支干净；也可以继续在zhang_san分支做新的开发，提交新的Merge_Request,进行新一轮的Code Review。

Answer 2

code review 就是程式碼審查，你可以百度搜尋一下，有很多文章

Answer 3

你寫的程式碼，別人看，別人的程式碼，你也看

review 過了，再上線

Answer 4

程式碼審查，一般大公司都需要code review。如果程式碼有問題或不符合規範，修改正確後再上線。

Answer 5

要你命三千：老程式碼中的那些坑