吐槽：维护别人的代码真蛋疼！

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

• 请不要在回答技术问题时复制粘贴 AI 生成的内容

这是一个创建于 4609 天前的主题，其中的信息可能已经有所发展或是发生改变。

今天修改一个同事写的java代码，一个方法整整八百行，当时真想把《重构》甩他头上！

代码

真蛋

吐槽

37 条回复 • 1970-01-01 08:00:00 +08:00

kylefeng

2013 年 6 月 6 日

重构后：xxxAdapter, xxxFactory, xxxBuilder, xxxManager, xxxHandler, xxxFilter, xxxFacade, xxxComponent 等等，一大坨。

dreampuf

2013 年 6 月 6 日

怎么现如今还有人天真的以为一本《重构》就能呼唤醒人性的洁癖？！

起码得九本：《GoF》，《重构》，《代码匠艺》，《代码大全》，《代码简洁之道》，《编码》，《人月》，《人件》，《修炼之道》

啪啪啪！啪啪啪！啪啪啪！的甩在他的脸上，让他知道，什么叫做码农！

heroicYang

2013 年 6 月 6 日

既然都接手维护了，还有什么比重构烂代码更紧急的事情呢，哈哈，祝楼主好运！

saharabear

2013 年 6 月 6 日

见过一个牛人，过他手的代码，40%都会变得好起来。

vietor

2013 年 6 月 6 日

重构是“纯心情”活，如果真如 @kylefeng 吐槽的那样，还不如不进行结构呢，按着原来的风格改。
重构改的是“功能和扩展性”，不是“风格”。java这种语言最大的问题就是“类”的膨胀，如果将800行代码分出4、5个类来就不合算喽。

Golevka

2013 年 6 月 6 日

@dreampuf 虽但厉

vvii

2013 年 6 月 6 日

@saharabear 60%的隐喻。

acpp

2013 年 6 月 6 日

反之，别人维护你的也是一样。

akira

2013 年 6 月 6 日

今天修改一个同事写的java代码，一个方法能实现的东西，整了八个类出来，当时真想把《GoF》，《重构》，《代码匠艺》，《代码大全》，《代码简洁之道》，《编码》，《人月》，《人件》，《修炼之道》甩他头上！

别人看你代码会这样吐槽不。。。

brando

2013 年 6 月 6 日

不要抱怨，默默的改進...加油！

tyzc

2013 年 6 月 6 日

@saharabear 必须对业务觉得了解才能修改，要不然就Bug了！！

banxi1988

2013 年 6 月 6 日

@kylefeng
@akira
重构最多的是extract method。
method name as a direct comments。
哪里像你们说的就一定要extract 这么多类来，
而且如果像你们说的搞这么多类的话，肯定是设计模式患者了。
800多行的代码，要么是逻辑真的太复杂了。要不就是。太懒了。
一直写，写啊写。吃完饭回来，写啊写。
一不小心800多行了。下班了。......，然后.....

wzxjohn

2013 年 6 月 6 日

呵呵

liwei

2013 年 6 月 6 日

彼此彼此

judasnow

2013 年 6 月 6 日

他的变量名全是用的拼音缩写而且没有文档吗（如 XX XQ CSRQ XGID XJJ ..）？
没有的话他对你已经很仁慈了。。。

saharabear

2013 年 6 月 6 日

@tyzc 靠单元测试。

saharabear

2013 年 6 月 6 日

@vvii 我很佩服能改10%以上的人。

kk71

2013 年 6 月 7 日

pythoner飘过……java的类膨胀太可怕了

aurorawu

2013 年 6 月 7 日

跟楼主握个爪，这两天我也是修改之前项目别人写的Java代码要疯掉了

xiluo

2013 年 6 月 7 日 via Android

不要为事情的开始而兴奋，不要为事情的结束而懊恼，不要为生活的开始而疯狂，不要为旅程的结束而伤感，一切随风而来，随风而去，自然自由自在，如云如海如山。

RisingV

2013 年 6 月 7 日

写点简洁点的代码，真的需要看那么多书吗？不妨自己仔细思考，提取抽象，照样能写得很漂亮。

要知道设计模式不是本来就存在的，也是别人的经验。

按照个人经验来讲，可以概括一下自己的抽象方式，然后和书上一比对，你发现自己重新发明了这些设计模式，你明了这些设计模式真正的适用场景，而不是死搬硬套，除了成就感还得到了很好的修炼。