重构长方法之保留整个对象

在开发中我们会遇到需要从同一个对象中获取多个值的情况,例如从对象rectangle 中获取长方形的宽width和高height,然后将这个两个值传递给方法GetArea去计算面积:

csharp 复制代码
public class Demo
{
	public void Method()
	{
		//---------------
		//more code
		//---------------
		
		float width=rectangle_a.Width;
		float height=rectangle_a.Height;
		float are_a=GetArea(width,height);
		
		//-------------------
		//more code
		//------------------

		float width=rectangle_b.Width;
		float height=rectangle_b.Height;
		float are_b=GetArea(width,height);

		//------------------
		//more code
		//-----------------
	}
	private float GetArea(float width,float height)
	{
		return width*height;
	}
}

这种方式看起来很简单,但是存在一个问题,如果rectangle对象的内部不再通过属性的方式来获取去widthheight,而是通过方法的形式来获取,或者需要再增加获取一个属性 Color 传递个GetArea 方法,那么我们就要大量的修改代码了(前述代码我们要修改两个地方)。那么,要解决这个问题,我们可以将整个对象传递给要调用的方法,让要掉用的方法内部去负责获取这些属性。以下是修改后的代码:

csharp 复制代码
public class Demo
{
	public void Method()
	{
		//---------------
		//more code
		//---------------
		
		float are_a=GetArea(rectangle_a);
		
		//-------------------
		//more code
		//------------------
		
		float are_b=GetArea(rectangle_b);

		//------------------
		//more code
		//-----------------
	}
	private float GetArea(Rectangle rectangle)
	{	
		return rectangle.Width*rectangle.Height;
	}
}

"保留整个对象" 这种重构方式的优点是我们看到的不再是一大堆参数,而是一个具有易懂名称的单一对象。并且如果方法需要从对象中获取非常多的数据,那么无需重写所有使用该方法的地方,只需要在方法内部重写就可以了。缺点也是显而易见的,在某些情况下这种转换会导致方法的灵活性降低,原本可以从许多不同的来源获取数据,但是由于重构,我们只能将其用于具有特定类型的对象。

相关推荐
shawxlee10 小时前
vue3在public下封装config.js自定义配置动态数据,可在打包后直接修改,方便后端部署及后续维护
前端·javascript·经验分享·vue·团队开发·js·项目优化
kyriewen11 小时前
我给前端项目的接口请求套了6层保护——才发现以前一直在裸奔
前端·javascript·面试
郝亚军12 小时前
如何安装webstorm、Node.js和vue CLI
前端·javascript·vue.js
an3174214 小时前
6MB 组织树大文件性能优化全流程
前端·javascript·vue.js
TeamDev15 小时前
JxBrowser 9.3.1 版本发布啦!
java·前端·javascript·c#·混合应用·jxbrowser·浏览器控件
整點薯條15 小时前
Vue3+vite创建项目架构文件说明总结
前端·javascript·vue.js
前端甜糖17 小时前
记录We码开发者工具的一个bug
前端·javascript
猫32817 小时前
echarts 日常问题解决
前端·javascript·echarts
不简说18 小时前
JS 代码技巧 vol.8 — 20 个函数式编程实战,把 if/else 拍扁的骚操作
前端·javascript·面试
Cobyte18 小时前
模板 DSL 解析器中的状态机设计
前端·javascript·vue.js