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

baz 发表于 2019-2-25 13:41
[mw_shl_code=cpp,true]#include
struct str_rd{
. check 1point3acres for more.    int len;

啊？我哪里想杠啦？
这个如果让我review的话 肯定会问说foo是做什么的啊？能不能改成更加expressive的function名？str_rd里面的c[0]具体代表什么？

有些大牛一眼就能看出来你分享的函数具体做什么的，但是大多数公司里面的小螺丝都不是大牛啊。那代码可读性不就变得重要了吗？上面分享的代码有多少新人敢去改？

baz 发表于 2019-2-25 13:41
[mw_shl_code=cpp,true]#include
struct str_rd{
    int len;

”但自己看不懂就让别人改的话 哈哈哈哈 服“
感谢你的指正，我这个确实说的比较招黑。这句话的本意是鼓励大家去做CR，不要太过担心自己提的可能弱/傻逼所以就止步不前。

normanwu1990 发表于 2019-2-26 10:46
”但自己看不懂就让别人改的话 哈哈哈哈 服“
感谢你的指正，我这个确实说的比较招黑。这句话的本意是鼓 ...

哈哈哈哈 明白了
确实有时候要无畏一点 感谢你的回复

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

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

哈哈，你的回复让我想到一个同事。月平均500 review comments.

作为一个工作经验不多而且来新公司业务逻辑也不是非常熟悉的萌新，我该怎么提高写PR comments的能力呢？要不要固定每天花30分钟看PR提高理解能力？我认识的一个同事出活很多还看了好多好多外组的PR，提的建议都很有用，怎么样能达到那样的水平？
虽然理智知道他们都是好意，但是同事给我写很多comments的时候我心里还是有点着急的。PR不能在我期待的时间上线，本来觉得已经快完成的活又要大改结构，心里些焦虑。如何克服这些负面情绪呢？

为了不影响timeline，或者不招人烦，我们以前常做的一件事是小问题让他过，但是comment下次PR一起改。

楼主空洞地说了大半天都是虚的，都是很肤浅的想法。新手吧？你这样轻的会被senior臭骂一顿，重的直接给你下个套等着pip吧。

code review差不多相当于做人，要be honest，要言之有物，一天到晚咋咋呼呼的那是给自己找麻烦。你们组的senior们给approval是因为自己不清楚business logic, 出于信任就approve了。你如果同样不清楚逻辑，不要胡乱review。

Code review的一个（或者说最重要的）功能是找bug，你找不出bug整天在瞎蒙一些styling/refactoring/code quality之类的无关紧非常主观公说公有理婆说婆有理的事，new grad还允许，超过一年经验还无法帮助作者找bug反而还在纠缠这些细枝末节的东西，肯定就会有人开骂了(至少我就会骂)，你这是严肃code review还是恶意灌水？你到底看懂了还是在瞎写凑数？

Code review的另外一个目的是简化逻辑，包括但不限于严厉制止无聊的design pattern，限制无聊的inheritance 之类。你得精通语言特性, 言之有物。简化逻辑并不等于精简代码，要写可维护的代码。注意这和精简的代码是两回事。

至于通过code review来学习business logic，这明显是本末倒置。你首先要确实理解了business logic，才能有资格review代码。 你看不懂说明你不了解business logic。

想通过code reivew来展示自己的才能的想法基本上是**才会有。你找不出bug来，整天在code review里胡言乱语浪费大家时间，，production出了事你不也灰头土脸？

clean code那本书不能说纯属瞎扯淡，也差不多了。interview的时候有一些red flag, 比如读过man month myth, clean code, 精通design pattern之类的，一听到这些我一般会追问技术底层细节，一直到kernel/CPU level，基本必死无疑，无一例外。我想这些人的共性就是喜欢一些非逻辑的东西比如code smell之类的胡说八道，思维能力上无法handle真正的in-depth的东西。

随便写写，想到哪写到哪。

狂赞！！

匿名用户 发表于 2021-08-29 13:39:00
楼主空洞地说了大半天都是虚的，都是很肤浅的想法。新手吧？你这样轻的会被senior臭骂一顿，重的直接给你下个套等着pip吧。

code review差不多相当于做人，要be honest，要言之

膜拜大神 zszszs