重构笔记——以查询取代临时变量

版权声明:本文为博主原创文章,未经博主允许不得转载。 https://blog.csdn.net/pistolove/article/details/42386305

本文是在学习中的总结,欢迎转载但请注明出处:http://blog.csdn.net/pistolove/article/details/42386305


        在上一篇文章中介绍了“ 内联临时变量“。本文将介绍“以查询取代临时变量”这种重构手法。

        下面让我们来学习这种重构手法吧。


开门见山

        发现:你的程序以一个临时变量保存某一表达式的运算结果。

        解决:将这个表达式提炼到一个独立函数中。将这个临时变量的所有引用点替换为对新函数的调用。

	//重构前
	double basePrice = _quantity * _itemPrice;
	if(basePrice > 5000)
		return basePrice * 0.95;
	else
		return basePrice * 0.98;
	//重构后
	if(basePrice() > 5000)
		return basePrice() * 0.95;
	else
		return basePrice() * 0.98;
	...
	
	double basePrice(){
		 return  _quantity * _itemPrice;
	}


动机

        我们都知道临时变量都是暂时的,而且只能在所属的函数中使用。所以它们会驱使你写出更长的函数,因为只有这样你才能访问到需要的临时变量。如果把临时变量替换为一个查询,那么同一个类中所有函数都将可以获得这份信息。这将带给你极大帮助,使你能够为这个类编写出更清晰的代码。

        该重构方法往往是你运用提炼函数之前必不可少的一个步骤。局部变量会使得代码难以提炼,所以应该尽可能把它们替换为查询式。比较简单的情况是:临时变量只被赋值一次,或者赋值给临时变量的表达式不受其它条件影响。


做法

        对于简单的情况:
(1)找出只被赋值依次的临时变量(如果被使用多次,考虑将其分割成多个变量,对应在后续重构方法中)。
(2)将该临时变量声明为final。
(3)编译(确保该临时变量的确只被赋值一次)
(4)将“对该临时变量赋值”之语句的等号右侧部分提炼到一个独立的函数中。(首先将函数声明为private。日后你可能会发现还有好多地方需要使用它,那时放松对它的保护也很容易;确保提炼出的函数无任何副作用,即该函数不修改任何对象内容)
(5)编译,测试。
(6)对该临时变量实施“内联临时变量”重构方法。

        我们常常使用临时变量保存循环中的累加信息。在这种情况下,整个循环都可以被提炼为一个独立的函数,这可以减少原函数中几行循环逻辑代码。该手法的使用可能会让你担心性能上的问题。就像和其它性能问题一样,我们现在不用管它,因为十有八九根本没有造成任何影响。就算真的出问题了,你也可以在优化时期解决它。代码组织良好,你也会发现更有效的优化方案;如果没有进行重构,好的优化方案就可能与你失之交臂。


示例

        我们从一个简单函数开始:
//重构前
double getPrice() {
	int basePrice = _quantity * _itemPrice;
	double discountFactor;
	if (basePrice > 5000)
		discountFactor = 0.95;
	else
		discountFactor = 0.98;
	return basePrice * discountFactor;
}
        我们希望将其中的两个临时变量都去掉。当然,每次一个进行处理。
        代码虽然简单,但是还是有必要先把临时变量声明为final,检查它们是否的确只被赋值一次。
double getPrice() {
	final int basePrice = _quantity * _itemPrice;
	final double discountFactor;
	if (basePrice > 5000)
		discountFactor = 0.95;
	else
		discountFactor = 0.98;
	return basePrice * discountFactor;
}
        这样,一旦有问题编译器就会警告。之所以需要这个做是因为如果临时不安路口不只被赋值一次,就不该进行这项重构。
        下面开始替换临时变量,每次一个,不要着急,按部就班。
        首先,把赋值动作右侧表达式提炼出来:
double getPrice() {
	final int basePrice = basePrice();
	final double discountFactor;
	if (basePrice > 5000)
		discountFactor = 0.95;
	else
		discountFactor = 0.98;
	return basePrice * discountFactor;
}

private double basePrice() {
	return _quantity * _itemPrice;
}
        然后,编译并测试,再开始使用内联临时变量。
        首先把临时变量basePrice的第一个引用点替换掉:
double getPrice() {
	final int basePrice = basePrice();
	final double discountFactor;
	if (basePrice() > 5000)
		discountFactor = 0.95;
	else
		discountFactor = 0.98;
	return basePrice * discountFactor;
}
        编译、测试,然后进行下一个。由于下一个是最后一个引用点,所以把basePrice临时变量的声明式一并去掉:
double getPrice() {
	final double discountFactor;
	if (basePrice() > 5000)
		discountFactor = 0.95;
	else
		discountFactor = 0.98;
	return basePrice()  * discountFactor;
}
        完成basePrice之后,以类似的方法提炼出discountFactor():
double getPrice() {
	final double discountFactor = discountFactor() ;
	return basePrice()  * discountFactor;
}

private double discountFactor(){
	if (basePrice() > 5000)
		return 0.95;
	else
		return 0.98;
}
        你会发现,如果没有把临时变量basePrice替换为一个查询式,将多么难以提炼discountFactor()!
        最终,getPrice()变成这样:
//重构后
double getPrice() {
	return basePrice()  * discountFactor();
}

        本文主要介绍了重构手法——以查询取代临时变量。通过重构后代码可见,代码变的相当地简洁了,代码所表达的意图也变的很清晰了。一旦代码中出现问题,修改起来也比较轻松,因为代码很简短、清晰。我想你是否也慢慢喜欢上重构了呢!如果是,那么就一起学习、进步,让代码变得漂亮吧!
        最后,希望本文对你有所帮助。有问题可以留言,谢谢。(PS:下一篇将介绍重构笔记——引入解释性变量)


重构笔记文章如下

       重构笔记——入门篇

       重构笔记——代码的坏味道(上

       重构笔记——代码的坏味道(下)

       重构笔记——构筑测试体

       重构笔记——提炼函数

       重构笔记——内联函数

       重构笔记——内联临时变量


没有更多推荐了,返回首页

私密
私密原因:
请选择设置私密原因
  • 广告
  • 抄袭
  • 版权
  • 政治
  • 色情
  • 无意义
  • 其他
其他原因:
120
出错啦
系统繁忙,请稍后再试