重复代码
第一个例子:
第一个方法:getCount
第二个方法:pageQueryKbdb
上面两个方法中,第一个是求总数,第二个方法是分页查询详细内容。
这两个方法的SQL中除了select count(ID)不一样,其他一模一样,这样的代码,如果将来SQL变了,你需要同时修改两个地方,如果没有注意,两个SQL不一致,结果会差很多。
上面的的SQL完全可以从from KBJCJL这里开始单独创建一个方法,方法返回一个String类型的SQL即可。
将上面的两个方法修改后如下:
增了一个方法getKbdsSql
原来的两个方法修改为:
上面这种修改方式很容易,主要是因为这两个SQL差异太小,但是如果JAVA处理SQL时,不同的地方很多,可能需要上面的方法再拆分成多个方法重新组合。
这个类中,还有多处类似的代码,看下面的GetClCount方法:
你觉得这个方法眼熟吗?
但是这个方法写的和前面的两个方法不一样,但是逻辑呢?
其实逻辑一模一样,而且结果也一样,并且这里写的switch-case比上面的要规范,为什么?
因为这里的case是从1到18排列的,并且其中的13、14是用default处理的。
只有13对应的值不一样
所以这看着不一样的代码却做着几乎相同的事,非常的不合理。
完全可以在提取的方法中,单独写出case 13进行处理。
然后GetClCount方法完全可以调用修改后的方法。
记住:
a. 如果有一段代码出现了两次,如果代码很短(4,5行),而且结构不是很好,可以直接复制。如果代码很长,将代码独立出来。
b. 如果方法出现了两次以上,将代码独立出来。
c. 不同类如果调用相同的方法,可以独立到一个公共类中,所有的类都调用这一个类的方法,而不是把该方法分别复制到每个类中。
不知道你们觉得上面的方法修改之后感觉怎么样??????
如果你觉得这样就可以了,那你就错了。
你有没有注意到这样的代码:
像这种代码,如果维护起来,极其的不方便。
你应该注意到这里的代码有一个特点,就是case对应的值,并且这些值只是在SQL中使用的,那么为什么不将这种对应关系存入到数据库中呢?
如果将上面的关系存入到表中,那么完全不需要switch处理,直接用SQL查询关联就可以处理。