发现:你有一个复杂的表达式。
解决:将该复杂的表达式(或其中的部分)的结果放进一个临时变量,并以此变量名称来解释表达式用途。
//重构前 if((platform.toUpperCase().indexOf(MAC) > -1) && (browser.toUpperCase().indexOf(IE) > -1) && wasInitialized() && resize > 0) { //do something }
//重构后 final boolean isMacOs = platform.toUpperCase().indexOf(MAC) > -1; final boolean isIEBrowser = browser.toUpperCase().indexOf(IE) > -1; final boolean wasResize = resize > 0; if(isMacOs && isIEBrowser && wasInitialized() && wasResize){ //do something }
在某些情况下,表达式可能非常的复杂以至于难以阅读。这样,临时变量可以帮助你将表达式分解为比较容易管理的形式。
在条件逻辑中,引入解释性变量就显得比较有价值:你可以用这项重构将每个子句提炼出来,以一个良好命名的临时变量来解释对应条件子句的意义。另一种可能的情况是,对于那些比较长的算法,可以运用临时变量来解释每一步运算的意义。
本文的重构手法是比较常见的手法之一,但是对其的使用又不是那么的多。因为一般情况下,我们都可以使用提炼函数来解释一段代码的意义。毕竟临时变量只有在它所处的那个函数中才有意义,局限性较大,函数则可以在对象的整个生命周期中都有用,并且可被其它对象使用。但是,当局部变量使用提炼函数难以进行时,就可以尝试使用引入解释性变量。
//重构前 double price(){ // 价格 = basePrice - quantity discount + shipping return _quantity * _itemPrice - Math.max(0, _quantity - 800) * _itemPrice * 0.15 + Math.min(_quantity * _itemPrice * 0.25, 100); }
double price(){ // 价格 = basePrice - quantity discount + shipping final double basePrice = _quantity * _itemPrice; return basePrice - Math.max(0, _quantity - 800) * _itemPrice * 0.15 + Math.min(basePrice * 0.25, 100); }
double price(){ // 价格 = basePrice - quantity discount + shipping final double basePrice = _quantity * _itemPrice; final double quantityDiscount = Math.max(0, _quantity - 800) * _itemPrice * 0.15; return basePrice -quantityDiscount+ Math.min(basePrice * 0.25, 100); }
//重构后 double price(){ // 价格 = basePrice - quantity discount + shipping final double basePrice = _quantity * _itemPrice; final double quantityDiscount = Math.max(0, _quantity - 800) * _itemPrice * 0.15; final double shipping = Math.min(basePrice * 0.25, 100); return basePrice - quantityDiscount + shipping; }
对于上述代码,通常不以临时变量来解释其动作意图,而是更喜欢使用提炼函数。 让我们从头开始:
//重构前 double price(){ // 价格 = basePrice - quantity discount + shipping return _quantity * _itemPrice - Math.max(0, _quantity - 800) * _itemPrice * 0.15 + Math.min(_quantity * _itemPrice * 0.25, 100); }
double price(){ // 价格 = basePrice - quantity discount + shipping return basePrice() - Math.max(0, _quantity - 800) * _itemPrice * 0.15 + Math.min(basePrice() * 0.25, 100); } private double basePrice(){ return _quantity * _itemPrice; }
//重构后 double price(){ // 价格 = basePrice - quantity discount + shipping return basePrice() - quantityDiscount() + shipping(); } private double basePrice(){ return _quantity * _itemPrice; } private double shipping(){ return Math.min(basePrice() * 0.25, 100); } private double quantityDiscount(){ return Math.max(0, _quantity - 800) * _itemPrice * 0.15; }