V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
jakezh
V2EX  ›  程序员

遇到沙雕同事怎么办

  •  
  •   jakezh · 2019-09-24 02:36:26 +08:00 · 5893 次点击
    这是一个创建于 1648 天前的主题,其中的信息可能已经有所发展或是发生改变。

    一个中东的 senior,写了个 100 行代码的程序让我 review 100 行里面找出来 15 个 issue

    我说改完了我再 approve

    这傻吊居然跟 manager 说

    I don’t think we are building the next generation of google services… I can live with how this is right now.

    然后手工做了个效果图,趁 manager 今天 WFH,偷偷跟 manager 汇报,manager 居然就 tmd 同意了。。。。。。。 我看到后想联系 manager 一直没联系上。。。。。。

    懒得撕逼了,希望早点上线打他的脸。。。 这种人都能做 senior。。。。。。看来这破公司是没啥前途 准备滚了

    30 条回复    2019-09-25 11:09:16 +08:00
    nuistzhou
        1
    nuistzhou  
       2019-09-24 03:48:38 +08:00 via iPhone
    你在 issues 里面 @一下 manager 呗,起码以后说起来有凭有据
    lbfeng
        2
    lbfeng  
       2019-09-24 05:13:26 +08:00
    伊朗?
    ericgui
        3
    ericgui  
       2019-09-24 05:53:17 +08:00
    啥公司?让兄弟们也避免采坑
    andrewpsy
        4
    andrewpsy  
       2019-09-24 06:16:38 +08:00   ❤️ 1
    不要说早点上线打他脸,如果有证据是你没卡住让垃圾上线了,你的脸也不保。
    这种事情越早 escalate 到上级越好。
    fuermosi777
        5
    fuermosi777  
       2019-09-24 06:20:46 +08:00
    没看太懂,他啥意思?不想解决你提出的 review ?
    dangyuluo
        6
    dangyuluo  
       2019-09-24 06:40:42 +08:00
    4 楼说的对,这种事不要自己解决,直接 explicitly 捅到上面,留下书面证据。

    另外让他 git signoff 一下,然后你留言说明问题。

    这种事情在国内建筑行业见得多了。有监理去施工单位检查的时候,施工方一个劲给好处要监理签字,这时候就考验你推卸责任的能力了。
    terence4444
        7
    terence4444  
       2019-09-24 07:01:38 +08:00 via iPhone
    你有权不 approve,manager 同意了让 manager approve 去。
    whoamiatvip
        8
    whoamiatvip  
       2019-09-24 07:51:32 +08:00
    100 行 15 个 issue,可以。
    这事不用考虑,抓紧上报。
    k9982874
        9
    k9982874  
       2019-09-24 08:02:48 +08:00 via iPhone
    你也可以往上捅啊,manager merge 了,你就发信给 manager 说明情况,随便 cc cto,留下书面证据以后撕逼用
    cmdOptionKana
        10
    cmdOptionKana  
       2019-09-24 08:07:23 +08:00
    你都准备离开公司了,就不要怕事情闹大了
    jakezh
        11
    jakezh  
    OP
       2019-09-24 08:39:30 +08:00
    missmimimi
        12
    missmimimi  
       2019-09-24 08:42:35 +08:00
    虽然没看太懂 但是评论看懂了 hhh
    justrand
        13
    justrand  
       2019-09-24 08:43:46 +08:00
    为什么有种似曾相识的感觉?在别的平台发过么?
    nvioue
        14
    nvioue  
       2019-09-24 09:02:46 +08:00 via Android
    啥 楼主在谷歌上班???大佬好!!!
    hereIsChen
        15
    hereIsChen  
       2019-09-24 09:51:05 +08:00
    100 行 15 个
    也真的太牛了
    notreami
        16
    notreami  
       2019-09-24 11:18:25 +08:00
    大家不能盯着 100 行 15 个,这种数字。
    15 个 issue 是什么?要是变量名规范、换行规范啥的、日志规范啥的,怕不是真有让人觉得你在刁难我胖虎
    fvckDaybyte2
        17
    fvckDaybyte2  
       2019-09-24 11:23:25 +08:00 via iPhone
    同意啥了……不 building the next generation of google services?
    bccoder
        18
    bccoder  
       2019-09-24 12:21:47 +08:00 via Android
    大佬大佬
    Hanggi
        19
    Hanggi  
       2019-09-24 13:44:29 +08:00
    你们用啥 review 啊?
    userdhf
        20
    userdhf  
       2019-09-24 14:00:23 +08:00
    I can live with how this is right now. 这句话该怎么理解?
    deanguqiang
        21
    deanguqiang  
       2019-09-24 14:06:29 +08:00 via iPhone   ❤️ 1
    @userdhf 大意是我们又不是造原子弹,现在这样凑合也没事
    hhhsuan
        22
    hhhsuan  
       2019-09-24 14:17:44 +08:00
    楼主在国外?
    pkookp8
        23
    pkookp8  
       2019-09-24 14:18:17 +08:00 via Android
    @hereIsChen 15 个其实还好吧。就我而言,很多应该是
    单词换一个说法
    空格,换行
    日志级别
    注释
    换一种更通用的写法
    之类的
    真正的代码问题其实不多
    wuweijia
        24
    wuweijia  
       2019-09-24 14:53:48 +08:00
    @userdhf 就是差不多说我觉得没啥问题啊,可以合并。
    userdhf
        25
    userdhf  
       2019-09-24 14:58:02 +08:00
    @deanguqiang
    @wuweijia
    谢谢大佬
    原来是句读
    I can live ( with ( how this is ) right now ).
    zek
        26
    zek  
       2019-09-24 15:05:30 +08:00
    愣是没看懂楼主发的帖子
    vcode
        27
    vcode  
       2019-09-24 15:11:13 +08:00
    看不懂。。。
    jakezh
        28
    jakezh  
    OP
       2019-09-24 19:56:43 +08:00
    @notreami #16
    大佬 这些问题有 lint 脚本自己跑, 不需要人工 review 的

    我挑的都是些逻辑 /err handling/锁 之类的问题
    15 个是总数,一共打回去三次 第四次胖虎不开心了
    ericgui
        29
    ericgui  
       2019-09-25 02:34:36 +08:00
    @zek
    @vcode 我看懂了,这个在外企或者国外很普遍
    SmiteChow
        30
    SmiteChow  
       2019-09-25 11:09:16 +08:00
    很正常 做好本职工作就是 我以前在一个公司被一个东欧程序员小窗说 don't want touch any code of yours

    当时他在做某个模块的重构,言下之意就是嫌弃我 code 垃圾呗 我只回了句 its your business,me too
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   5889 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 28ms · UTC 02:29 · PVG 10:29 · LAX 19:29 · JFK 22:29
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.