有学生问我，重构是什么？我应该如何回答？

桃扇骨 2024-03-24 18:02 50阅读 0赞

重构到底是什么？只是代码的推倒重新编码？还是有规则、有方法可寻？当然，结论肯定是有的，本文，我们通过一个简单的实例，来理解一下重构。

## 1.借助一个实例需求 ##

这是一个影片出租店用的程序，计算每一位顾客的消费金额并打印详单。操作者告诉程序：顾客租了哪些影片、租期多长，程序便根据租赁时间和影片类型计算出费用。影片分为三类：普通片、儿童片和新片。除了计算费用，还要为常客计算积分，具体的租赁用户积分规则为：

> 租赁规则
> 
>  *  价格计算规则：
>  *  普通片儿 —— 起步价2¥，超过2天的部分每天每部电影收费1.5元
>  *  新片儿 —— 每天每部3元
>  *  儿童片 —— 起步价2¥，超过3天的部分每天每部电影收费1.5元
> 
> 积分计算规则：
> 
>  *  每借一部电影积分加1，新片每部加2

## 2. 实现&重构 ##

我们很容易实现了代码，类图如下：  
![在这里插入图片描述][d96dc625f36143198564419604dfa75b.png]

但是此时考虑几个需求，

/**
    * 打印顾客的订单详情
    * TODO 函数复杂
    * TODO 如果有需求，需要更改打印样式，或者换一个html样式，那么需要把statement copy一次
    * TODO 如果需要修改计价规则，则需要变更所有的计价函数
    * TODO 如果需要新增类型，则需要变更过所有的函数
    * @return
    */

自然而然，我们首先想到statment函数，功能太复杂了，那我们需要吧这个函数功能分解，最简单的，计算价格，应该分离出来，根据输入的影片类型还有租借天数，得到了租赁的价格？

**Extract method：将方法抽离**

/**
    * Extract method
    * @param rentUnit
    * @return
    */
    private double getRentPrice(RentUnit rentUnit) {
        
    	double temp = 0;
    	switch (rentUnit.getMovie().getMovieType()) {
        
    		case NEW:
    			temp = rentUnit.getDays() * 3;
    			if (rentUnit.getDays() > 2) {
        
    				temp += (rentUnit.getDays() - 2) * 2.5;
    			}
    			break;
    		case NORMAL:
    			temp = rentUnit.getDays() * 2;
    			if (rentUnit.getDays() > 2) {
        
    				temp += (rentUnit.getDays() - 2) * 1.5;
    			}
    			break;
    		case CHILDREM:
    			temp = rentUnit.getDays() * 1;
    			if (rentUnit.getDays() > 2) {
        
    					temp += (rentUnit.getDays() - 2);
    			}
    			break;
    		}
    	return temp;
    }

这时大家发现，抽离的方法，依然有问题，因为切记：任何一个傻瓜都可以写出计算机理解的代码，但是唯有写出人类容易理解的代码，才是优秀的程序员。  
所以，这里我们采用 **Rename field and method**，继续优化

大家此时发现没有，其实这个方法，和Customer没有关系的，是和租赁类有关系的，也就是每个租赁实体类，应该有这样一个方法，可以计算返回它的租赁价格  
所以我们采用Move method，移到合适的类中

![在这里插入图片描述][790b4eb6c13d488bab2257b654c4885d.png]

积分规则，同样如此操作  
![在这里插入图片描述][604561886fff4fe69ef7fb720bb5865d.png]

此时还有什么问题？计价规则和积分规则，其实是日后最容易变动的地方，所以我们需要将其抽离  
![在这里插入图片描述][c462bb0adaf640f3a91253deea9c6a96.png]

大家返回来看，通过一系列简单的提炼操作，是否，现在代码对于需求的兼容性更高了呢？  
例如：

*  我现在要改变积分规则或者计价规则，只需修改RentUnit类即可
 *  我现在想要添加一个htmlstatement打印函数，那么也可以自己调用相应的价格和积分计算函数

但是需要考虑一个事情，影片分类增加怎么办？某种影片类型的计价规则或者积分规则发生变化，不应该是整体发生变化？其实关键在于switch语句，每次修改，都需要修改这语句，对于代码整体健壮性来说，肯定是不对的。

那么我们把计价规则和积分规则，先抽离到Movie类中，这时，有人会说，这就简单了，只需要去新增不同的Movie类，然后去继承Movie，从而实现计价规则和积分规则的变动，但是大家切记，如果是在一个大的系统中，那么这样将是灾难性的后果，因为这样变更之后，意味着上层所有调用Movie实例的地方，都必须去区分到底想要去调用哪种类型？所以的地方都需要去改

但是我们站在开发使用的角度来讲，我新建一个Movie类，只需要告诉你类型就可以，我不想关心这么多的东西，我只想告诉你类型，你让我新增相应的计价规则和积分规则即可。  
有两个东西需要去做

*  一个是switch语句，需要借助多态性，去除
 *  另外一个，不可以直接movie继承的方式去搞，不然上层就得跟着变动，而且这样设计，后期上层的使用上也会诸多不便

![在这里插入图片描述][8b50c12d50e540a4a90e8d63af7c3b9b.png]

重构到这里，实例的所有需求都已兼容，而且都是在不影响最上层调用的前提下完成的，最主要我们是一步一步配合测试完成的  
接下来思考，这里还有什么问题？

后期如果新增影片类型，那么需要修改枚举定义中添加类型，还需新增具体的影片的计价规则和积分计算规则，也就是新增一个price类即可。

*  但是这里有一个问题，就是，需要修改Movie类，因为这里有一个根据类型，去新建price的switch语句，那么这里应该怎么去优化呢？

[d96dc625f36143198564419604dfa75b.png]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/24/2ccce2f452f940d2b0b7e0868fa64b9f.png
[790b4eb6c13d488bab2257b654c4885d.png]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/24/e8d5083f86be47529b1207beda96804d.png
[604561886fff4fe69ef7fb720bb5865d.png]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/24/51d6f23e1eda4d6cb9e784ba6fbf8b55.png
[c462bb0adaf640f3a91253deea9c6a96.png]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/24/499c2cdb8c8b4305bc340ebccbb54bfe.png
[8b50c12d50e540a4a90e8d63af7c3b9b.png]: https://image.dandelioncloud.cn/pgy_files/images/2024/03/24/34e8a8f3af954bfdb5555b52f9bfc741.png