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

[sqfphoenix]

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

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

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

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

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

[InDom]

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

[erwin985211]

上级反映，申请跟换项目

[Jaosn]

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

[signalyc]

1. 分开，使之没有交集
2. 把他 /她的工作都做了

[fkname]

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

[jonathanchoo]

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

[kop1989]

1 、合作常见，但为何会导致逻辑污染？（换句话说，你们会经常修改对方的代码吗？）
2 、一切交集以文档与方法注释为准。起名你看不顺眼并不意味着是“代码烂”。（别人名字你看不顺眼你也要逼着对方改户口本么？）

[sqfphoenix]

@InDom 兄弟你就是始作俑者，必须给你一个赞

[sqfphoenix]

@kop1989
第一个问题是这样的 比如我的代码是 1.1 -> 1.2 -> 1.3 这样执行，他的代码是 2.1 -> 2.2，是通过一个状态判断到底走哪个逻辑的，结果他一改，把逻辑入口给改了，所有逻辑全部都走 2，也就是永远走不到 1 的逻辑了，这种情况我真的没有办法控制逻辑了
至于第二个问题，我说的只是冰山一角，我对代码是非常宽容的，一般只要正常运行用例通过都 OK，可是这种代码，完全就是地雷，你永远不知道调用这个函数有什么后果

[micean]

好像我接手的那位，尤其是第二点，令人记忆深刻

[3dwelcome]

明显是模块责任没划分好，为什么要共同改一个模块呢？把大模块分开，每人只改自己那部分的多好。

[shapl]

还是按模块划分吧。说好尽量别碰对方代码。

[woomly]

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

[sqfphoenix]

@3dwelcome 这个确实，当时的背景是我手上堆了一堆需求做不完，只能让他写了，结果这块代码现在就是永远的痛

[erlking]

多大点事，默默的重命名为 isNotXXX 完事

[scxiazi]

你见过只会写单表增删改查的吗, 让他提供接口就说业务入侵了

[zcfnc]

我已经麻了 我甚至还有写方法名带数字的同事 他还是我的领导 你敢信？

[banmuyutian]

@zcfnc
这事我也干过，用'2'代替'to'

[LowBi]

一个人写项目自由自在，虽然会累会很忙

[czzhengkw]

模块拆解+单元测试+代码 review

改代码可以，几百个测试全部能跑通，pr 才能合并进来