请问同事代码实在是烂到让人无法忍受怎么办

[sqfphoenix]

利益相关，经常需要合作开发一个功能，所以在一些模块中会有交集，做不到眼不见为净

不说远的，就说今天发生的事情

• 共同改一个模块，直接把我好好的代码改崩了，我排查了半天发现其中判断逻辑匪夷所思

• 他在 helper 中加了自己写的一个 IsXXX() bool 方法，返回值居然和函数名称是相反的，然后调用的时候全部都反向判断

我真的血压升高，拳头都看硬了，也不是没有做过代码 review，也分析了为什么代码写的不好，改完还是老样子，这种同事应该怎么相处，我现在天天跟坐牢一样好苦

[Perry]

一看就是测试都没有

[vZexc0m]

@zxCoder #34 如果不是驼峰命名，就得写成 xml_to_json

[steptodream]

可能会有人看到你的代码也是一样的感觉

[kiritoxf]

我有个同事还不如你这个同事……
工作三年了编码能力和学习能力像实习生一样
关键是也得一起合作，唉

[66beta]

一遍跟前同事吐槽，一遍擦屁股，还能怎么办...

[entro]

建议反馈给领导，理由是影响项目开发进度

[Kimen]

世界是有层次的，建议跑路

[tonzeng]

淦他娘草鸡掰

[jiayong2793]

向项目经理反馈，请求让他做独立模块

[piloots]

不敢点进来，怕你说的是我。

[gulullu]

不敢点进来，怕你说的是我。

[dfkjgklfdjg]

@rainymoring1995 #30，屎山就是这样出来的....
[Dev_1] 随性开发，到最后改不动了直接走人，[Dev_2] 接手直接脑溢血送 ICU，[Dev_3] 接盘重构。

[balezhang]

不敢点进来，怕你说的是我。

[Brentwans]

1.上策，尽快溜掉，避免和他合作，省时省力。
2.中策，让直接领导知道并认同你的看法，让领导去沟通。就算沟通无果，后面也好甩锅。
3.下策，你说沟通无果。那首先划分好模块，但是你们是一个功能，唇亡齿寒吧。加强单测确保功能，只要他的功能没问题，屎山又奈你何。
4.地下室策。大哥喝茶，你的功能我帮你写

[coderluan]

职场上, 所有同事的问题, 都是领导的问题, 反应上去就完了.

[ZzFoo]

@kop1989 你再好好看看，第二点是顺不顺眼的问题吗？

[wupher]

我之前也碰到过类似情况。

当时是眼不见心不乱，反正能跑，我就不管，而且也是临时帮忙。

后来他离职了，这块代码在检查中发现，到处是各种匪夷所思的实现。比如明明在外面查询一次即可获取得的数据库开关属性，结果在 for 循环中每次都再查询一遍……不使用诸如对象或者 VO，而是使用 map 直接各种插入，而取的时候又完全不管可不可能为空。层级封装就更离谱。

最后和产品商量，通过某次界面改版直接把他原来的代码功能全废弃掉了，代码留在原地，添加注释引以为教训。

但从头管到尾，貌似也不好。

不过，我还是觉得 You are not your code. Be kind to coder, not to the code.

代码写的烂，但不意味着就是坏人。

[kop1989]

@ZzFoo #56 谁规定 isXXX，必须 true 就是 yes （是），false 就是 no （不是）了？
英语语境与汉语语境的“是”与“不是”往往是相反的。

[ZzFoo]

@kop1989 麻烦举个例子

[fareware]

当觉得周围都不如自己时，那就换一个更高的地方，来证明之前他们真的不如自己。