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

fanghao 发表于 2019-2-20 06:49
. .и借楼求问一下：
层主是new grad，正巧加入的组 是新组只有我和TL两个人，code review基本也是他给我做，我 ...

个人愚见：
拿几个问题开头： TL给你的意见是不是有价值？TL写的代码是不是有senior的样子？
如果说TL确实是大牛，那就安心的在他手下学习就好。
如果说TL只是弱鸡，那就寻求改变看看吧。

那我们怎么判断他到底是大牛还是弱鸡呢？知识就是力量，看看好的开源代码，或者阅读一下相关的文献（Clean Code），都有可能给到你一些帮助。

asdfyou6 发表于 2019-2-20 04:39
这并不能提高你的影响力和能见度
从career development角度帮助也很有限
就算是Google这样注重engineering  ...

感谢来自google的大牛分享。 我很赞同code review起到的效果都不会是立竿见影，一针见血的。更多的是间接的，潜移默化的提升。
. 1point3acres我想求教大神分享一些提高影响力和能见度的好建议。一些更加直接，我可以快速成长的方法。

有个家伙bibi，已经被我告发了， escalate了两层，大家还是收敛一下吧，升职要靠自己，别想着走捷径

我还是觉得我们应该做我们认为是正确的事情，别太功利了。当然，在公司工作，期间最好可以跟manager沟通一下你的想法，取得他的同意我感觉也是必要的。少说多做，一般情况下，你的成绩总会被发现的。

normanwu1990 发表于 2019/02/21 05:12:16


感谢来自google的大牛分享。 我很赞同code review起到的效果都不会是立竿见影，一针见血的。更多的是间接的，潜移默化的提升。
我想求教大神分享一些提高影响力和能见度的好建议。一些更加...

直接做个impact大的project就promote了
我见过无数写烂代码但是照样升的很快的

normanwu1990 发表于 2019-2-21 05:07
个人愚见：
拿几个问题开头： TL给你的意见是不是有价值？TL写的代码是不是有senior的样子？
如果说TL ...

:) 有道理，多谢！

asdfyou6 发表于 2019-2-21 06:04
直接做个impact大的project就promote了
我见过无数写烂代码但是照样升的很快的

理解了，我这种小虾米可能要靠这些细节来建立信任才能拿到impact大的项目吧。。
大神有什么拿impact大的项目的技巧/经验吗？我怎么做可以更加多机会拿到？

guwenjun2007 发表于 2019-2-21 05:23
有个家伙bibi，已经被我告发了， escalate了两层，大家还是收敛一下吧，升职要靠自己，别想着走捷径

这就是为什么我们自己bibi以前一定要和老板打个招呼 ：）

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

很好的问题！
我尝试来回答一下
1）我觉得你已经回答了自己啦，长期来看势必有好处。那有哪些好处呢？ 我通过阅读大家的解答和身边一些朋友的交流感觉下面两点比较直接：a. 花时间多/深入理解的PR做得多了以后，PR本身会越做越快 b.花时间多/深入理解的PR做了多了以后，如果自己以后继承了这些代码，或者要在上面修一个Bug，都会上手快很多

对于短期和自己本职工作的产出进行平衡的角度来说，我建议和自己老板在1 on 1的时候聊聊看这个。自己的performance毕竟还是直接跟他的看法挂钩的。我知道这个帖子里面有不少认为PR没有那么多好处的坛友，我觉得跟老板和所在组的文化都是有关系的。 我个人是得到了老板的大力支持以后才放手去做，并且持续寻求指导的。我老板这么跟我说“your performance has nothing to do with your job description, I evaluate your performance based on your value to the team. It is great to see you putting effort on PR to improve the team.”.这个只是我的老板，我的经历。你的老板怎么想，他是不是把你做的story数量直接跟你的performance挂钩，需要你去了解。

2）PR会阶段性的密集到来，在不了解context 如何选择性的reivew 的 PR？
我个人会先找自己以后更有可能会去维护的PR，然后就是对scalability更有要求的PR。比如说一个是小Bug fix，或者feature tweak，另一个是新开的shared service framework。我肯定先看shared service framework。

个人愚见，欢迎指正！

楼主说的太对了 PR不能惯着 你做出了我一直想做却没做成的事