Method Object理解与应用

原创 2018年04月15日 22:22:59

背景

这段时间在重构自己写过的烂代码(尴尬)。本着单一原则想把堆成一堆的代码按照功能拆分成一个个小方法,按照顺序整齐摆放,清晰好看易读有条理。但是遇到了问题,这堆代码涉及到了较多的变量,而且不同部分代码的变量都是另一部分代码需要的参数,所以没有办法单纯的抽出成一个个小方法。

举个例子

比如这一堆代码是一个更新活动的逻辑,其中包含活动的时间转换,活动信息校验,活动额外信息的获取,活动信息拼装以及更新等操作。(当然,实际情况比这个复杂,这里只是作为一个简单的例子来看,而且这个例子单纯是为了解释Method Object,也许有其他更好的重构方式)

/**
 * 前面的一堆代码
 * ……
 */

// 1.入参时间转换
long startTime = getStartTime(actParam.getStartTime());
long endTime = getEndTime(actParam.getEndTime());

// 2.活动信息查询以及校验
ActBean actBean = getActBean(actParam.getActId());
checkRepeatAct(actBean,startTime,endTime);

// 3.获取活动额外信息
ExtraActInfo extraActInfo = getExtraActInfo(actBean);
checkExtraActInfo(extraActInfo);

// 4.活动信息拼装以及更新操作
ActBean updateActBean = getUpdateActBean(startTime, endTime, actBean, extraActInfo);
actService.updateActBean(updateActBean);

/**
 * 后面的一堆代码
 * ……
 *
 */

需求

我需要将这堆代码中的1、2、3部分代码放到一个叫checkActInfo的大方法里,统一做校验以及信息的修正。之后里面的每一部分的校验都封装成一个小方法,只做这部分信息的校验以及转换。比如把第1部分改成checkTime等。但是问题就出现了,1、2、3部分变成一个大方法后,在第4部分需要的入参,没办法通过checkInfo全部返回。而在checkActInfo方法里,如果第1部分变成了一个checkTime方法,那么第2部分中需要的startTime和endTime也没办法通过checkTime全部返回。这样就没办法直接extract method了。

Method Object

于是大佬在我旁边指了一下,说:用Method Object啊,你去看看《重构改善既有代码的设计》这本书。

何为Method Object

以函数对象取代函数(Replace Method with Method Object)
你有一个大型函数,其中对局部变量的使用使你无法采用 Extract Method (提炼函数)。
将这个函数放进一个单独对象中,如此一来局部变量就成了对象内的字段。然后你可以在同一个对象中将这个大型函数分解为多个小型函数。

我的理解就是,以函数对象取代函数实质上是转变一下思维,将一个method(方法)抽象成一个object(对象),而之前方法用到的局部变量都变成了对象中的变量。这样的话,就可以把堆在一起的代码分成几个小方法,而且也不用考虑把每个小方法里处理过的参数返回到主方法里,再传递给其他方法了。因为之前对每个参数的处理,都是对重构后方法对象里的变量做了处理。这样就可以肆无忌惮的分拆这个大方法了。

分拆!

书上说

我们来看看书中介绍的操作流程

  1. 建立一个新类,根据待处理函数的用途,为这个类命名。
  2. 在新类中建立一个final字段,用以保存原来大型函数所在的对象。我们将这个字段称为“源对象”。同时,针对原函数的每个临时变量和每个参数,在新类中建立一个对应的字段保存之。
  3. 在新类中建立一个构造函数,接收源对象及原函数的所有参数。
  4. 在新类中建立一个compute()函数。
  5. 将原函数的代码复制到compute()函数中。如果需要调用源对象的任何函数,请通过源对象字段调用。
  6. 编译。
  7. 将旧函数的函数本体替换为这样一条语句“创建上述新类的一个新对象,而后调用其中的compute()函数”。

我们应该怎么做

我建议直接用idea自带的功能,选中需要重构的代码,右键→Refactor→Extract→Method Object。在弹出来的对话框里填写要生成的类的名称checkActInfo。新生成的checkActInfo类如下:

private class CheckActInfo {
    private ActParam actParam;
    private long startTime;
    private long endTime;
    private ActBean actBean;
    private ExtraActInfo extraActInfo;

    public CheckActInfo(ActParam actParam) {
        this.actParam = actParam;
    }

    public long getStartTime() {
        return startTime;
    }

    public long getEndTime() {
        return endTime;
    }

    public ActBean getActBean() {
        return actBean;
    }

    public ExtraActInfo getExtraActInfo() {
        return extraActInfo;
    }

    public CheckActInfo invoke() {
        // 1.入参时间转换
        startTime = getStartTime(actParam.getStartTime());
        endTime = getEndTime(actParam.getEndTime());

        // 2.活动信息查询以及校验
        actBean = getActBean(actParam.getActId());
        checkRepeatAct(actBean,startTime,endTime);

        // 3.获取活动额外信息
        extraActInfo = getExtraActInfo(actBean);
        checkExtraActInfo(extraActInfo);
        return this;
    }
}

CheckActInfo类中的私有变量包含了我们源代码块中的变量参数。构造方法通过接收入参actParam实例化CheckActInfo对象,同时调用CheckActInfo中的invoke方法,会执行之前的那些代码块,并返回一个新的CheckActInfo对象。我们只需要从这个CheckActInfo对象中获取到需要的变量,拼装出一个updateActBean对象就可以按照原有的逻辑更新对象了。这样的话,我们就成功的完成了Replace Method with Method Object。

进一步完善

当然,只是Replace Method with Method这样还不够,我们的目标是把大方法分拆,不过分拆就容易多了。只要肆无忌惮的按照原有的思路把三部分分拆成三个小方法,就ok了。不过这里有个问题,因为之前的方法是有序的,所以我们可以写一个excuteAll的方法,把之前的方法都串起来执行,无需在外面的主方法里依次调用了。改进如下:

private class CheckActInfo {
    private ActParam actParam;
    private long startTime;
    private long endTime;
    private ActBean actBean;
    private ExtraActInfo extraActInfo;

    public CheckActInfo(ActParam actParam) {
        this.actParam = actParam;
    }

    public long getStartTime() {
        return startTime;
    }

    public long getEndTime() {
        return endTime;
    }

    public ActBean getActBean() {
        return actBean;
    }

    public ExtraActInfo getExtraActInfo() {
        return extraActInfo;
    }
    public CheckActInfo transferTime() {
        // 1.入参时间转换
        startTime = getStartTime(actParam.getStartTime());
        endTime = getEndTime(actParam.getEndTime());
        return this;
    }
    public CheckActInfo checkRepeatAct() {
        // 2.活动信息查询以及校验
        actBean = getActBean(actParam.getActId());
        checkRepeatAct(actBean, startTime, endTime);
        return this;
    }

    public CheckActInfo checkAndGetExtraActInfo() {
        // 3.获取活动额外信息
        extraActInfo = getExtraActInfo(actBean);
        checkExtraActInfo(extraActInfo);
        return this;
    }
    public CheckActInfo executeAll(){
        this.transferTime().checkRepeatAct().checkAndGetExtraActInfo();
        return this;
    }
}

之前的主方法调用:

/**
 * 前面的一堆代码
 * ……
 */

CheckActInfo checkActInfo = new CheckActInfo(actParam).executeAll();
// 活动信息拼装以及更新操作
ActBean updateActBean = getUpdateActBean(checkActInfo.getStartTime(), checkActInfo.getEndTime(), 
        checkActInfo.getActBean(), checkActInfo.getExtraActInfo());
actService.updateActBean(updateActBean);
/**
 * 后面的一堆代码
 * ……
 */

完成!

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

EJB商业探讨的白皮书

1 业界支持  任何一个商业化的技术要想拥有良好的市场态势,业界内重要企业的大量参与是十分重要的。J2EE是由SUN引导,各厂商共同发起的,并得到广泛认可的工业标准。业内企业计算领域的大企业如:IBM...
  • DrunkenLion
  • DrunkenLion
  • 2001-06-20 08:49:00
  • 656

Java动态代理的自己理解

前几天看到java的动态代理机制,不知道是啥玩意,然后看了看。死活不知道 nvoke(Object proxy, Method m, Object[] args)种的proxy是个什么东西,放在这里...
  • zhiaizj1112
  • zhiaizj1112
  • 2013-04-15 16:13:11
  • 587

ognl.MethodFailedException: Method "setDate" failed for object解决

环境:MyEclipse6.5+ Tomcat6.0.18+Struts2.1+Hibernate3.2+Spring2.5.6+MySQL Server 5.1 问题:通过时间查询记录列表时报如下...
  • Wentasy
  • Wentasy
  • 2012-08-09 09:50:55
  • 4465

python笔记四-字典

1、当索引不好用时候,使用字典。 2、
  • deerleaper
  • deerleaper
  • 2014-08-27 09:37:30
  • 1706

java学习笔记14--动态代理

InvocationHandler接口 public interface InvocationHandler{ public Object invoke(Object proxy,Method ...
  • itmyhome
  • itmyhome
  • 2013-05-21 00:29:09
  • 3590

Method "login" failed for object com.java.action.UserAction@779c656b解决方法

这是我的第一篇博客,有点小紧张。哈哈。 很高兴以前为我们解决问题的大神博客,所以我自己也学着写一下我在学习的过程之中遇到的问题,供大家参考下。 Method “login” failed for ...
  • cjs520131499
  • cjs520131499
  • 2017-04-09 23:03:02
  • 2561

struts2异常ognl.MethodFailedException: Method "setId" failed for object

页面传递的数据类型错误导致struts2抛出ognl.MethodFailedException异常。 解决方式: 修改log4j的日志输出级别. log4j.properties文件添...
  • sd4015700
  • sd4015700
  • 2013-05-20 10:41:00
  • 4754

ognl.MethodFailedException: Method "setXX" failed for object

由于数据类型不匹配而导致的异常ognl.MethodFailedException: Method "setId" failed for object cn.xyurp.system.bean.Dom...
  • id19870510
  • id19870510
  • 2010-05-12 10:03:00
  • 31427

零基础学python-9.2 文件

1.文件的操作: 操作 解释 output=open(r'c:\123.txt','w') 创建输出文件,w写入 input=open('data','r') 创...
  • raylee2007
  • raylee2007
  • 2015-08-29 22:46:47
  • 2389

报错:HTTP Status 500 -Method "execute" failed for object……

整合hibernate和Struts2的时候,报出了一个错误,检查了一遍所有的方法,都是正确的,但是运行起来,还是报这个错误 本来是将hibernate的包放在 Struts2...
  • limin0017
  • limin0017
  • 2015-11-12 20:52:49
  • 4199
收藏助手
不良信息举报
您举报文章:Method Object理解与应用
举报原因:
原因补充:

(最多只允许输入30个字)