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

news/2024/7/8 3:21:44

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

1.借助一个实例需求

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

租赁规则

  • 价格计算规则:
  • 普通片儿 —— 起步价2¥,超过2天的部分每天每部电影收费1.5元
  • 新片儿 —— 每天每部3元
  • 儿童片 —— 起步价2¥,超过3天的部分每天每部电影收费1.5元

积分计算规则:

  • 每借一部电影积分加1,新片每部加2

2. 实现&重构

我们很容易实现了代码,类图如下:
在这里插入图片描述

但是此时考虑几个需求,

/**
* 打印顾客的订单详情
* 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,移到合适的类中

在这里插入图片描述

积分规则,同样如此操作
在这里插入图片描述

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

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

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

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

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

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

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

在这里插入图片描述

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

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

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

http://lihuaxi.xjx100.cn/news/1026355.html

相关文章

OpenAI-ChatGPT最新官方接口《文本交互》全网最详细中英文实用指南和教程,助你零基础快速轻松掌握全新技术(一)(附源码)

Text completion 文本交互前言Introduction 导言Prompt design 提示设计Basics基础知识TroubleshootingClassificationImproving the classifiers efficiency 提高分类器的效率Generation 总结Conversation 对话Transformation 变化Translation 翻译Conversion 转化Summarizati…

【分布式应用】Zabbix——安装

目录 第一章zabbix概述 1.1.什么是监控概念 1.2.zabbix概述 1.3.zabbix 主要特点 1.4.zabbix主要功能 1.5.zabbix运行机制 1.6.zabbix 监控原理 1.7zabbix的架构,数据流向以及原理在图中做总结 1.8.数据采集模式 1.9.zabbix监控模式 第二章.安装 部署 za…

李宏毅教程系列——增强学习

目录 0. 强化学习wiki 1. 介绍 2. Exploration vs Exploitation 探索与开发 3. 各类最优化方法 3.1 Brute force猛兽蛮力法(暴力搜索) 3.2 Value function estimation(价值函数估计) 3.2.1 Monte Carlo methods 蒙特卡洛方…

苹果AirPods耳机推送新固件更新,TWS耳机与Find My可实现智能防丢

苹果推送 iOS / iPadOS 16.5 Beta 2 更新的同时,原本还为 AirPods 耳机推送了固件更新 5E133。苹果在临时撤回之后,再次向用户推送了本次更新, 苹果官方表示当你的 AirPods 在充电且在 iPhone、iPad 或 Mac 的蓝牙通信范围内时,…

JavaScript对象类型之Array及Object

目录 一、Array (1)语法 (2)API 二、Object (1)语法 (2)特色:属性增删 (3)特色:this (4)特色&#xf…

spring boot项目:实现与数据库的连接

步骤【写在前面】定义数据库连接信息:引入数据库驱动:创建数据源:创建JdbcTemplate:编写DAO层:使用Service注解标注Service层:使用RestController注解标注Controller层:示例代码:app…

Flink CDC入门案例

由于Flink CDC是基于日志的方式,因此需要开启MySQL的binlog日志。 开启binlog日志的配置如下 #1.编辑MySQL的配置文件 vim /etc/my.cnf #添加如下内容 [mysqld] log-binmysql-bin # 开启 binlog binlog-formatROW # 选择 ROW 模式 server_id1 # 配置 MySQL replact…

第十九章 弹性池塘

“我?你是想问我的师父是谁?嘻嘻,你猜?谅你也猜不到,我师父是大酒鬼克雷季特!你还没见过这位受人爱戴的老爷爷对吧?嘻嘻,这样说他似乎有些不厚道。等你明天上他的课时就知道了。他的…