刚入职,就被各种 code review,真的有必要吗? -云顶yd2223线路检测
59 个回答
code review应该是收费的,免费指导是给了你便宜。
我当时刚毕业的时候,去了微软,组里有个日本人,工作十几年了,态度非常严谨认真,很仔细的给我做code review,先不说功能方面,从注释到代码风格,事无巨细,一点一点的给我指出可以改进的地方。
我第一眼看到了也觉得烦,就这么几十行代码,这么多的comments,是隐含的说我代码水平不行吗?后来一想,我一个小破应届生,有什么拉不下面子的,就积极和他交流,慢慢改正。人家有些意见,的确是事出有因,能跑的代码,和product code,那还是真的不一样。
不敢说我从这件事里受益了多少吧,去年跳槽的时候,有个面试官也是十几年经验,先在google后在meta最后跳到我们公司的,当时我写完了题之后习惯性调整了一下格式,把一个用了两次的逻辑放到了helper function里,然后把一个分开写了几段的if/else if/else给调整成两段的if/else。后来跑出来功能方面没问题,这个面试官还和我说,我非常喜欢你这个代码风格。
我个人认为是有必要的,新入职的,要么是毕业生,要么是社招。
前者一般经验不足,写代码比较随意,属于不知道或者不太了解编码规范的,所以code review能够帮助毕业生从开始就打好基础,培养良好的编程习惯。
后者有一些已经有较好的编码规范,但是不同公司甚至同一公司的不同项目组的编码规范是不一样的,所以初期还是要适应一下;有一些则由于工作初期加入的没有编码规范的小公司,工作几年出来,完全没有遵守编码规范的意识,再加上编码习惯已经形成,积习难改,这种其实最需要code review来约束的。
所以总体来说,code review是必须的,不过执行过程要注意对事不对人,这是一个基本原则,也是比较难处理的一个点,因为人都是有主观倾向的,这其实对于管理者素质是有一定要求的。
当然有必要,而且很有必要。
在给你做code review的时候,特别是给你一行行代码提comment时候,其实是你学习的一个好机会。
没良心的才不帮你review呢。