Code review故意找一个容易过的人看？

匿名用户 发表于 2022-04-20 22:55:38
让我想起我们senior经常两个comment最后得用句号结尾也要comment一下

恕我直言 这种人就是找抽

我觉得最sb的是那种，我是service owner，结果另一个tenure比我长的人过来在我的code下面一通评论，不懂business logic还问这问那，鸡毛蒜皮的小改动也要block。

Pizi-G 发表于 2022-04-20 10:21:28
屎山代码就是这么来的啊... 虽然代码和你无关, 但是你是这堆屎山的 owner, 还要天天在这堆 shit code 里工作, 这就和自己有关了啊
难道景区总有人随意大小便的解决方案不是 "不让人

同意同意。起码如果自己在组里待好久，就得努力避免屎山，不然下次改代码头疼的就是自己/自己组。

匿名用户 发表于 2022-04-20 22:02:12
看来很多人都有被reviewer挑刺的PTSD，哈哈哈。。。
楼主这情况应该技术上就可以解决，一般都可以设置必须owner的approval才能算signed off。
如果技术上都想尽办法了，那

很中肯的建议，我以后也会更注意点！谢谢。

adddaa 发表于 2022-04-20 22:12:32
寻求经验我让你发个review history有问题吗？凭什么啥信息不说，就一口咬定是别人的问题而不是自己吹毛求疵？就你这个态度，可以想象和你工作绝对不是什么愉快的体验。

你觉得上来张口骂人的，我还会向你寻求经验？我在帖子里已经收获足够的经验，人家也没让我贴代码啊。公司的代码我为什么要贴出来？我哪里写了一口咬定别人的问题？不好意思，我们组的人和我合作都挺愉快，这个人新来的而已，我还庆幸不用和你这样的人做同事呢，素质太低！

aliwithtaozi 发表于 2022-04-20 22:42:59
Due近 时间紧 你跟我讲这两个声明要换一下顺序 提高readability？我可qnmd吧

怎么YY出来的？上来骂人，**吧，有多远滚多远。

aliwithtaozi 发表于 2022-04-20 23:40:27
恕我直言 这种人就是找抽

我想问下，如果代码之间多余一行空白，只需要一行给了两行，要不要comment?我们组一般会……

匿名用户 发表于 2022-04-20 21:21:01
你的这个从test出发考虑的思路挺好！不过我就是觉得你每次主动去看，人家还故意不加，就很膈应吧

大多数人还是相对nice，对事关正确性的comments还是接受，就算不是自己加的reviewer，还渐渐的会视你为domain expert。

主要楼主也说了出了问题还是要你自己修，往自私了讲，这些comments不少都可以作为升职时候insightful comments的support，还有老板视不视你为这个service的guardrail。

有不少人推荐楼主把自己设成这个service包的required reviewer，如果是一个team makes sense，但对个人大概率会把自己累死也失去选择的余地，你也可能会成为bottleneck。但你可以把自己设成这个包的optional的reviewer，我看到过不少人干过。

pancakezzz 发表于 2022-04-21 02:01:40
大多数人还是相对nice，对事关正确性的comments还是接受，就算不是自己加的reviewer，还渐渐的会视你为domain expert。

主要楼主也说了出了问题还是要你自己修，往自私了讲，这

嗯嗯，话说给别人的cr review 的comments在晋升里面占的比重大吗？我可能之前也没太注意积累这些😂

匿名用户 发表于 2022-04-21 02:38:41
嗯嗯，话说给别人的cr review 的comments在晋升里面占的比重大吗？我可能之前也没太注意积累这些

一个比较容易勾掉的选项，平时收集几个就行了，临时收集容易想不起来。属于有几个就行，没有可能被卡，但升职肯定还是看各方面综合