你们会去“优化”同事的代码吗？

[yxaaa123]

update 的时候发现我的代码被修改了，原来新来的同事看我的代码觉得有问题直接改了，顺便还给“优化”了一下。。。

我很无语啊。。。

[ifconfig]

更好，以后出了事找他就行了

[dongyado]

改的时候最好跟你讨论一下

[yxaaa123]

@dongyado 我觉得看到别人写的东西有问题，可以提，但是不能自己来

[bk201]

不会，最多提一下，除非自己是技术主管，否则这样做只能说心智太低。

[foomorrow]

code review 的时候让他自己去改咯

[reeco]

不会改别人的代码

[dongyado]

@yxaaa123 对，告知，或者讨论是最基本的尊重，还能相互学习提高，如果直接改，只能说这人情商着急。

[66450146]

你们缺少 code review...

[thinkif]

说句题外话，改写的部分写的好么？

[zhouyg]

你这种情况还好，我突然发现某个功能不正常，然后发现的代码是被直接清除了....虽然这些兼容代码是丑了点..

[cxbig]

更新的是代码规范还是逻辑实现？
公司应当有一个统一的编码规范，大家都必须遵守，不应当在这个点上产生冲突；
至于逻辑实现，先行告知并讨论，选最优。

[9hills]

关键是改好了还是改坏了，从 lz 的描述，应该确实是改好了。。。

[specita]

最好不要，如果真的要优化，应该先和同事讨论一下

[Sharuru]

会事先和同事沟通一下，但是大部分情况更偏向于直接一个 if else 把自己要走的逻辑单独出去。

[tjxiter]

会

[hantsy]

在代码合并到 master 就应该做 Code Review 。

[charlie21]

完全不懂你在说什么
git 新手在 commit 或 push 文件时出错了呢 也说不定
把 commit -m 发出来或许就更明白你在说什么了

[shiye515]

这种问题应该贴代码，我就经常改别人的代码

[Septembers]

会 但是会事先沟通说明以及解释

[sweetcali]

好在意他到底改的好还是改的坏。本身他未经你允许就改这个很粗鲁，沟通都没有的。