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

| 添加认证 | · 2022-4-20 21:23:11

注册一亩三分地论坛，查看更多干货！

您需要登录才可以下载或查看附件。没有帐号？注册账号

x

本人负责一个service，对里面的代码设计都有一些原因和思考吧，团队有人改代码，因为我对代码比较严格，所以会提很多comment，然后这个人后面有code review就不叫我看了，找一些没太大关系的人看，经常改出一些问题，引起一些TT。想知道这样的人多吗？老大太忙了，根本没时间管这些，但是我作为service owner看着很不爽啊，该怎么处理？

| 添加认证 | · 2022-4-20 21:35:09

这是Interview BQ的问题!
一个Code review comments太多时, 就不应该继续加comments了. 这时你该跟作者schedule 1:1 and start pair programming.

| 添加认证 | · 2022-4-21 00:43:00

提示: 作者被禁止或删除内容自动屏蔽

| 添加认证 | · 2022-4-21 01:00:30

我很appreciate那种insightful comments
因为review很花时间有那功夫自己写diff不好吗

可现实是，很多人留两个不痛不痒comment 加了blocking review
你回头找人两天不理你
好不容易回复了又加两个comment
来来回回经常一周就没了

我耗不起这时间下次直接绕过这种sb 你爱跟谁玩谁玩
作为reviewer一点职业素养没有

想要effective 就要自己maintain一个list里面是跟你打交道的靠谱service owner/reviewer 否则你的project就等着被block吧这也是重要技能之一

| 添加认证 | · 2022-4-20 21:56:08

只要能work就行，实在不爽就自己再改，在别人的change里面留一堆comments，别人也可以不爽
另外提交的code就是公司的财产，和你没啥关系

| 添加认证 | · 2022-4-20 22:34:34

本帖最后由匿名于 2022-4-20 08:14 编辑

我想问一下，如果说code review水水过，岂不是能快速增加impact？

只要不是重大bug，至于响应速度慢，质量差部分的问题，完全可以拖，顺便还增加on-call的解决次数，还能增加ticket

然后由于质量不算太高的代码导致接收你代码的人还经常问你，leadership+1

然后甩手一扔，换组升职跳槽，岂不美滋滋？
说白了就是如果遵循良好的code review和保证代码质量
1. 你让组员讨厌
2. 你影响了你们组的产出

这种真是血妈坑，看不懂要做高质量代码review的目的，正所谓，上有所好。就和最近那个NYU DS作弊爆料的一样。人家公司就喜欢这样简历和内推的，正经努力的看都不带看一眼的。与其举报，不如让这类公司都被烂货占领吧。 现在很多大企业在加速走下坡路是不争的事实，人家公司领导层都不在乎公司发展，我们当打工人的何必在意？？？

illumine

有时候会这样，因为觉得leave很多comments的那个人完全没道理 😂 但是通常我自己是service owner，觉得别人根本不懂还在那说一堆干嘛 ……… 如果别人是service owner，肯定不能饶过他找其他人review的。

lxxxxxxx

匿名用户发表于 2022-04-20 06:56:08
只要能work就行，实在不爽就自己再改，在别人的change里面留一堆comments，别人也可以不爽
另外提交的code就是公司的财产，和你没啥关系

。。。。。。。。。。。。。。。。。如果是我自己代码写得烂别人帮我指出了我为什么不爽？😂

lxxxxxxx

匿名用户发表于 2022-04-20 07:34:34
我想问一下，如果说code review水水过，岂不是能快速增加impact？

只要不是重大bug，至于响应速度慢，质量差部分的问题，完全可以拖，顺便还增加on-call的解决次数，还能增加ti

要是每个人都这么做甩手一扔那你就是从自己造的屎坑跳到别人造的屎坑里了啊😂 你跳sde3就已经需要你在一个组积累两三年了更别说更高的级别你如果愿意放低要求每天deal with屎坑代码那我只能佩服

| 添加认证 | · 2022-4-20 23:39:58

这个就是度的把握问题，有些时候，如果不影响功能又没有bug，我会放的。

lxxxxxxx

我们组一般都是service owner一定需要被included 除非是ta没那么多时间处理不过来而且大家都一般很配合也想找个权威的人double check啊对方如果都搞出很多问题了还不涨教训你完全可以跟manage说同时他不加你应该也加了team吧你主动去看就是了

反正第一步就是绿茶卖惨说问问manager那个人对你有没有什么feedback说你很喜欢ta但是感觉ta好像对你有什么意见这个service的cr都不发给你了结果怎么怎么样了然后问你manager是不是我review cr太严格了老臣心里也苦还是为了项目代码质量高 boo 既表达了自己的ownership跟insist on high standard又把皮球踢给老板

nickbear

我的经验是大改动必须经过owner，有开会讨论过或者design approval保证大方向没问题
小的比如coding style之类的东西无所谓，组里有coding standard就按那个来

地里匿名用户	推荐匿名用户-WKNVU \| 添加认证 \| 2022-4-21 00:43:00 提示: 作者被禁止或删除内容自动屏蔽
地里匿名用户	职场新人升级通关攻略新人在组里如何站住脚，快速上手，迅速成长，获得认可 intern如何拿到return offer
	回复支持 41 反对 0 使用道具举报

[职场感言] Code review故意找一个容易过的人看？

注册一亩三分地论坛，查看更多干货！

相关帖子

评分

评分

浏览过的版块