git - code review是什么？

Question

新手刚入门，能否简单讲一下code review是什么东西？怎么进行code review？
谢谢

Answer 1

Code Review, 指代码审阅；场景：新人或者初级工程师，做了一些改动，但是需要在老程序员的确认后再添加至主分支或特性分支；不可能让老程序员坐你旁边看代码吗，这样太浪费时间了；

举个例子：
A项目有两个分支master分支和dev分支；现在公司在开发，所有人完成任务，都应该把分支合并至dev分支；

张三在自己的本地开发环境，基于dev分支，创建zhang_san分支；花了三天，完成一个功能；

老程序员觉得张三是新手，不放心就这么将他的改动直接放进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,进行新一轮的Code Review。

Answer 2

code review 就是代码审查，你可以百度搜索一下，有很多文章

Answer 3

你写的代码，别人看，别人的代码，你也看

review 过了，再上线

Answer 4

代码审查，一般大公司都需要code review。如果代码有问题或者不符合规范，修改正确后再上线。

Answer 5

要你命三千：老代码中的那些坑