本文用eclipse的自动重构功能对一个程序实例进行重构,目的是探索Eclipse自动重构可以在多大程度上辅助重构这个过程。程序实例使用《Refactoring:Improving the Design of Existing Code》一书中的例子。 Eclipse的自动重构功能能够很好地支持各种程序元素的重命名,并自动更新相关的引用。Eclipse能够支持方法、字段在类之间移动,并自动更新引用。Eclipse较好地支持内联字段、函数的更新替换。Eclipse较好地支持抽取方法、变量等程序元素。 重构的过程是一个不断尝试和探索的过程。Eclipse的重构支持撤销和重做,并且能够预览重构结果,这些是很实用的功能。 Eclipse的重命名、抽取方法、移动、内联功能、更改方法特征符等代码结构级别的重构方法,是比较成熟同时也值得使用的功能。至于设计结构上的重构,eclipse还不能很好地支持。但是作者相信,自动重构的理念应该是"工具辅助下的重构工作",人仍然承担大部分重构工作。 一、预备工作 本文使用《Refactoring:Improving the Design of Existing Code》一书第一章的例子。重构前的代码及每一步重构后的代码见附件。读者最好配合《Refactoring:Improving the Design of Existing Code》一书阅读本文。 Eclipse使用如下版本:
"; while(rentals.hasMoreElements()){ Rental each = (Rental)rentals.nextElement();double thisAmount = amountFor(each); frequentRenterPoints ++; if((each.getMovie().getPriceCode())==Movie.NEW_RELEASE &&each.getDaysRented()>1) frequentRenterPoints ++; result += " " + each.getMovie().getTitle() + " " +String.valueOf(thisAmount) + "
"; totalAmount += thisAmount; } result += "Amount owed is " + String.valueOf(totalAmount) + "
"; result += "You earned " + String.valueOf(frequentRenterPoints) + " frequent renter points"; return result;}/*** @param each* @return*/private double amountFor(Rental each) { double thisAmount = 0; switch(each.getMovie().getPriceCode()){ case Movie.REGULAR: thisAmount += 2; if(each.getDaysRented()>2) thisAmount += (each.getDaysRented()-2)*1.5; break; case Movie.NEW_RELEASE: thisAmount += each.getDaysRented()*3; break; case Movie.CHILDRENS: thisAmount += 1.5; if(each.getDaysRented()>3) thisAmount += (each.getDaysRented()-3)*1.5; break; } return thisAmount;} 2、选中amountFor()的参数each,在右键菜单中选择"重构/重命名",在对话框中输入新的名称:aRental,选择确定,amountFor()中所有each的引用全部被替换成新的名称。用同样的办法修改amountFor()中的局部变量thisAmount为result。重构后的amountFor()代码如下:/*** @param aRental* @return*/private double amountFor(Rental aRental) { double result = 0; switch(aRental.getMovie().getPriceCode()){ case Movie.REGULAR:result += 2; if(aRental.getDaysRented()>2)result += (aRental.getDaysRented()-2)*1.5; break; case Movie.NEW_RELEASE:result += aRental.getDaysRented()*3; break; case Movie.CHILDRENS:result += 1.5; if(aRental.getDaysRented()>3) result += (aRental.getDaysRented()-3)*1.5; break; } return result;}