如果你认为这是一个标题党，那么我真诚的恳请你耐心的把文章的第一部分读完，然后再下结论。如果你认为能够戳中您的G点，那么请随手点个赞。
把三千行代码重构为15行
那年我刚毕业，进了现在这个公司。公司是搞数据中心环境监控的，里面充斥着嵌入式、精密空调、总线、RFID的概念，我一个都不懂。还好，公司之前用Delphi写的老客户端因为太慢，然后就搞了个Webform的替代，恰好我对Asp.Net还算了解，我对业务的不了解并不妨碍我称成为这个公司的一个程序员。小公司也有小公司的好，人少，进去很快负责代码开发。我当然也就搞这个数据中心智能管理系统啦。
这个系统非常的庞大，尤其牛逼的是支持客户端组态，然后动态生成网页，数据还能通过Socket实时监控(那时我还真就不懂网络编程)。这个对于当时的我来说，真真是高、大、上呐！！当时跟着了解整个系统大半个月才算能够调试，写一些简单的页面。
在维护系统的过程中，时不时要扩展一些功能，也就接触了下面这个类：
看到没有，就是当年最最流行的三层架构的产物，对于刚出茅庐的毛头小子来说，这是多么专业的文件头注释，还有反射也就算了，这构造函数还能静态的，还能私有的？那时刚接触这么高大上的代码的我，瞬间给跪了！
但是，类写多了，我就感觉越来越别扭，就是下面这段代码：
每增加一个表，除了要改接口、要改DAL、要改BLL之外，还得在这个工厂类添加一个方法，真真是累到手抽筋，即使有当时公司了的G工给我推荐的神器——动软代码生成器，这粘贴复制的几遍，也是让我感觉到异常繁琐，有时候打键盘稍微累了点，还把复制出来代码改错了，你妹的，难道这就是程序员该干的事情，不，绝对不是！我想起了一句至理名言：当你觉得代码重复出现在程序中的时候，就应该重构了。是的，在这句话的指导下，我开始了折腾，决定挑战这个高大上的代码，事实证明，思想的力量是无穷的。
那么，怎么修改呢，仔细观察之后，发现其中className的生成跟返回的类型非常类似，只是一个是类名，一个是字符串，这两者之间应该能够关联起来。于是google了一下(当时GFW还没猖獗起来哈)，隐隐约约就找到了“反射”这两个字，深入了解之后，确定可以完成。
接下来，就是返回的类型了，返回的类型并不固定，但是似乎很有规律……这个似乎好像在哪里见过，对了，模板，C++课程上有讲过的，于是再次google，了解到了C#中使用了泛型代替了C++中的模板。在学习完泛型和反射之后，并参考了网上的一些文章，我捣鼓出了下面的代码：
没错，就是它了，三层架构年代最流行的工厂类……
看着原来滚十几屏幕的代码，变成了十多行的代码，真是爽到了骨子里去了，太干净了！唯一让我担忧的是，我进公司的时候，帮忙整理公司申请软件著作权都是需要代码量的，根据代码多少行来评估软件的大小，万一老板知道了我非但没有帮公司增加代码量，还减少了，会不会立即把我开掉？我没敢给我们老板展示我优秀的成果，所幸，这段代码非但没有出过任何问题，还避免了以前同事老是在新增一个类之后，把代码复制过来，但是没有正确修改的问题，大大提高了效率。虽然，我没敢大事宣布我的劳动成果，但是这次成功的修改，则彻底让我走上了代码重构的不归路。
看到这里，大家应该知道这个案例是否真实的了吧。我相信，从08年开始的码农们，看到这种类似的代码绝对不比我少。那么，我想告诉你们的是什么呢？
- 要在编程过程中多思考
- 编程的思想很重要，请多看点经典的书
- 从小处着眼，慢慢重构，尤其在应对一个大型的系统
- 当重复出现的时候，你应该考虑重构了
- 粘贴复制的代码越少，你的系统越稳定
少用代码生成器
我们来分析一下，为什么我之前的前辈会写出上面的代码。我归结起来有以下几点：
- 因为使用了动软代码生成器，生成代码方便，就没多想了。
- 三层架构的概念倒是了解了，但是没有去深入思考就拿来应用
- 遇到重复的代码，没有重构的概念，这是思想的问题——思想比你的能力重要
至今为止，还是很多人使用代码生成器，那么我们应该怎么对待这个问题呢。我认为，代码生成器确实可以减少你不少工作，但是少用，那些重复性的工作，除了部分确实是没有办法的，其他大部分都是可以通过框架解决的，举例来说，像三层架构，真正需要用到代码生成器的，也就是Model类而已，其他的完全可以在框架中完成。因此你要竭尽全力的思考怎么在框架中来减少你的重复性工作，而不是依赖于代码生成器。
另外，如果你还是在用相关的代码生成工具，请重新定义“动软代码生成器”的代码模板，自己写一个模板；或者使用CodeSmith来完全制定自己的代码生成，因为动软给的代码模板真心乱，比如下面这段代码：
|
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
|
for
(
int
n = 0; n < rowsCount; n++)
{
model =
new
DBAccess.Model.eventweek();
if
(dt.Rows[n][
"GroupNo"
].ToString()!=
""
)
{
model.GroupNo=
int
.Parse(dt.Rows[n][
"GroupNo"
].ToString());
}
if
(dt.Rows[n][
"Week0"
].ToString()!=
""
)
{
model.Week0=
int
.Parse(dt.Rows[n][
"Week0"
].ToString());
}
if
(dt.Rows[n][
"Week1"
].ToString()!=
""
)
{
model.Week1=
int
.Parse(dt.Rows[n][
"Week1"
].ToString());
}
}
|
首先，你就不能用 var row=dt.Rows[n] 替代吗?其次，直接用int.Parse效率多低？再次，dt.Rows[n]["Week0"]为NULL怎么办？
不要重复发明轮子
我们再来看看其他的一些代码：
|
1
2
3
4
5
6
7
8
9
10
11
12
13
|
|
最低0.47元/天 解锁文章
344
被折叠的 条评论
为什么被折叠?
到【灌水乐园】发言
