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

[sqfphoenix]

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

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

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

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

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

[goodboy95]

@kop1989 什么情况下汉语英语的“是”与“不是”相反，举几个例子吧，我自己是想不太出来，唯一能想到但又特别牵强的就是“Don't you know”这种带 not 的疑问句

[Yc1992]

review 直接写 comment 啊，只要你不尴尬，尴尬的就是他

[lxfcool]

@InDom 笑死

[comoyi]

让他全部改回来

[Paaranoia]

跟楼主有同样的感受，分享下我的经历
我写了一个，根据版本号检查更新的逻辑
新来的同事，改了我的代码，把一个“1.0.0”这样的字符串用 Integer.parseInt()转换
导致 App 上线以后，所有检查更新都崩了
我当时血压直接就上来了

[z740713651]

python 的话写个 typing?

就有理由告老板了

[Felldeadbird]

代码隔离不到位。或者是，你的业务对方不熟悉。没有留下文档或注释。团队肯定有水平高低的，让管事的来规划好代码开发

[est]

生气归生气，你应该做一个单元测试把自己的方法守住。别人改的话测试直接不让通过。

[free9fw]

离职吧，迟早会被祸害

[zzx0403]

之前见过各种简写的😂function 本来简写 fun，他来个 fun 。teacher 硬是要简写成 tea，各种简写看都看不懂

[sexyback]

跟同样是应届生的同事一起写一个模块，像 status 这种状态标志，有人 int8 有人 int32 有人 int，真的难顶

[elevioux]

再包装一下 notXXX(){return !isXXXX();}😂

[rrZ2C]

@banmuyutian 呀 学到了,长一点的方法名真讨厌写 To

[nicetoomeetyou]

看描述，应该是不属于一个部门和领导的同事。所有反映是没啥用的。
这时候建议听下歌，https://www.kuwo.cn/play_detail/236015

[hatsuyuki]

@zzx0403 Rust 里定义函数的关键字是 fn，Kotlin 里定义函数的关键字是 fun，Go 里定义函数的关键字是 func，这样来看的话 function 简写成 fun 也不是不能接受😂

[zzx0403]

@hatsuyuki 主要是我上次看到了他自己的项目 目录名称 funny1 funny2 funny3 当时我真的蚌埠住了😅

[uselessVisitor]

帮他把代码改对

[cloudfstrife]

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

[NoDocCat]

不敢点进来, 怕你说的是我.
https://imgur.com/a/BcuKMdq

[TeaPolyphenols]

最后我给他重写了..