分享
 
 
 

敏捷开发的必要技巧:移除重复代码

王朝other·作者佚名  2008-05-31
窄屏简体版  字體: |||超大  

重复代码是怎么产生的?

请观察下面的代码,我们已经有一个根据出租记录的id取出租用客户姓名的方法:getCustomerName。

public class BookRental { //该类描述出租记录

String id;

String customerName;

...

}

public class BookRentals {

private Vector rentals;

public String getCustomerName(String rentalId) { 根据出租id取出客户姓名

for (int i = 0; i < rentals.size(); i++) {

BookRental rental = (BookRental) rentals.elementAt(i);

if (rental.getId().equals(rentalId)) {

return rental.getCustomerName();

}

}

throw new RentalNotFoundException();

}

}

public class RentalNotFoundException extends Exception {

...

}

假定现在你要增加一个新的方法,该方法是根据出租记录的id删除该记录,你把这方法命名为deleteRental(String rentalId)。现在你已经考虑到,就像getCustomerName这个方法一样,也要一个一个遍历出租记录。所以你就将getCustomerName这个方法里面的一些代码拷出来,然后稍微修改一下:

public class BookRentals {

private Vector rentals;

public String getCustomerName(String rentalId) {

for (int i = 0; i < rentals.size(); i++) {

BookRental rental = (BookRental) rentals.elementAt(i);

if (rental.getId().equals(rentalId)) {

return rental.getCustomerName();

}

}

throw new RentalNotFoundException();

}

public void deleteRental(String rentalId) {

for (int i = 0; i < rentals.size(); i++) {

BookRental rental = (BookRental) rentals.elementAt(i);

if (rental.getId().equals(rentalId)) {

rentals.remove(i);

return;

}

}

throw new RentalNotFoundException();

}

}

现在这样的代码看起来怎么样?不怎么样,两个方法有大多的同样的代码了。

移除重复代码吧!

要移除所有的重复代码,你可以将BookRentals这个类修成如下的样子(也就是“重构”了):

public class BookRentals {

private Vector rentals;

public String getCustomerName(String rentalId) {

int rentalIdx = getRentalIdxById(rentalId);

return ((BookRental) rentals.elementAt(rentalIdx)).getCustomerName();

}

public void deleteRental(String rentalId) {

rentals.remove(getRentalIdxById(rentalId));

}

private int getRentalIdxById(String rentalId) { //新增加的一个方法

for (int i = 0; i < rentals.size(); i++) {

BookRental rental = (BookRental) rentals.elementAt(i);

if (rental.getId().equals(rentalId)) {

return i;

}

}

throw new RentalNotFoundException();

}

}

为什么我们要移除重复代码?

我来向各位程序员同学稍微说一下,在BookRentals这个类中,rentals这个属性的类型是Ventor,假如我们需要将它改为数组,那我们就必须将所有的"rentals.size()"改为"rentals.length". 在重构以后的版本中,我们只需要在getRentalIdxById这个方法中修改一次,而在原来的版本,我们就得在getCustomerName跟deleteRental两个方法中都改一次。类似的,我们还要将所有的"rentals.elementAt(i)" 改为 "rentals[i]". 又是改一次跟改两次的比较!

大多数情况中,假如类似这样的代码在10个地方重复,当我们修改代码的时候,就要修改10个地方,我们并不能保证能把这10个地方都记住了,而一旦漏掉了几个地方,等待我们的,是一处一处的错误去修复。而最致命的是,当我们修改的是业务逻辑时,这时候,不管我们漏掉了几个地方,IDE都不会报错,那么,等待我们的,将是一堆Bug去检查,而造成的一些bug中,很可能是短时间内还发现不了的. 惨-_-!!

 
 
 
免责声明:本文为网络用户发布,其观点仅代表作者个人观点,与本站无关,本站仅提供信息存储服务。文中陈述内容未经本站证实,其真实性、完整性、及时性本站不作任何保证或承诺,请读者仅作参考,并请自行核实相关内容。
2023年上半年GDP全球前十五强
 百态   2023-10-24
美众议院议长启动对拜登的弹劾调查
 百态   2023-09-13
上海、济南、武汉等多地出现不明坠落物
 探索   2023-09-06
印度或要将国名改为“巴拉特”
 百态   2023-09-06
男子为女友送行,买票不登机被捕
 百态   2023-08-20
手机地震预警功能怎么开?
 干货   2023-08-06
女子4年卖2套房花700多万做美容:不但没变美脸,面部还出现变形
 百态   2023-08-04
住户一楼被水淹 还冲来8头猪
 百态   2023-07-31
女子体内爬出大量瓜子状活虫
 百态   2023-07-25
地球连续35年收到神秘规律性信号,网友:不要回答!
 探索   2023-07-21
全球镓价格本周大涨27%
 探索   2023-07-09
钱都流向了那些不缺钱的人,苦都留给了能吃苦的人
 探索   2023-07-02
倩女手游刀客魅者强控制(强混乱强眩晕强睡眠)和对应控制抗性的关系
 百态   2020-08-20
美国5月9日最新疫情:美国确诊人数突破131万
 百态   2020-05-09
荷兰政府宣布将集体辞职
 干货   2020-04-30
倩女幽魂手游师徒任务情义春秋猜成语答案逍遥观:鹏程万里
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案神机营:射石饮羽
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案昆仑山:拔刀相助
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案天工阁:鬼斧神工
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案丝路古道:单枪匹马
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案镇郊荒野:与虎谋皮
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案镇郊荒野:李代桃僵
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案镇郊荒野:指鹿为马
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案金陵:小鸟依人
 干货   2019-11-12
倩女幽魂手游师徒任务情义春秋猜成语答案金陵:千金买邻
 干货   2019-11-12
 
推荐阅读
 
 
 
>>返回首頁<<
 
靜靜地坐在廢墟上,四周的荒凉一望無際,忽然覺得,淒涼也很美
© 2005- 王朝網路 版權所有