search

haizlin / fe-interview

前端面试每日 3+1，以面试题来驱动学习，提倡每日学习与思考，每天进步一点！每天早上5点纯手工发布面试题（死磕自己，愉悦大家），6000+道前端面试题全面覆盖，HTML/CSS/JavaScript/Vue/React/Nodejs/TypeScript/ECMAScritpt/Webpack/Jquery/小程序/软技能……
http://www.h-camel.com
MIT License
25.46k stars 3.26k forks source link

[软技能] 第16天 你在的公司有没有做代码审查（CodeReview）？如果有是怎么做的？如果没有你觉得应该怎么做才更好？ #53

Open haizhilin2013 opened 5 years ago

haizhilin2013 commented 5 years ago

第16天 你在的公司有没有做代码审查（CodeReview）？如果有是怎么做的？如果没有你觉得应该怎么做才更好？

Damon99999 commented 5 years ago

不知道您说的是哪一种codereview 1：提交代码会把代码link发群里，全员都可以进行codereview（大佬一定会过一遍）有不合理代码直接提comment改好了再合并 2：每周一次的codereview全员参加，指定两位至三位小伙伴将本周开发的内容拿出来全部过一遍，全员现场提问现场解答，时间大概1~3h

AricZhu commented 5 years ago

公司采用gerrit服务器进行代码的统一托管，一般组内开发人员把代码git push上去以后，需要有权限的人进行code review，并且通过，然后才能合代码进入生产服务器

seho-dev commented 5 years ago

小公司没有，但是越小公司我觉得越要搞这样的，因为能很提高代码水平，全员监督你的代码，是非常好的做法

xcLtw commented 5 years ago
  1. 有独立的代码审查部门，定期发送邮件给相关人员，里面有本部门全部项目的代码质量统计，在代码过差时依次向上级发通知
  2. 依据每个组内风格，有的组在每次合并生产环境都会review
  3. 总的来说代码审查是好事，但如果出现咸鱼池塘以及产品流程不规范导致迭代需求过多而不合理，会造成很多困扰，自身也可能流于形式，一定要结合实际情况来看
blueRoach commented 4 years ago

目前没有，不过确实有必要，以后会尝试做的 具体的，比如每周五下午抽几个小时把自己本周的代码过一遍。 1.肯定会发现很多可以优化的地方 2.也可以复习下，在写代码的时候查资料的一些没有掌握的知识

smile-2008 commented 4 years ago

不知道您说的是哪一种codereview 1：提交代码会把代码link发群里，全员都可以进行codereview（大佬一定会过一遍）有不合理代码直接提comment改好了再合并 2：每周一次的codereview全员参加，指定两位至三位小伙伴将本周开发的内容拿出来全部过一遍，全员现场提问现场解答，时间大概1~3h

Gun-yx commented 3 years ago

1.每天要把代码放进项目组里，组长会来检查，有问题的话会直接进行沟通

WangXi01 commented 2 years ago

以任务为导向，一般不做review