V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX 提问指南
312ybj
V2EX  ›  问与答

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

  •  1
     
  •   312ybj · 2020-05-20 10:37:28 +08:00 · 6542 次点击
    这是一个创建于 1682 天前的主题,其中的信息可能已经有所发展或是发生改变。

    故事简介

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

    先贴代码

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

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

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

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


    btn[1]->...


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

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

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