浅谈利用Code Review来增加自己在组内的影响力和Visibility

大家都不给comments  就一个人反复给大量的 “此处应有一空行”  “这个logging phrase的不够好”

所有人都知道他工作量不饱和了

哇， 帖子里的干货满满， 点赞！

raikkonenzhong 发表于 2019-2-20 06:17
关于code review有几个问题，抛出来跟楼主和大家讨论。
1）关于花在code review的时间和产出比：本人所在 ...

- 对我来说code review是很好的学习手段，用来熟悉自己scope以外的context，或者大神的design/coding技巧。所以虽然我觉得对impact没有直接的用处，还是间接有用。
- 在没有特别忙的时候，我会有10-20%的cr是我不熟悉的diff。会比容易review的diff花更多时间，最近看的一个这种diff看了一两天。时间久了review效率会高很多，其实也帮助自己的productivity。我发现自己在review复杂的diff最多的那阵以后coding速度暴增。
- 尽量每个code review能提一点design上的建议/问题，对自己对别人都有提高。当然真的很简单很好的diff直接给过就完了。
- 虽然review是服务，但是认真做大家还是appreciate的。我有多个同事都在给我的feedback里感谢我认真给他们review了，我给别的同事的feedback里也提了ta对我code review很严对我帮助很大。

我个人的经历感觉是 - CR 是一件很顺其自然的事，就是说会随着你工作经验和眼光见识的增长很自然的越来越丰富。
刚入职的时候什么都不懂自然comment不出什么有价值的东西 - 再努力去憋也憋不出来 - 所以这个时候也没必要鼓足了劲儿为了刷visibility而给意见
反之现在见的懂得多点儿了 很容易随便看看就能给不少 还在点子上

所以其实影响力和visibility什么的说到底还是你对架构，业务逻辑，code base了解多少吧 - 大家心悦诚服的也主要是这个方面吧

想起了之前别人提过的一个有趣的问题：一个公司的CTO该不该写代码？如果写代码的话谁来review？

Cats881119 发表于 2019/02/20 08:08:07
大家都不给comments  就一个人反复给大量的 “此处应有一空行”  “这个logging phrase的不够好”

所有人都知道他工作量不饱和了

非常同意！review还是要有点深度才行哒。

tbu 发表于 2019/02/20 10:04:07
我个人的经历感觉是 - CR 是一件很顺其自然的事，就是说会随着你工作经验和眼光见识的增长很自然的越来越丰富。
刚入职的时候什么都不懂自然comment不出什么有价值的东西 - 再努力去憋也憋不出来...

我很喜欢顺其自然这个词。同时我觉得顺其自然也有一定的先决条件，首先组里本身要有好的review习惯，其次我们各人也要有积累学习的心态。
我希望职场新人们读完帖子能意识到code review的一丢丢重要性
职场大神们读完给我很多实际操作中的建议。
再次给其他新人们推荐Clean Code这本书，他不一定说的都对，但可以给新人一些头绪

14417335 发表于 2019/02/20 06:07:51
赞文章。同意其它楼层提到的心态的正确。

楼主一开始提到的“自己又没有那么多时间去仔细研究这个User Story到底是要做什么的”。

我觉得很多时候就是因为时间的有限。自己也有事情已经需要...

感谢你的回复和提问
我想问问看你们组的架构和设计是不是在每个story开始之前就做得比较完备了？如果是的话我绝对支持专心于架构和设计。
我所在的组存在架构没理清就开始写代码的问题，所以有时候会需要利用CR来反过来引导设计。其实是比较low的一个现状。。。
时间紧的情况下就全看priority啦，老板说做什么我们能升职加薪我们就做什么 ：）。前提是老板有脑子

基本上3,4等级的会觉得review别人的code本身并没有自己的产出那么重要，等到了6及以上我觉得是eng、tech lead、tech manager每天花时间比自己coding时间更多的地方了。毕竟3，4写的code很可能没考虑到一些方面，有建设性的意见、疑问是至关重要的

看大神们的回复学了很多东西！多谢多谢！！