出差回来发现代码被重构了！

[dayoushen]

题主属于两个产品线，出差一周搞 A 产品线事情回来，一个星期快过去了，发现 B 产品线提交了一个新版本，本着好奇的心理看看更新了什么，最后自己完成的功能被重构了，心理 MMP。首先被重构了，还是很开心，终于少了一个包袱，不用调高校提供的 SDK，他们每次更新版本都不如旧版本好用，总是缺东西；其次发现组长在毫无征兆的情况下，被代码重构了，才意思到有很大的信任危机。问问大伙有没有相关经验，如何调整心态进入下一个产品线。

[dayoushen]

@imn1 谢谢，看得开。

[laike9m]

惊了，重构代码居然不需要之前写的人 review。。。这种公司还是离职吧

[GDC]

@libook

如果写得太垃圾 确实没有存在的意义，同事一场还撕破脸皮跟你说你写得太垃圾？多次被人重构自己心里没点数？如 3 楼说的，如果不是太垃圾，是不会一声不吭去重构的，这是缺一不可的条件。

你说的那种尊重 沟通 告知，是建立在还认为你的代码由自己改改就能更好的基础上，这种情况没谁闲着还去动别人代码的。

别扯什么团队每个人都缺一不可，不过都是个挨踢民工罢了，对公司来说更不会缺一不可，所以不要自欺欺人，讲什么职场道德，什么抢你客户，无论什么行业什么岗位，提升自己的能力水平才是王道

[lzhCoooder]

换了一个部门，上任领导被踢出开发队伍，新任领导让我把上任留下的所以代码重构...花了两个月，难免不让人多想为啥突然给他换岗位

[Maboroshii]

我最近也重构了一份代码。没办法啊，让我在之前的上面改实在是下不去手。

[LewisW]

@ssynhtn 一声不吭就重构？迟早被测试打死

[reus]

@libook 你们写的代码都属于公司的代码，不存在什么“你的代码”

[LowBi]

如果重构的代码比原来的好，那是支持的了，对于我这种菜鸡，多参考别人写代码的风格习惯及思维是最好不过了

[Macolor21]

想起我之前的代码
if(xxxx) throw ....

[Macolor21]

想起我之前的代码
if(xxxx) throw ....

[Macolor21]

想起我之前的代码 WTF,CTRL + ENTER 竟然是提交，不好意思，影响看帖体验了
if(xxxx) throw ....
if(yyyy) throw ....
被改成
if(xxxx) {
if(yyyy) {
........N 个 if throw
}
}
..然后我就申请离职了。

[publicvoid]

代码轻易不要重构，除非你很懂这块的业务，不然 bug 无限多。。。

[kkkkkrua]

看重构的好不好吧，不好可以准备走了。

[ian19znj]

既然 review 和 test 都过了，也 check in 进去了，说明重构的没啥问题啊，就不要在意啦。
话说你们公司是不是规模不大呀？要是大的公司，千百号开发，几百万的代码，每次重构还要逐个征得同意，不累死啊？
至于以后出问题，那是重构跟审核的锅。

[amumu666]

这是对你代码的否定，心里毫无波澜才是假的。
也许出差找不到你人才让别的同事改这块业务，接盘的同事也是心里 mmp，谁愿意改别人的代码啊。

[jimrok]

忍无可忍我才会重构某个别人的代码，通常我会指明代码有什么问题。如果我做这个事，估计这个人我不会再放在重要位置上了。

[specita]

我一般是先告诉别人我的想法，看他会不会自觉去优化，不会的话，我还是会手痒优化的。。。

[Keyes]

@jinhan13789991 +1，不过最近研发老大和大部门经理终于忍受不了我的碎碎念，把重构提上日程了，要坚持啊盆友

[Keyes]

@Macolor21 hhhhh 所谓的“嵌套层次不够”

[signalas1]

不要随意重构别人代码，特别是复杂度的部分。。虽然一般写业务搬砖没什么复杂度..