到底改还是不改

[guoyuchuan]

看见了同事写的一段代码不严谨，数据多了会出错，心痒痒想给改过来，但是怕改了项目会出错，不改看起心里总不是滋味，纠结。你们遇到同事的代码不严谨会怎么去做（修改比较麻烦的那种）？

[scipio]

不要自作主张地改！
不要自作主张地改！
不要自作主张地改！

但是作为一个团队而言，最好也不要视而不见。

我个人建议：

你觉得这段代码有问题的话，最好能构建一个能让这段代码跑出问题的用例:

- 如果你们有较完善的 BTS，就直接提 BUG 出来，并把你的定位结果也附上去。

- 如果你们只是一个不太正规的小团队，那么就喊上这部分代码的负责人，以及你们的 Leader 聊一下这个事情，先把问题抛出来(这里的建议是： 千万不要一上来就说人家的代码有问题。而是一上来只谈你的用例)。如果大家都认为这是个问题了，你可以提出你调查的结果(即 xxxx 的 xxx 代码有问题)说出来。如果连 Leader(负责人)都不认为是个问题的话，那就不要再继续往下聊了。但最好能形成一个简单的备忘录来证明你组织大家讨论过这个问题，结论是啥啥啥。省得万一以后有什么衍生问题发生时，别人责怪你没早把问题提出来。

[jimi2018]

提醒一下不就 ok 了吗。

[cxh116]

不改。
看人，对方是对技术有追求的话，提醒一下，否则当作没看见。


@otakustay 技术氛围是技术 leader 搞出来的。当 leader 有工具格式化不格式化，要打空格的地方不打空格，导致代码都不能正常高亮，虽然能运行。没错，只要能运行就行了。

你作为下属要去大谈代码质量吗？

所以，
只跟同道中人谈追求。

这些都不是问题，给的工资过的去就行了。

[uuus007]

很多时候你看的问题都有其内在的逻辑。 可以询问同事，你的想法对不对。 但是千万不能你来改
千万不能你来改
千万不能你来改
千万不能你来改

[annielong]

这个和氛围什么的没关系吧，作为程序员肯定不能私自动别人的代码，尤其是 GIT 的公共环境下

[miemiekurisu]

Please DO NOT touch anything if it still woks well now, unless you REALLY KNOW what you're doing.