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

magicsets 发表于 2019/02/17 17:43:43
严谨的Code Review肯定是很重要的，但希望借Code Review提高影响力则有点本末倒置了。

如果自身资历不高不是TL不是模块Owner的话，那么需要明确一点Code Review是服...

非常感谢大神的分享！服务性 服务性 服务性 我会牢记在心。
从您分享的里面引申出来的话还有：1. 在code review的方式方法上注意自己的位置，不同资历具体做法也会不同。2. 如果是别人的模块，非常宝贝的东西，也要更加谨慎
我会注意不变成你提到的那种反面教材。我现在所在的组确实出现了没有大佬的问题。本来两个大佬一个变成了我们的manager，忙到飞起不做code review。另一个转职architect专心做架构，也没有时间来打理我们的code review。接下来包括我在内的三个lead技术水平参差不齐，如果不站出来做的话整体质量就会被最短的那块板决定。

hxtang 发表于 2019/02/18 02:00:32
my 2 cents
- CR贵在真诚，也就是不为了自己的visibility提不必要的request，也不为了避免得罪人就不提自己的concern。长期这样做你的cr才有分量。为visibilit...

非常同意！每个user story都要根据具体情况具体操作。写得好的地方一定要表扬！

bearjoe 发表于 2019/02/17 11:50:23
很好的经验总结。
进一步建议，要有stand on quality 的装逼觉悟，比如，要求多加复杂的test。一次在review同事的PR的时候，要求他增加一个closed course test来...

很棒的事例！好的code review就应该facilitate conversation，然后协同合作让双方共同进步。

你说的很有道理
不过大部分公司恐怕都是得罪人的事情，码农们本来都是互相不服

flymgz 发表于 2019/02/18 11:16:42
你说的很有道理
不过大部分公司恐怕都是得罪人的事情，码农们本来都是互相不服

具体操作中确实会有得罪到人，下面很多大神的回复也都很有帮助。我觉得我的标题可能改成“从亲身经历看code review的重要性”更加好 ：） 说到底我一开始的问题就是因为害怕得罪人而基本没有去做有意义的code review

本文被选为02/19/2019全站置顶文章之一。
作者获得大米奖励。谢谢你的分享

好的讨论，也会有加分的。跟今日话题一样待遇

这并不能提高你的影响力和能见度
从career development角度帮助也很有限
就算是Google这样注重engineering effort的公司也这样
更不用说其他公司了

赞文章。同意其它楼层提到的心态的正确。

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

我觉得很多时候就是因为时间的有限。自己也有事情已经需要加班，这种时候经常有不深究或者干脆不究。而把极其有限的时间限于探讨架构和设计。

不知道楼主在时间紧的情况下对于CR有什么tips？

关于code review有几个问题，抛出来跟楼主和大家讨论。
1）关于花在code review的时间和产出比：本人所在组里senior居多，所以大部分时候给他们的都是一些不疼不痒的style啊，typo这类的comments. 而且对context 了解有限，就会导致要仔细review 一个PR 会花比较久的时间。长期看势必有好处，但短期不如优先完成自己本职工作产出高。这点如何均衡？

2）PR会阶段性的密集到来，在不了解context 如何选择性的reivew 的 PR？

借楼求问一下：
层主是new grad，正巧加入的组 是新组只有我和TL两个人，code review基本也是他给我做，我不给他做。而且也不是提交PR来code review的，而是他定期跟我开会，来review我的code。
请问一下这种是正常的吗（对于新组），还是说我需要提一提意见 改变一下。。