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

[sqfphoenix]

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

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

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

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

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

[darksword21]

@zcfnc 我也干过 2 4

[zcfnc]

@banmuyutian #17
@darksword21 #20
啊这，两位无意冒犯。这里特指我的领导（：

[mrxun1998]

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

[Shook]

之前我写的分页组件，计算当前页的算法有点问题。
我后面搞别的项目去了，接手我项目的同事因为老有分页问题被骂了好多次。

偷偷看他们项目的接口文档，发现 start 是从 0 开始的，我的算法是从 1 开始算的。

[YhXyt]

@zcfnc 这不是很正常吗。xml2Json()

[ALVC666]

代码和人 有一个能跑就行[Doge]

[zhuangjia]

还是要加注释

[lizenghui]

哈哈哈 楼主太纠结
This is just a job

[leafre]

模块继续细分

[rainymoring1995]

不想干就走呗，贼烦对同事指手画脚的，你又不是直系领导

[fengfuliu]

@banmuyutian 这个用法还好吧 我看到很多出名的库也是这样的

[lzjamao]

你同事工作方式有问题。
改别人代码逻辑，最好找负责过该处的同事商量好再改。

[MiniGhost]

1. 分工的时候尽量避免特别密集的代码交叉，尽量同期需求各自负责各自的
2. 推 Code Review，开发需求强制要求拉新分支，合并分支强制走 Review，同事 Review 不通过不给合并

[zxCoder]

@YhXyt xmlToJson 也没啥不好吧 能问下用 2 代替 to 的好处吗

[dongdongdong]

我了

[SjwNo1]

如果你是 reviewer 的话，不改就不给合，如果你不是的话，好像没啥办法

[wisetc]

你们约定一下子，以及讲讲这么做为什么好，以及对他也好，用权威的客观的主流观点代替自己说话，告诉别人你的边界，以及他的边界，避免相互影响相互伤害，否则兵戎相见也不好看。

[wisetc]

有些人水平比较低也不尊重人，这种人还可能你躲也躲不过，讲道理还不一定听，哪你岂不是很难受？怎么办呢？要么建墙阻隔侵扰，要么交流到达彼此同化，要么强迫把人家赶走，你觉得哪种更好？

[lululau]

个人经验：只要是你俩共同开发的任务，把他的活都干了； 多干就是少干，你不管他的，可能得陪他一块加班，你多干了可能反而不需要加班

[pengtdyd]

这就体现一个人写一个模块的好处了