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

news2024/11/24 12:08:00

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

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://www.coloradmin.cn/o/426282.html

如若内容造成侵权/违法违规/事实不符,请联系多彩编程网进行投诉反馈,一经查实,立即删除!

相关文章

管理笔记2职级管理

一职级目的: 1 员工发展的职业通道 2 招聘、晋升的公平性 二 能力模型 类似职级 三 晋升 当前级别的事做到了,已经做到下一个级别的事 考虑点:1考虑当前阶段时间,不会频繁晋升,2考虑绩效。 不断成长、主动做事&a…

认证授权SpringSecurity

如何引入SpringSecurity作为项目的权限认证服务 1.引入依赖 <dependency><groupId>org.springframework.cloud</groupId><artifactId>spring-cloud-starter-security</artifactId> </dependency> <dependency><groupId>org.…

客快物流大数据项目(一百一十三):注册中心 Spring Cloud Eureka

文章目录 注册中心 Spring Cloud Eureka 一、Eureka 简介 二、架构图

不得不了解的linux网络配置

目录一、查看及测试网络1.1查看网络配置1.1.1ifconfig命令—查看网络接口地址1.1.2route命令—查看路由条目1.1.3hostname命令—查看主机名称1.1.4netstat命令—查看网络连接情况1.1.5 ss命令/lsof 命令— 获取socket(套接字)统计信息1.2测试网络连接1.2.1ping命令—测试网络连…

1 Nginx跨域配置

跨域问题在之前的单体架构开发中&#xff0c;其实是比较少见的问题&#xff0c;除非是需要接入第三方SDK时&#xff0c;才需要处理此问题。但随着现在前后端分离、分布式架构的流行&#xff0c;跨域问题也成为了每个Java开发必须要懂得解决的一个问题。 跨域问题产生的原因 产…

项目管理职业发展前景怎么样?

项目管理职业发展前景怎么样&#xff1f;我们可以从这三个问题出发—— 第一个问题&#xff1a;在中国「项目管理」的专业性的体现/认同度如何&#xff1f;缺少专业知识对未来工作选择的限制程度如何&#xff1f; 由于现在有不少跨专业从业者&#xff0c;他们通过自学考证等途…

计算广告(十二)

FFM模型 FFM&#xff08;Field-aware Factorization Machine&#xff0c;领域感知因子分解机&#xff09;是一种广泛应用于推荐系统和点击率预测&#xff08;CTR&#xff09;等任务的机器学习模型。 它是基于FM&#xff08;Factorization Machine&#xff0c;因子分解机&…

nuscenes instance 调研笔记

nuscenes instance 调研笔记&#xff1a; 前言 nuscenes这个自动驾驶数据集挺大&#xff0c;官方devkit还提供了各种操作用于提高科研人员的效率&#xff0c;但是吧&#xff0c;东西多了学起来就乱七八糟的&#xff0c;本文仅以提取每个instance关联的所有images为例子&#…

USRP具有MIMO系统如何同步

MIMO系统要求 时间和频率同步&#xff0c;并且每个通道必须满足下面两个要求&#xff1a; 1&#xff09;时钟必须同步对齐 2&#xff09;对DSP操作时间也要对齐&#xff0c;来自同一的时钟边缘 波束成形和测向要求 除了采样时间和采样时钟对准外&#xff0c;系统还必须在每个…

华为OD机试 - 相同数字组成图形的周长(Java JS Python)

题目描述 有一个6464的矩阵,每个元素的默认值为0,现在向里面填充数字,相同的数字组成一个实心图形,如下图所示是矩阵的局部(空白表示填充0): 数字1组成了蓝色边框的实心图形,数字2组成了红色边框的实心图形。 单元格的边长规定为1个单位。 请根据输入,计算每个非0值…

java 数组和字符串操作

目录一、前言二、数组操作1.charAt操作2.getBytes操作3.toCharArray操作4.String.valueOf5.substring&#xff0c;toUpperCase&#xff0c;toLowerCase&#xff0c;concat6.indexOf7.Arrays使用一、前言 本文章主要讲解数组的一些基本操作&#xff0c;让我们写代码更加方便&am…

1.docker-安装及使用

1.安装步骤 Install Docker Engine on CentOS 1. 确定CenOS7及以上版本 cat /etc/redhat-release2.卸载旧版本 yum remove docker \docker-client \docker-client-latest \docker-common \docker-latest \docker-latest-logrotate \docker-logrotate \docker-engine3.yum安…

软件测试岗位求职避坑,今年毁约应届生的公司有这么多?….

不知不觉九月已至&#xff0c;“金三银四”招聘旺季也落入尾声&#xff0c;大学生也迎来毕业季&#xff01; 这个时候&#xff0c;应届生开始集体走向社会&#xff0c;而职场人也迎来了跳槽涨薪的好时机。 可现实的求职情况却是一片困境&#xff1a;从中小微到“大厂”&#…

ICPC SWERC 2020 K - Unique Activities(SAM记录子串第一次出现的位置 or SAM + hash)

两种做法的效率差异 做法一&#xff1a;SAM记录子串第一次结束位置 做法二&#xff1a;SAM hash 题意&#xff1a; 给定一个字符串&#xff0c;让你找到只出现过一次&#xff0c;且长度最短的子串并输出&#xff0c;如果有多个则输出最先出现的那个。 思路&#xff1a…

迷宫问题-DFS-BFS

迷宫问题迷宫问题简介BFS解决迷宫最短路径问题DFS记录迷宫路径DFS解决迷宫所有路径问题迷宫问题简介 &#x1f680;学习过算法程序设计的应该都学习过迷宫这个问题&#xff0c;迷宫问题主要设计的算法就是DFS-深度优先遍历和BFS-广度优先遍历。 &#x1f680;在一个二维数组中…

Jmeter和Testlink自动化测试框架研究与实施

摘 要 目前基于Jmeter的接口自动化测试框架&#xff0c;大多只实现脚本维护和自动调度&#xff0c;无法与Testlink进行互通&#xff0c;实现测试方案与自动化实施流程连接&#xff0c;本文基于Testlink、Jmeter、Jenkins实现&#xff1a;通过Testlink统一维护接口自动化测试用…

搭建es集群

单点es的缺点&#xff1a;1.存储数据有限&#xff1b;2.单点故障问题&#xff08;es出现故障则整个服务会直接宕机&#xff09; 解决存储数据有限&#xff1a;搭建多台es服务器实现集群 解决单点故障&#xff1a;在不同的es服务器中进行备份数据&#xff08;例&#xff1a;在…

车载测试需要有哪些知识需要学习的?

一、车载行业前景 其中的车载测试也随着国家对新能源、智能驾驶等领域的支持&#xff0c;而异常活跃&#xff0c;目前我国共有9000家自动驾驶相关企业&#xff0c;而今年从华为、中兴、大唐等通信领域的企业到以阿里、腾讯、小米等为代表的互联网企业&#xff0c;均已布局自动…

浅谈继承和发扬传统文化路径

中华民族五千年文明历史,造就了博大精深的中华传统文化。如何继承和发扬传统文化,是当代文化爱好者和工作者所关注的&#xff0c;实现这一使命,文化传承需要在以下路径上发力: 1. “微更新”路径。在传承的基础上进行融合拓展,实现内涵丰富和更新。 2. “强保护”路径。利用立…

空压机远程监控系统解决方案

一、项目背景 随着物联网各种技术快速发展,各物联网远程监测应用场景也应用而生&#xff0c;空压机是一种空气压缩和气体输送设备&#xff0c;广泛运用于矿山、机械、电子、医疗等各行业。空压机常规都是需要人在现场监测和维护,现在通过物联网技术,远程监控成为可能&#xff…