在 955 的公司，却加班到深夜😩

[intmax2147483647]

最近组内招了几个新选手，因为经验不足吧，写的代码质量不是很好，再加上有部分在别的团队的 repo 里面做修改，在被 review 代码的时候收到了好多“建议”... 哎只能我来加班帮忙重构一下啦，好累啊

[RainCats]

拒绝加班，到点下班，又不是生产出问题

[lakehylia]

重构会出 bug，吃力不讨好。谁的代码谁清楚一点。

[drunkdog]

老实人楼主

[SlipStupig]

这么干真心不好，自己代码写不好应该自己去解决，这样以后才知道写出 BUG 来就要熬到半夜，以后做事会谨慎一些，从描述上来说，贵公司可能 CI 机制不完善，完善的 CI 机制应该会把很多垃圾代码给干掉，建议加强测试用例和关键流程设计，这样大家都好

[nowto]

把他们叫在一起，你重构他们看着，这样他们才有长进

[way2create]

我这边情况是 自己倒是做完了 别人加班老是 @ 我。。。产品也不干事

[xunxuntj]

要加班也是他们自己加啊
谁都没义务做救世主，自己的屁股自己擦

[twoconk]

楼主是好人！

[windyCity1]

@djs #7 被 review 代码压力其实还蛮大的，特别是刚入职的时候。。。。。。

[zakokun]

你这种就是牺牲自己，恶心别人。
为啥要半夜改？明天早起要上线吗？
别人的代码，有修改意见了，为什么你来改？他们自己不知道下次还犯这个问题你再帮他们改？团队还有进步吗？
你大半夜改别人代码，别人啥感受？

[konakona]

= = 我晕啊，评审员不需要改代码的好吧，最好也不要改，除非你就是那个项目组的前端或者后端，如果你是 team leader 你更不应该改。
1. 你不了解具体业务逻辑
2. 开发人员的心智模式你不清楚，所以你看到的只是他思考后的结果代码，而原因只有他清楚，你做评审的时候如果看不懂就要在里边问，但绝对不是你去改，改坏了你要花更多时间去解决
3. 要加班也是新来的加班啊，因为不熟悉业务和代码熟练度不高

[johnsonqrr]

不合适吧= =

[zw1one]

你改完这次，等下次他们写代码的时候，又有问题了怎么办

[zw1one]

@djs 我上家公司，例会的时候我经常提开发没有做 code review 。好嘛，现在每周安排了一个下班后的 code review

[Lee2019]

楼主不要卷了，给同事一点活路

[gbin]

看标题还以为说我呢, 我是为了跨时区协作晚上工作, 上午慢慢才来上班.

[coder9778]

谁写的代码谁改啊，不要帮别人改代码，哪怕是一小行改变。

[18500592934]

虽然你是出于好心，但这个事情办的真的是损人不利己，话如果重了的话，可以换成“好心帮倒忙”吧
我还有个地方比较好奇，那就是：如果生产上你重构的一段代码出问题了，这要算谁的责任？

[callmexiaodeng]

那为啥不让写的人改 ，为啥不上班做？

[thunderstormhit]

楼主人太好了 你应该在 review 中直接当头棒喝指出他们的问题 几次 review 下来屡教不改那种新人 直接 pip 甭客气