代码重构
今天作了一天的代码重构,主要是对一个文件进行重构:BlogentrySessionBean.java。
这个文件是我们博客最核心的一个文件,也是上次重构的重点。
上次重构主要是把应该归在Blogentry 这个EJB里面的东西,划走,划到其它几个它们应该呆的地方,为此新建了几个EJB。
这次重构,主要是在这个文件的内部进行,除了划了2个method到新建的BlogentryDAO里面外。
看来复制粘贴所在的地方就是最需要重构的地方。
重构前文件长达近900行,重构后只有412行。
除了移走的150行左右的代码外,内部消化了300多行代码。
这就是说,原来的900行代码里面,有150位置不当外,竟然有1/3的费代码,真水呀,都不好意思承认,那样的代码居然是自己写的。
主要重构手段,将几个相似的method的共同部分抽出来,新写一个 private method,后来又进一步从private method 里面走出新的 private method。
重构前,原来的主方法(major methods),个个长有100行,现在不到10行,有好几个方法体只有清清爽爽的2行,感觉真是舒服多了。
以前这个文件900行,看得人头大,不愿意碰它,现在清爽多了,扩展起来也容易了。
这个文件是我们博客最核心的一个文件,也是上次重构的重点。
上次重构主要是把应该归在Blogentry 这个EJB里面的东西,划走,划到其它几个它们应该呆的地方,为此新建了几个EJB。
这次重构,主要是在这个文件的内部进行,除了划了2个method到新建的BlogentryDAO里面外。
看来复制粘贴所在的地方就是最需要重构的地方。
重构前文件长达近900行,重构后只有412行。
除了移走的150行左右的代码外,内部消化了300多行代码。
这就是说,原来的900行代码里面,有150位置不当外,竟然有1/3的费代码,真水呀,都不好意思承认,那样的代码居然是自己写的。
主要重构手段,将几个相似的method的共同部分抽出来,新写一个 private method,后来又进一步从private method 里面走出新的 private method。
重构前,原来的主方法(major methods),个个长有100行,现在不到10行,有好几个方法体只有清清爽爽的2行,感觉真是舒服多了。
以前这个文件900行,看得人头大,不愿意碰它,现在清爽多了,扩展起来也容易了。
hofman
2004-11-20 02:32:57
评论:0
阅读:1843
引用:0
