312ybj
V2EX  ›  问与答

同事离职了,留下了屎一样的代码给我改

  •  1
     
  •   312ybj · May 20, 2020 · 7838 views
    This topic created in 2363 days ago, the information mentioned may be changed or developed.

    故事简介

    同事跑路,我来背锅 苦不堪言,发发牢骚

    先贴代码

    //      主服务器上的日志路径
            String path1 = "";
            String path2 = "";
            //vpm 服务器上的路径
            String ip3 = "";
            String path3 = "";
            String username3 = "";
            String pwd3 = "";
            // 从服务器上的路径
            String ip4 = "";
            String username4 = "";
            String pwd4 = "";
            String path4 = "";
            String path5 = "";
            String path6 = "";
            String path7 = "";
            String path8 = "";
            DailyManage[] dailyManageList = JacksonUtils.stringToObject(dailyManageListString, DailyManage[].class);
    //      主服务器第三个路径:产品 /工装顶图导入
            path1 = dailyManageList[0].getPaths()[2];
    //      主服务器第四个路径:工装 GCO 导入
            path2 = dailyManageList[0].getPaths()[3];
            ip4 = dailyManageList[1].getIp();
            username4 = dailyManageList[1].getUsername();
            pwd4 = dailyManageList[1].getPwd();
            path4 = dailyManageList[1].getPaths()[0];
            path5 = dailyManageList[1].getPaths()[1];
    //      产品最新已发布导入
            path6 = dailyManageList[1].getPaths()[2];
            path7 = dailyManageList[1].getPaths()[3];
            path8 = dailyManageList[1].getPaths()[4];
            ip3 = dailyManageList[2].getIp();
            path3 = dailyManageList[2].getPaths()[0];
            username3 = dailyManageList[2].getUsername();
            pwd3 = dailyManageList[2].getPwd();
            String logType = strs[0];
    
    1. 一个函数 300 行, 好几个 300 行的函数
    2. 没有变量名,自己瞎定义,"list1", "list2", "path1-8", "s", "ss"
    3. 半点注释都没,仅有的注释还是我为了重构加的

    虚晃一枪

    上周末他请假,说家里有事。经理就把他的活交给我了, 因为客户催的急。 结果我改的一头雾水,只能给他打电话, 我还不太好意思,毕竟他说家里有事情,请假了我也不好意思打扰。

    秃如其来

    结果今天他回来就辞职跑路。 搞了半天,他请假是去面试了, 他朋友缺个测试,问了他的意向,他跑去面试了。我今天本来还满心欢喜,以为他回来了,代码就有人能改了(我是真看不懂他的代码), 结果他就辞职跑路了,然后呢,还开开心心的说“辞职也不麻烦嘛, 也没什么好交接的”, 我擦, 你那狗都不认识的代码就直接推给我了,你当然没事,我可倒霉了。

    欲哭无泪

    果然有些人只能是过客,不是归人,结果这位归人还给我送了分离别礼物----重构代码的机会 我可开心了

    参考文章

    教你如何在 V2EX 发图片 /插链接 /插代码 /插视频

    41 replies  •  2020-05-20 21:29:13 +08:00
    xfxly
    xfxly
    May 20, 2020
    1
    重构过 1W+行的业务功能函数.....
    alienx717
    alienx717
    May 20, 2020
    2
    啥公司啊,说走就走,不安排一下么
    guojiaxiang
    guojiaxiang
    May 20, 2020
    3
    @alienx717 这代码不用交接都有人可以改,还有啥好安排的
    6oML852dJf9Kn2l7
    6oML852dJf9Kn2l7
    May 20, 2020
    4
    这种人才应该是 35 岁应该剔除的人
    SteveZou
    SteveZou
    May 20, 2020 via Android❤️39
    5
    太神奇了,隔壁帖子同事走了,那个兄弟失魂落魄仿佛失恋;而这边楼主鼻子都气歪了……
    lzdyes
    lzdyes
    May 20, 2020
    6
    哈哈哈啊哈
    m0kha
    m0kha
    May 20, 2020❤️5
    7
    周末请假面试。。
    LG3xFA6kpn88HxEW
    LG3xFA6kpn88HxEW
    May 20, 2020
    8
    我最近也准备要提桶跑路了,不知道接手我代码的人会是什么表情,希望不要喷得太惨
    cnmllll
    cnmllll
    May 20, 2020
    9
    这变量名命名.....
    HongJay
    HongJay
    May 20, 2020❤️2
    10
    上周末他请假
    kop1989
    kop1989
    May 20, 2020
    11
    没准他手上有两套代码,在公司用混淆版,自己手里有逻辑版。😂
    kop1989
    kop1989
    May 20, 2020
    12
    之前我遇到的接盘代码情况比喻起来是。原本的需求目标就在马路对面。结果这位仁兄不找斑马线,也不会看红绿灯。
    然后为了达到需求,在四九城里溜了一遍。关键竟然还没迷路……
    295464512
    295464512
    May 20, 2020
    13
    @kop1989 细思极恐
    hoyixi
    hoyixi
    May 20, 2020
    14
    为啥要改,能跑就继续跑。新增功能再自己写
    coderluan
    coderluan
    May 20, 2020❤️4
    15
    职场原则之一:同事的问题,都是领导的问题。

    你同事菜鸡和你本身没啥关系,但是楼主现在难受,是因为你们领导允许他写屎代码,允许他离职不交接,说到底,你领导不行,这时候怎么办,要么狠要么忍要么滚。
    kaiki
    kaiki
    May 20, 2020
    16
    我自己的代码这样写第二天我都会看不懂,他能不带注释全用混淆名,很难不怀疑他特地在生产环境使用这样的代码的。
    建议拿着代码上门打一架。
    MrStark
    MrStark
    May 20, 2020
    17
    @SteveZou 我也是刚看完那个帖子,突然看到这个贴,这落差有点大,哈哈哈
    yuang
    yuang
    May 20, 2020 via Android
    18
    他跳你也跳,不能跳就忍着,等能跳的时候跳
    luo1945049855
    luo1945049855
    May 20, 2020 via Android
    19
    今天的 v2 好欢乐
    SmallBox
    SmallBox
    May 20, 2020 via Android
    20
    你这还有点注释不错了
    5G
    5G
    May 20, 2020❤️1
    21
    @SmallBox #20 楼主说仅有的注释是他自己加的
    Adreamer
    Adreamer
    May 20, 2020
    22
    You jump,I jump
    Justin13
    Justin13
    May 20, 2020 via Android
    23
    如果是测试的话。。能跑起来就是万幸了,别想什么代码规范。
    lazypu
    lazypu
    May 20, 2020
    24
    自带代码混淆功能
    mahone3297
    mahone3297
    May 20, 2020
    25
    你同事是开发?然后,转测试了?
    他走了,你接手,看来还是蛮近的?平常没 review ?
    szyp
    szyp
    May 20, 2020
    26
    @SteveZou 哈哈,果然你有 10 个感谢
    jiom
    jiom
    May 20, 2020
    27
    要么..要么..
    GeruzoniAnsasu
    GeruzoniAnsasu
    May 20, 2020 via Android
    28
    hhhh 之前有个同事比你这个聪明一点:

    Button **btn = new ...
    for(...)btn[...]= new ...

    btn[0]->setText();
    btn[0]->move(x,y)
    ...


    btn[1]->...


    问他为啥写屎他说这才是灵活的好代码,这样界面布局改了改坐标数字就行了,而且能随时加按钮

    “这代码你看得出哪个是哪个吗”
    “看 text 啊,你没用 ide 吗,看一下引用就知道是哪个了啊”

    卧槽好像很有道理!
    szl06dl
    szl06dl
    May 20, 2020 via iPhone
    29
    周末还要请假,太惨了,跑路也正常。
    amwyyyy
    amwyyyy
    May 20, 2020
    30
    @kop1989 这个我是深有体会,上家公司一个同事差不多是这样。思路非常绕,测试出 bug 之后,他自己也不好解决,就通过调试,各种 if-else 硬编码把坑填上。
    Miscedence
    Miscedence
    May 20, 2020
    31
    离职吧
    98jiang
    98jiang
    May 20, 2020
    32
    周末请假??
    Erroad
    Erroad
    May 20, 2020
    33
    北医三院遇到过一个叫达芬奇的小孩看儿童口腔,候诊的全笑喷了
    Erroad
    Erroad
    May 20, 2020
    34
    @Erroad #33 回串了,艹...
    wanghao2018
    wanghao2018
    May 20, 2020
    35
    ' 你那狗都不认识的代码' , 笑喷了 😄
    wangxiaoaer
    wangxiaoaer
    May 20, 2020 via Android
    36
    @SteveZou 我也是一下子想到那个了,真是鸟大了什么林子都有。
    Norie
    Norie
    May 20, 2020 via iPhone
    37
    程序猿也是需要迭代的
    hugoqin
    hugoqin
    May 20, 2020
    38
    还有一种情况是同事留下屎一样的代码,然后当领导了
    pengjay
    pengjay
    May 20, 2020
    39
    去做测试,可想而知
    NonClockworkChen
    NonClockworkChen
    May 20, 2020
    40
    @GeruzoniAnsasu 有本事让他把这个写在简历里,不让都是放屁,逃
    ochatokori
    ochatokori
    May 20, 2020 via Android
    41
    @m0kha #7 盲生你发现了华点
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Privacy   ·   Solana   ·   1667 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 83ms · UTC 13:22 · PVG 21:22 · LAX 05:22 · JFK 08:22
    ♥ Do have faith in what you're doing.