84669 人學習
152542 人學習
20005 人學習
5487 人學習
7821 人學習
359900 人學習
3350 人學習
180660 人學習
48569 人學習
18603 人學習
40936 人學習
1549 人學習
1183 人學習
32909 人學習
新手剛入門,能否簡單講一下code review是什麼?怎麼進行code review? 謝謝
认证高级PHP讲师
Code Review, 指代碼審閱;場景:新人或初級工程師,做了一些改動,但是需要在老程式設計師的確認後再添加至主分支或特性分支;不可能讓老程式設計師坐你旁邊看程式碼嗎,這樣太浪費時間了;
Code Review
舉例:A計畫有兩個分支master分支和dev分支;现在公司在开发,所有人完成任务,都应该把分支合并至dev分支;
master
dev
張三在自己的本地開發環境,基於dev分支,创建zhang_san分支;花了三天,完成一個功能;
zhang_san
老程式設計師覺得張三是新手,不放心就這麼將他的改動直接放進dev分支;就讓張三提個Merge申請,讓他dev分支;就让张三提个Merge申请,让他Code Review ;
Merge
於是張三就將自己的zhang_san分支提到远程服务器上,操作“New Merge Request”,发起一个Merge请求,将zhang_san合并至dev分支,指定審閱人是老程式設計師;
老程式設計師找了個空閒時間,查看這個Merge Request,覺得有點問題, 於是就提了一些建議(提的建議會自動發送到相關人的關聯郵箱);
Merge Request
張三後來透過郵箱查看到老程式設計師提的建議,在自己本地zhang_san进行了修改,并git push至远程zhang_san分支;
git push
這時的「Merge Request」會更新最新的Commit;老程序员再看这次总的改动,发现不错,挺好的,点击确认请求后,就成功将zhang_san分支上的改动合并至dev分支啦;【到這就完成了Code Review】
Commit
張三通過郵件,知道自己的分支已經被合併進開發分支dev很開心;可以刪除這個本地zhang_san分支和遠端zhang_san分支,保持分支乾淨;也可以繼續在zhang_san分支做新的開發,提交新的Merge_Request,進行新一輪的dev很开心;可以删除这个本地zhang_san分支和远程zhang_san分支,保持分支干净;也可以继续在zhang_san分支做新的开发,提交新的Merge_Request,进行新一轮的Code Review。
Merge_Request
code review 就是程式碼審查,你可以百度搜尋一下,有很多文章
你寫的程式碼,別人看,別人的程式碼,你也看
review 過了,再上線
程式碼審查,一般大公司都需要code review。如果程式碼有問題或不符合規範,修改正確後再上線。
要你命三千:老程式碼中的那些坑
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 就是程式碼審查,你可以百度搜尋一下,有很多文章
你寫的程式碼,別人看,別人的程式碼,你也看
review 過了,再上線
程式碼審查,一般大公司都需要code review。如果程式碼有問題或不符合規範,修改正確後再上線。
要你命三千:老程式碼中的那些坑