V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
intmax2147483647
V2EX  ›  职场话题

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

  •  1
     
  •   intmax2147483647 · 11 天前 · 6578 次点击

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

    39 条回复    2021-04-11 23:24:46 +08:00
    nianyu
        1
    nianyu   11 天前
    lz 好人一生平安
    weimo383
        2
    weimo383   11 天前 via Android
    外企吗
    wszgrcy
        3
    wszgrcy   11 天前 via Android
    ………一般被 review 的时候,什么方面的比较多?
    ericls
        4
    ericls   11 天前 via iPhone   ❤️ 2
    呃 不要开了这个先例…… 该谁做就谁做
    Lemeng
        5
    Lemeng   11 天前
    辛苦了
    iSteven
        6
    iSteven   11 天前
    楼主明天上班请到我办公室领取一张好人卡 [狗头]
    djs
        7
    djs   11 天前   ❤️ 1
    羡慕有人 review 代码,我们都是瞎写,写瞎了也没人说
    ysicing
        8
    ysicing   11 天前
    好人一生平安。为啥不是他们修改呢?
    tojonozomi
        9
    tojonozomi   11 天前
    啊这

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

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

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