V2EX = way to explore

V2EX 是一个关于分享和探索的地方

已注册用户请登录

这是一个创建于 1435 天前的主题，其中的信息可能已经有所发展或是发生改变。

故事简介

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

先贴代码

//      主服务器上的日志路径
        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];

一个函数 300 行，好几个 300 行的函数
没有变量名，自己瞎定义，"list1", "list2", "path1-8", "s", "ss"
半点注释都没，仅有的注释还是我为了重构加的

虚晃一枪

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

秃如其来

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

欲哭无泪

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

参考文章

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

41 条回复 • 2020-05-20 21:29:13 +08:00

xfxly

2020-05-20 11:04:54 +08:00

重构过 1W+行的业务功能函数.....

alienx717

2020-05-20 11:12:33 +08:00

啥公司啊，说走就走，不安排一下么

guojiaxiang

2020-05-20 11:16:07 +08:00

@alienx717 这代码不用交接都有人可以改，还有啥好安排的

g5hdyd

2020-05-20 11:17:29 +08:00

这种人才应该是 35 岁应该剔除的人

SteveZou

2020-05-20 11:17:52 +08:00 via Android

太神奇了，隔壁帖子同事走了，那个兄弟失魂落魄仿佛失恋；而这边楼主鼻子都气歪了……

lzdyes

2020-05-20 11:19:20 +08:00

哈哈哈啊哈

m0kha

2020-05-20 11:22:17 +08:00

周末请假面试。。

l1nyanm1ng

2020-05-20 11:24:22 +08:00

我最近也准备要提桶跑路了，不知道接手我代码的人会是什么表情，希望不要喷得太惨

cnmllll

2020-05-20 11:26:33 +08:00

这变量名命名.....

HongJay

2020-05-20 11:29:43 +08:00

上周末他请假

kop1989

2020-05-20 11:39:36 +08:00

没准他手上有两套代码，在公司用混淆版，自己手里有逻辑版。😂

kop1989

2020-05-20 11:44:46 +08:00

之前我遇到的接盘代码情况比喻起来是。原本的需求目标就在马路对面。结果这位仁兄不找斑马线，也不会看红绿灯。
然后为了达到需求，在四九城里溜了一遍。关键竟然还没迷路……

295464512

2020-05-20 11:46:05 +08:00

@kop1989 细思极恐

hoyixi

2020-05-20 11:50:10 +08:00

为啥要改，能跑就继续跑。新增功能再自己写

coderluan

2020-05-20 11:58:07 +08:00

职场原则之一：同事的问题，都是领导的问题。

你同事菜鸡和你本身没啥关系，但是楼主现在难受，是因为你们领导允许他写屎代码，允许他离职不交接，说到底，你领导不行，这时候怎么办，要么狠要么忍要么滚。

kaiki

2020-05-20 12:01:41 +08:00

我自己的代码这样写第二天我都会看不懂，他能不带注释全用混淆名，很难不怀疑他特地在生产环境使用这样的代码的。
建议拿着代码上门打一架。

MrStark

2020-05-20 12:20:08 +08:00

@SteveZou 我也是刚看完那个帖子，突然看到这个贴，这落差有点大，哈哈哈

yuang

2020-05-20 12:22:25 +08:00 via Android

他跳你也跳，不能跳就忍着，等能跳的时候跳

luo1945049855

2020-05-20 12:24:21 +08:00 via Android

今天的 v2 好欢乐

SmallBox

2020-05-20 12:34:07 +08:00 via Android

你这还有点注释不错了

2020-05-20 12:54:28 +08:00

@SmallBox #20 楼主说仅有的注释是他自己加的

Adreamer

2020-05-20 13:03:38 +08:00

You jump,I jump

Justin13

2020-05-20 13:08:26 +08:00 via Android

如果是测试的话。。能跑起来就是万幸了，别想什么代码规范。

linxl

2020-05-20 13:12:32 +08:00

自带代码混淆功能

mahone3297

2020-05-20 13:45:35 +08:00

你同事是开发？然后，转测试了？
他走了，你接手，看来还是蛮近的？平常没 review ？

szyp

2020-05-20 13:50:08 +08:00

@SteveZou 哈哈,果然你有 10 个感谢

jiom

2020-05-20 14:06:42 +08:00

要么..要么..

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 吗，看一下引用就知道是哪个了啊”

卧槽好像很有道理！