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

2020-05-20 10:37:28 +08:00
 312ybj

故事简介

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

先贴代码

//      主服务器上的日志路径
        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 发图片 /插链接 /插代码 /插视频

6306 次点击
所在节点    问与答
41 条回复
5G
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
如果是测试的话。。能跑起来就是万幸了,别想什么代码规范。
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
hhhh 之前有个同事比你这个聪明一点:

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

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


btn[1]->...


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

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

卧槽好像很有道理!
szl06dl
2020-05-20 14:44:27 +08:00
周末还要请假,太惨了,跑路也正常。
amwyyyy
2020-05-20 14:45:34 +08:00
@kop1989 这个我是深有体会,上家公司一个同事差不多是这样。思路非常绕,测试出 bug 之后,他自己也不好解决,就通过调试,各种 if-else 硬编码把坑填上。
Miscedence
2020-05-20 16:11:34 +08:00
离职吧
98jiang
2020-05-20 17:49:05 +08:00
周末请假??
Erroad
2020-05-20 17:54:35 +08:00
北医三院遇到过一个叫达芬奇的小孩看儿童口腔,候诊的全笑喷了
Erroad
2020-05-20 17:54:58 +08:00
@Erroad #33 回串了,艹...
wanghao2018
2020-05-20 18:03:04 +08:00
' 你那狗都不认识的代码' , 笑喷了 😄
wangxiaoaer
2020-05-20 18:35:59 +08:00
@SteveZou 我也是一下子想到那个了,真是鸟大了什么林子都有。
Norie
2020-05-20 18:56:35 +08:00
程序猿也是需要迭代的
hugoqin
2020-05-20 21:08:34 +08:00
还有一种情况是同事留下屎一样的代码,然后当领导了
pengjay
2020-05-20 21:10:22 +08:00
去做测试,可想而知
NonClockworkChen
2020-05-20 21:20:30 +08:00
@GeruzoniAnsasu 有本事让他把这个写在简历里,不让都是放屁,逃

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/673513

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX