老同事不让重构代码怎么破？

[CrazyMoon]

最近组里开始做代码互审，同事审到偶修改的一个程序，有意见。意见是偶除了修改需求要求的内容之外，还做了一些其它修改。

偶的确改了一些其它内容，这因为偶发现程序里面的一些 hard code 比较难读（比如 type = 'P', type = 'A'之类的），于是创建了几个常量，以便看代码的人通过常量名理解这些编码的意义。

偶觉得这样改很好，但同事不同意，她说 1)改的地方太多，后人比较版本的时候，会对修改内容感到很困惑。2)改了需求外的东西，可能会导致意料外的 bug 。

同事资格比较老，且一向比较看重规则。偶年轻而且又比较散漫，没能坚持自己的观点，最后还是把这些东西又改回去了。不过偶依然觉得自己原本的做法没错。。。要怎么办呢？

[wysnylc]

偶?现在是 2010 年?

[thefack]

信老同事的

[AngryPanda]

她是女人，听她的

[a719114136]

你没错，他也没错。就一风格问题，没那么多可纠结的，既然他资格老就听他的呗。

等你资格老了就你说的算了

[th00000]

信老同事的,
觉得不爽, 后台再提交一个 pull request 请原本开发这个地方的同事帮你 review 一下, 确定你没有将老代码改出 bug, 再把代码合进去.

[imn1]

你的项目，你对
公司的项目，她对，尤其是她说的两点，注意并不是说你改错了

[wutiantong]

又是标题党

[NonClockworkChen]

改了，荣誉算你的，出了事，你赔钱离职即可。
就怕你担不起责，想起了几天前 800w 损失的运维。

[blindie]

就让你们看不懂 她这个项目才稳稳归她 own 饭碗捧的牢。其他修改感到困惑就抽离这个修改单独 commit 并完善注释文档 意料外的 bug 没测试的吗？总而言之就是让你别碰这些 那你就理解别碰就好了

[opengps]

你是做技术的，但是眼里不要只有技术。改动他的代码，及时你在技术上得满分，但在这件事的处理上也未必能及格。指出他人代码不足会让人感觉不爽，自然也就会不顺从你的意思。

另外，少改动线上代码，这本身也是个非常安全的做法

[yikyo]

你的错，你在当前任务中做了无关此次任务的工作。你要重构可以，单独提任务，测试，上线。想法是好的，做法是错的。

[zhuawadao]

加注释

[littleylv]

听她的。

你这偶，仿佛让我回到了 200x 年

[ccoming]

如无必要，勿重构代码。

[crazybinggan]

听 MM 的。
GG，你也网上冲浪...

[hpashencedany1]

大胆一点, 就说除了问题偶负责

[pan176]

听偶的

[glfpes]

这个‘偶’，有 15 年前内味了

[lneoi]

这也没不让你改吧，改了需求以外的东西以后不好复查，可以另外提一个

[xuarongla0000]

做多错多