到底应不应该重构别人写的代码？

按照童子军原则来说，看到有问题的代码就应该马上修正它。但是实际上重构别人的代码可能会让别人觉得没有尊重他的劳动成果，特别是，重构一个比自己更年长的人的代码。

yangqi

2016-06-22 10:56:31 +08:00

不应该，浪费时间浪费资源，很多情况觉得可以重构的其实是自作聪明的行为。

awanabe

2016-06-22 11:11:55 +08:00

当你因为不了解业务，没有在系统的讨论下重构之后踩了坑，你就知道线上故障都是你们年轻人的

chilaoqi

2016-06-22 11:21:06 +08:00

到底应不应该帮隔壁老王媳妇生个像我孩子。

MajestySolor

2016-06-22 11:23:30 +08:00

楼主肯定以前没试过
亲身体验，四个字，自讨苦吃

chilaoqi

2016-06-22 11:24:19 +08:00

老王年纪比我大。
老王孩子（视代码如己出）太讨厌太烦人啦。
这孩子从小这样，以后肯定危害社会。（这垃圾代码上线一定崩）
为了公司利益。。。

老王会 neng 死你的，哈哈

fwrq41251

2016-06-22 11:45:05 +08:00

@yangqi 这次是真的应该重构的地方，因为之前这个模块都是我负责的，而且我已经写过很多遍了（每次重写都会比之前多一些心得）。之前因为出差了一个多月，同事才在新项目里写了这一个模块。

ChiangDi

2016-06-22 12:12:45 +08:00

特别是，看着长者的代码写得非常烂

yuekcc

2016-06-26 13:11:57 +08:00

我也遇到了。

长者代码很垃圾，沟通未果，果断重构。为什么？因为我才是被烂搞死的那个！

skinheadbob

2016-07-20 06:57:11 +08:00

跟老板打好招呼，重构前先写足 test case

这是一个专为移动设备优化的页面（即为了让你能够在 Google 搜索结果里秒开这个页面），如果你希望参与 V2EX 社区的讨论，你可以继续到 V2EX 上打开本讨论主题的完整版本。

V2EX 是创意工作者们的社区，是一个分享自己正在做的有趣事物、交流想法，可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.