这是一个创建于 1645 天前的主题，其中的息可能已经有所发展或是发生改变。
最近组内招了几个新选手，因为经验不足吧，写的代码质量不是很好，再加上有部分在别的团队的 repo 里面做修改，在被 review 代码的时候收到了好多“建议”... 哎只能我来加班帮忙重构一下啦，好累啊
 | 1 nianyu 2021-04-08 21:36:40 +08:00 lz 好人一生平安 |
 | 2 weimo383 2021-04-08 21:39:55 +08:00 via Android 外企吗 |
 | 3 wszgrcy 2021-04-08 21:53:59 +08:00 via Android ………一般被 review 的时候，什么方面的比较多？ |
 | 4 ericls 2021-04-08 23:22:26 +08:00 via iPhone  2 呃 不要开了这个先例…… 该谁做就谁做 |
 | 5 Lemeng 2021-04-08 23:24:40 +08:00 辛苦了 |
 | 6 iSteven 2021-04-08 23:32:03 +08:00 楼主明天上班请到我办公室领取一张好人卡 [狗头] |
 | 7 djs 2021-04-08 23:41:58 +08:00 1 羡慕有人 review 代码，我们都是瞎写，写瞎了也没人说 |
 | 8 ysicing 2021-04-08 23:42:52 +08:00 好人一生平安。为啥不是他们修改呢？ |
 | 9 tojonozomi 2021-04-08 23:46:11 +08:00 啊这 和里面的新选手干相反的事情 刚入职不久，已经在重构项目的部分代码了 真就屎山耕耘 |
 | 10 fpure 2021-04-09 00:04:33 +08:00 via Android 重构为什么要加班？又不是线上紧急生产事故，有必要搞到深夜吗？ |
 | 11 MYDB 2021-04-09 00:19:06 +08:00 6 不要有奋斗比迹象哦，很多 955 公司都是有人开头加了班：） |
 | 12 anxn 2021-04-09 00:47:42 +08:00 via Android 4 这不就是“卷”的开端吗 |
 | 13 cnrting 2021-04-09 01:12:51 +08:00 via iPhone 我是不提倡楼主这种做法，出了 bug 算谁的？为自己的事业奋斗到深夜也就罢了，你这是太把自己在公司里当回事了 |
 | 14 xupefei 2021-04-09 01:19:39 +08:00 via iPhone 你帮他们改好了，他们不就无法提升水平了吗 |
 | 15 Mutoo 2021-04-09 06:44:06 +08:00 要 review 也是上班 review 呀，为啥要加班。 |
 | 16 zjsxwc 2021-04-09 08:17:56 +08:00 via Android 同意楼上几位说的，谁写的 bug 谁去改，给人擦屁股只有 0 次和无数次，你给他们修 bug 他们付钱给你了吗？ |
 | 17 bleepbloop 2021-04-09 09:06:38 +08:00 楼主心态放正，工作的事情不要带到工作时间外，否则慢慢你就会变成工贼的样子 |
 | 18 amwyyyy 2021-04-09 09:08:18 +08:00 这是他们最好的成长机会，应该让他们来。 |
 | 19 zm8m93Q1e5otOC69 2021-04-09 09:14:26 +08:00 没必要，直接驳回就行了，他们不会念你的好 |
 | 20 DefoliationM 2021-04-09 09:15:05 +08:00 好人啊 好人 |
 | 21 RainCats 2021-04-09 09:57:26 +08:00 拒绝加班，到点下班，又不是生产出问题 |
 | 22 lakehylia 2021-04-09 10:12:33 +08:00 重构会出 bug，吃力不讨好。谁的代码谁清楚一点。 |
 | 23 drunkdog 2021-04-09 10:15:13 +08:00 老实人楼主 |
 | 24 SlipStupig 2021-04-09 10:20:09 +08:00 这么干真心不好，自己代码写不好应该自己去解决，这样以后才知道写出 BUG 来就要熬到半夜，以后做事会谨慎一些，从描述上来说，贵公司可能 CI 机制不完善，完善的 CI 机制应该会把很多垃圾代码给干掉，建议加强测试用例和关键流程设计，这样大家都好 |
 | 25 nowto 2021-04-09 10:27:23 +08:00 via Android 把他们叫在一起，你重构他们看着，这样他们才有长进 |
 | 26 way2create 2021-04-09 10:34:46 +08:00 我这边情况是 自己倒是做完了 别人加班老是 @ 我。。。产品也不干事 |
 | 27 xunxuntj 2021-04-09 10:47:07 +08:00 要加班也是他们自己加啊 谁都没义务做救世主，自己的屁股自己擦 |
 | 28 twoconk 2021-04-09 10:49:15 +08:00 楼主是好人！ |
 | 29 windyCity1 2021-04-09 10:53:06 +08:00 @djs #7 被 review 代码压力其实还蛮大的，特别是刚入职的时候。。。。。。 |
 | 30 zakokun 2021-04-09 11:12:37 +08:00 你这种就是牺牲自己，恶心别人。 为啥要半夜改？明天早起要上线吗？ 别人的代码，有修改意见了，为什么你来改？他们自己不知道下次还犯这个问题你再帮他们改？团队还有进步吗？ 你大半夜改别人代码，别人啥感受？ |
 | 31 konakona 2021-04-09 11:40:00 +08:00 2 = = 我晕啊，评审员不需要改代码的好吧，最好也不要改，除非你就是那个项目组的前端或者后端，如果你是 team leader 你更不应该改。 1. 你不了解具体业务逻辑 2. 开发人员的心智模式你不清楚，所以你看到的只是他思考后的结果代码，而原因只有他清楚，你做评审的时候如果看不懂就要在里边问，但绝对不是你去改，改坏了你要花更多时间去解决 3. 要加班也是新来的加班啊，因为不熟悉业务和代码熟练度不高 |
 | 32 johnsonqrr 2021-04-09 12:55:04 +08:00 不合适吧= = |
 | 33 zw1one 2021-04-09 13:44:03 +08:00 你改完这次，等下次他们写代码的时候，又有问题了怎么办 |
| 34 zw1one 2021-04-09 13:45:17 +08:00 @djs 我上家公司，例会的时候我经常提开发没有做 code review 。好嘛，现在每周安排了一个下班后的 code review |
 | 35 Lee2019 2021-04-09 13:55:46 +08:00 via Android 楼主不要卷了，给同事一点活路 |
 | 36 gbin 2021-04-09 15:24:21 +08:00 看标题还以为说我呢, 我是为了跨时区协作晚上工作, 上午慢慢才来上班. |
 | 37 coder9778 2021-04-11 12:21:19 +08:00 谁写的代码谁改啊，不要帮别人改代码，哪怕是一小行改变。 |
 | 38 18500592934 2021-04-11 19:08:42 +08:00 虽然你是出于好心，但这个事情办的真的是损人不利己，话如果重了的话，可以换成“好心帮倒忙”吧 我还有个地方比较好奇，那就是：如果生产上你重构的一段代码出问题了，这要算谁的责任？ |
 | 39 callmexiaodeng 2021-04-11 23:24:46 +08:00 那为啥不让写的人改 ，为啥不上班做？ |
 | 40 thunderstormhit 2021-06-17 08:17:38 +08:00 via Android 楼主人太好了 你应该在 review 中直接当头棒喝指出他们的问题 几次 review 下来屡教不改那种新人 直接 pip 甭客气 |
Select Language