git - code review是什麼?
PHP中文网
PHP中文网 2017-05-02 09:49:24
0
5
1299

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

PHP中文网
PHP中文网

认证高级PHP讲师

全部回覆(5)
小葫芦

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

淡淡烟草味

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

phpcn_u1582

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

review 過了,再上線

过去多啦不再A梦

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

Ty80

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

熱門教學
更多>
最新下載
更多>
網站特效
網站源碼
網站素材
前端模板