在 955 的公司,却加班到深夜

intmax2147483647 1月前 28

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

最新回复 (39)
  • nianyu 27天前
    引用 2
    lz 好人一生平安
  • weimo383 27天前
    引用 3
    外企吗
  • wszgrcy 27天前
    引用 4
    ………一般被 review 的时候,什么方面的比较多?
  • ericls 27天前
    引用 5
    呃 不要开了这个先例…… 该谁做就谁做
  • Lemeng 27天前
    引用 6
    辛苦了
  • iSteven 27天前
    引用 7
    楼主明天上班请到我办公室领取一张好人卡 [狗头]
  • djs 27天前
    引用 8
    羡慕有人 review 代码,我们都是瞎写,写瞎了也没人说
  • ysicing 27天前
    引用 9
    好人一生平安。为啥不是他们修改呢?
  • tojonozomi 27天前
    引用 10
    啊这

    和里面的新选手干相反的事情

    刚入职不久,已经在重构项目的部分代码了

    真就屎山耕耘
  • fpure 27天前
    引用 11
    重构为什么要加班?又不是线上紧急生产事故,有必要搞到深夜吗?
  • MYDB 27天前
    引用 12
    不要有奋斗比迹象哦,很多 955 公司都是有人开头加了班:)
  • anxn 27天前
    引用 13
    这不就是“卷”的开端吗
  • cnrting 27天前
    引用 14
    我是不提倡楼主这种做法,出了 bug 算谁的?为自己的事业奋斗到深夜也就罢了,你这是太把自己在公司里当回事了
  • xupefei 27天前
    引用 15
    你帮他们改好了,他们不就无法提升水平了吗
  • Mutoo 27天前
    引用 16
    要 review 也是上班 review 呀,为啥要加班。
  • zjsxwc 27天前
    引用 17
    同意楼上几位说的,谁写的 bug 谁去改,给人擦屁股只有 0 次和无数次,你给他们修 bug 他们付钱给你了吗?
  • bleepbloop 27天前
    引用 18
    楼主心态放正,工作的事情不要带到工作时间外,否则慢慢你就会变成工贼的样子
  • amwyyyy 27天前
    引用 19
    这是他们最好的成长机会,应该让他们来。
  • beichenhpy 27天前
    引用 20
    没必要,直接驳回就行了,他们不会念你的好
  • DefoliationM 27天前
    引用 21
    好人啊 好人
  • RainCats 27天前
    引用 22
    拒绝加班,到点下班,又不是生产出问题
  • lakehylia 27天前
    引用 23
    重构会出 bug,吃力不讨好。谁的代码谁清楚一点。
  • drunkdog 27天前
    引用 24
    老实人楼主
  • SlipStupig 27天前
    引用 25
    这么干真心不好,自己代码写不好应该自己去解决,这样以后才知道写出 BUG 来就要熬到半夜,以后做事会谨慎一些,从描述上来说,贵公司可能 CI 机制不完善,完善的 CI 机制应该会把很多垃圾代码给干掉,建议加强测试用例和关键流程设计,这样大家都好
  • nowto 27天前
    引用 26
    把他们叫在一起,你重构他们看着,这样他们才有长进
  • way2create 27天前
    引用 27
    我这边情况是 自己倒是做完了 别人加班老是 @ 我。。。产品也不干事
  • xunxuntj 27天前
    引用 28
    要加班也是他们自己加啊
    谁都没义务做救世主,自己的屁股自己擦
  • twoconk 27天前
    引用 29
    楼主是好人!
  • windyCity1 27天前
    引用 30
    @djs #7 被 review 代码压力其实还蛮大的,特别是刚入职的时候。。。。。。
  • zakokun 27天前
    引用 31
    你这种就是牺牲自己,恶心别人。
    为啥要半夜改?明天早起要上线吗?
    别人的代码,有修改意见了,为什么你来改?他们自己不知道下次还犯这个问题你再帮他们改?团队还有进步吗?
    你大半夜改别人代码,别人啥感受?
  • konakona 27天前
    引用 32
    = = 我晕啊,评审员不需要改代码的好吧,最好也不要改,除非你就是那个项目组的前端或者后端,如果你是 team leader 你更不应该改。
    1. 你不了解具体业务逻辑
    2. 开发人员的心智模式你不清楚,所以你看到的只是他思考后的结果代码,而原因只有他清楚,你做评审的时候如果看不懂就要在里边问,但绝对不是你去改,改坏了你要花更多时间去解决
    3. 要加班也是新来的加班啊,因为不熟悉业务和代码熟练度不高
  • johnsonqrr 27天前
    引用 33
    不合适吧= =
  • zw1one 27天前
    引用 34
    你改完这次,等下次他们写代码的时候,又有问题了怎么办
  • zw1one 27天前
    引用 35
    @djs 我上家公司,例会的时候我经常提开发没有做 code review 。好嘛,现在每周安排了一个下班后的 code review
  • Lee2019 27天前
    引用 36
    楼主不要卷了,给同事一点活路
  • gbin 27天前
    引用 37
    看标题还以为说我呢, 我是为了跨时区协作晚上工作, 上午慢慢才来上班.
  • coder9778 27天前
    引用 38
    谁写的代码谁改啊,不要帮别人改代码,哪怕是一小行改变。
  • 18500592934 27天前
    引用 39
    虽然你是出于好心,但这个事情办的真的是损人不利己,话如果重了的话,可以换成“好心帮倒忙”吧
    我还有个地方比较好奇,那就是:如果生产上你重构的一段代码出问题了,这要算谁的责任?
  • callmexiaodeng 27天前
    引用 40
    那为啥不让写的人改 ,为啥不上班做?
  • 游客
    41
返回