|
精华帖 (0) :: 良好帖 (1) :: 新手帖 (0) :: 隐藏帖 (0)
|
|
|---|---|
| 作者 | 正文 |
|
发表时间：2007-06-25
目前流行的新型的MVC框架 几乎都在"增强单元测试能力"上做了很多文章.
目的就是让 Controller 可以脱离web容器单独进行单元测试. 大多数采用的方法都是使 Controller 中的方法的参数 和 返回值 与 j2ee特有的类(如HttpXXXX)无关. 例如 传入的 是若干个 HashMap ,返回的是纯的字符串. 这样在单元测试的时候,只要new一个Controller,准备一些装有测试数据的Map, 然后执行相关的Controller方法,再然后看看返回值,就可以了. 显然,流行的Struts 1.X (>=1.2) 是不具备这样的特性的. 如果您现在有一个遗留的使用Struts 1.X(>=1.2) 的系统,或者因为人力资源的原因,不能立即引入ww或Struts2.x, 那么如何让您的Action类似的可测性了呢(其实除了可测试性,这样的改进往往也会大大的减少代码的开发量). 下面我就把我的一些心得写出来,和大家一起分享,欢迎大家拍砖. 注意:以下讨论均以改造 DispatchAction 为例(不使用DispatchAction的情况应该比较少吧 呵呵). 先说明一下改造的目的: 简化开发,减少代码量 增强单元测试能力 全面兼容现有DispatchAction(这点很重要) 一个传统的 DispatchAction 代码结构
public class MyClassicAction extends DispatchAction {
// 一个添加user的操作
public ActionForward addUser(ActionMapping mapping, ActionForm form,
HttpServletRequest request, HttpServletResponse response)
throws Exception {
// 1 取得参数 ...
// 2 执行相关的BO DAO方法
// 3 取得执行后的返回值
if (添加成功) {
return mapping.findForward("Succee");
}
return mapping.findForward("Fail");
// 还有一种情况是 返回 new ActionForward(....).
}
}
对于上面的代码大家再熟悉不过了,我们现在要做的事情就是通过改造,让上面的action方法变成下面的样子.
// 一个添加user的操作
public String addUser( 传入与web容器无关的参数 )
throws Exception {
// 1 取得参数 ...
// 2 执行相关的BO DAO方法
// 3 取得执行后的返回值
if (添加成功) {
return "Succee";
}
return "Fail";
// 对于action中返回 new ActionForward(....).的情况,我们可以设定另外一种特殊的前缀.
// 例如 return "NEW:/addUserFail.jsp";
}
上面的代码是一个最终的目标,我们先一步一步来. 首先这种改造我们一定要从 Struts 提供的 DispatchAction 做起. 总体思路很明显: 自己写个类 继承 DispatchAction ,并且添加或修改一些方法, 然后项目中的其他的DispatchAction都继承这个自己写的DispatchAction类. 简单说一下Struts 的 DispatchAction的工作流程. 首先 它也是一个action,所以请求来了 他会自动去执行execute方法 execute方法内部做一些操作后,会调用 dispatchMethod 方法 dispatchMethod方法会,调用getMethod方法 来取得欲执行的方法,并且执行之 改造的核心就是围绕最关键的dispatchMethod方法. 而由于我们要"全面兼容现有DispatchAction",所以对于getMethod我们不做修改,而是选择增加一个类似的方法 getMethodTD 首先写一个新的 .DispatchAction基类,他继承org.apache.struts.actions.DispatchAction.
public class TDDispatchAction extends DispatchAction {
protected Class[] typesTD = {TDServletWrapper.class };
protected final static String TD_METHODCACHE_PREFIX="TD_METHODCACHE_PREFIX";
protected final static String TD_DEFAULT_METHOD="defaultMethod";
protected final static String TD_NEW_FORWARD_PREFIX="NEW:";
protected Method getMethodTD(String name)
throws NoSuchMethodException {
String cacheKey=TD_METHODCACHE_PREFIX+name;
synchronized (methods) {
Method method = (Method) methods.get(cacheKey);
if (method == null) {
method = clazz.getMethod(name, typesTD);
methods.put(cacheKey, method);
}
return (method);
}
}
// 对unspecified这个方法的扩展实际上与本次讨论无关,只是为我们框架加的一个小功能
// 提供一个默认的方法.也就是说,如果您的action中,有名为 defaultMethod 的方法,那么无需在请求的参数中加入相关信息,就会自动执行.
protected ActionForward unspecified(ActionMapping mapping, ActionForm form,
HttpServletRequest request, HttpServletResponse response)
throws Exception {
return dispatchMethod(mapping, form, request, response, TD_DEFAULT_METHOD);
}
protected ActionForward dispatchMethod(ActionMapping mapping,
ActionForm form, HttpServletRequest request,
HttpServletResponse response, String name)
throws Exception {
// 后面再说
}
}
首先来看一下这个 getMethodTD 方法. 原getMethod(String name)方法的作用是: 取得一个名字为name,参数类型为 ActionMapping , ActionForm , HttpServletRequest ,HttpServletResponse 的一个方法. 而这个新写的 getMethodTD 方法的作用是: 取得一个名字为name,参数类型为 TDServletWrapper 的一个方法. 这个大家和 原getMethod一对比就能看出来作用和差异了,注意那个cacheKey=TD_METHODCACHE_PREFIX+name,这个是必须的,不能直接用name做key. 其中这个 TDServletWrapper 就是封装的一个"可以"与web容器脱离的对象,注意,这里用的是可以.而不是直接就脱离了. 对此后面再做解释. 要添加的 getMethodTD 方法添加完了. 下面来看一下重点 改造 dispatchMethod方法: 原dispatchMethod方法的流程其实很简单: 1 调用 getMethod(name)方法,取得真正要执行的方法 并执行(传入的参数类型为 ActionMapping , ActionForm , HttpServletRequest ,HttpServletResponse ). 2 返回 方法执行的 结果 (返回结果类型为 ActionForward ). 改造后的目的是: 1 调用 getMethodTD(name)方法,取得真正要执行的方法并执行(传入的参数类型为TDServletWrapper ). 2 取得 方法执行的 结果 (返回结果类型为 String ). 3 根据这个返回的String, 创建ActionForward对象,并返回 但是我们还有一个目的是"兼容",所以最后确定的流程是 1 调用 getMethod(name)方法,取得真正要执行的方法,如果取得,则按原方式继续. 2 如果没有取得 调用 getMethodTD(name)方法,取得真正要执行的方法,若取得则执行. 3 根据返回结果的类型不同 做不同的操作.
// 改造的 dispatchMethod方法,重点看 //TD...字样所在的代码段
protected ActionForward dispatchMethod(ActionMapping mapping,
ActionForm form, HttpServletRequest request,
HttpServletResponse response, String name)
throws Exception {
if (name == null) {
return this.unspecified(mapping, form, request, response);
}
Method method = null;
//TD: Modified by Wei Zijun
boolean useTDMethod=false;
try {
// 按传统方式取 action方法
method = getMethod(name);
} catch (NoSuchMethodException ex) {
try {
// 没取到的话 再按新方式取 action方法
method = getMethodTD(name);
// 标识一下到底是使用的原始方法 还是新方法
useTDMethod=true;
} catch (NoSuchMethodException e) {
String message =
messages.getMessage("dispatch.method", mapping.getPath(), name);
log.error(message, e);
String userMsg =
messages.getMessage("dispatch.method.user", mapping.getPath());
throw new NoSuchMethodException(userMsg);
}
}
ActionForward forward = null;
try {
//TD: Modified by Wei Zijun
Object forwardR=null;
if (!useTDMethod){
// 如果是原始 action 方法
Object[] args = { mapping, form, request, response };
forwardR= method.invoke(this, args);
}else{
// 如果是新 action 方法
// TDServletWrapperFactory.getInstance创建一个封装的对象
Object[] args = { TDServletWrapperFactory.getInstance(mapping, form,request, response) };
forwardR= method.invoke(this, args);
}
if (forwardR!=null){
if (forwardR instanceof ActionForward){
forward=(ActionForward)forwardR;
}else{
// 如果返回值是字符串:
// 以 TD_NEW_FORWARD_PREFIX("NEW:")开头,则new ActionForward;
// 否则 调用 mapping.findForward.
if ( (String.valueOf(forwardR) ).toUpperCase().startsWith(TD_NEW_FORWARD_PREFIX)){
forward = new ActionForward(String.valueOf(forwardR).substring(TD_NEW_FORWARD_PREFIX.length()) );
}else{
forward = mapping.findForward(String.valueOf(forwardR));
}
}
}
} catch (ClassCastException e) {
String message =
messages.getMessage("dispatch.return", mapping.getPath(), name);
log.error(message, e);
throw e;
} catch (IllegalAccessException e) {
String message =
messages.getMessage("dispatch.error", mapping.getPath(), name);
log.error(message, e);
throw e;
} catch (InvocationTargetException e) {
Throwable t = e.getTargetException();
if (t instanceof Exception) {
throw ((Exception) t);
} else {
String message =
messages.getMessage("dispatch.error", mapping.getPath(),
name);
log.error(message, e);
throw new ServletException(t);
}
}
return (forward);
}
这样做完后,我们的action就可以支持下面的任何一种类型的方法了: // 传统方式 public ActionForward testSubmit(ActionMapping mapping,ActionForm form, HttpServletRequest request,HttpServletResponse response) throws Exception; // 新方式 public String testSubmit1(TDServletWrapper servletWrapper) throws Exception; // 以下两者是则中方式. public String testSubmit2(ActionMapping mapping,ActionForm form, HttpServletRequest request,HttpServletResponse response) throws Exception; public ActionForward testSubmit3(TDServletWrapper servletWrapper) throws Exception; 下面简单说一下 TDServletWrapper . 这个类实际上就是封装了 ActionMapping mapping,ActionForm form, HttpServletRequest request,HttpServletResponse response, 同时提供了一些用来简化开发的方法,例如 public String getParameter(String key,String ifNull); public Integer getIntegerParameter(String key); public void bind(Object bean); 等等. 不再具体叙述,看一下代码就应该都明白了. TDServletWrapper 是一接口,他有两个实现: TDServletWrapperTestImpl 为测试专用的,是一个与web容器无关的实现. 他主要就是利用"模拟"request response的方式来处理 目前他的实现还不是很完全(例如没有模拟session等),大家可以根据自己的需求来补充. 大家也可以根据这个原理 不用TDServletWrapper ,而完全的使用Map等, 总之,通过dispatchMethod的改造,几乎可以实现你全部的合理的想法(别超出Struts1的能力极限就行),我在这里提供的只是一个思路而已. 经过上面的改造,一个可测试的action就诞生了:
public class MyClassicAction extends TDDispatchAction {
// 一个添加user的操作
public ActionForward addUser(TDServletWrapper servletWrapper) throws Exception {
UserInfoVO vo=new UserInfoVO();
servletWrapper.bind(vo);
// 校验,并执行相关DAO操作
if ( 添加成功 ) {
return "Succee";
}
return "Fail";
}
}
测试的可以这样写:
public static void testAddUser() throws Exception {
// 创建一个action对象
TestAction test=new TestAction();
//准备一些测试数据 放入map里,模仿request的Parameters
Map requestParameters=new HashMap();
requestParameters.put("userName", "asdqe");
requestParameters.put("password", "mghjghj");
requestParameters.put("userRole", "1");
requestParameters.put("email", "ut_email");
requestParameters.put("gender", "1");
requestParameters.put("age", "22");
// TDServletWrapperFactory.getTestInstance 方法用来取得一个脱离web容器的 TDServletWrapper对象 )
Assert.isTrue("Sucess".equals(test.testSubmit1(TDServletWrapperFactory.getTestInstance(requestParameters))) );
}
至此全部改造完毕. 这样的改造对于大家来说也许没什么意义,不过多少也是一种尝试. 由于Struts1.X(>=1.2)的 局限性,以及我个人能力的局限性,很多改造还不够彻底 不够合理, 对于前者我无能为力,对于后者,还恳请大家多多指正我的不足 谢谢了先. 上次发了一篇对 spring jdbctemplate的改造,这次又发了一篇对 struts 1 的改造, (其实我那个ecside 也是对老版extremecomponents的改造) 也许很多人会觉得我太落伍了,这些老掉牙的东西还改造个什么劲儿啊 呵呵 其实,很多时候,对于我来说,改造就是一个智力游戏,在一些现有的东西实现一些自己的新的想法,这样的感觉总是快乐的. 而且,我们的项目就是用的老技术 老东西,在不能换新东西的情况下,努力的挖掘现有东西的剩余价值,也许更具有实际意义. :D 声明：ITeye文章版权属于作者，受法律保护。没有作者书面许可不得转载。
推荐链接
|
| 返回顶楼 | |
|
发表时间：2007-06-25
弱弱问一句，这样做的目的是为了什么？仅仅为了单独进行单元测试？Struts是可以单元测试的啊 引入strutsTest包就可以，功能也都能满足需求啊
|
| 返回顶楼 | |
|
发表时间：2007-06-25
减少代码量 其实这个只是一部分
这个是根据我们框架的需求弄的 其实我们框架的dispatch里还做了好多事情呢 |
| 返回顶楼 | |
|
发表时间：2007-06-25
这样做对于新入门的人好处是容易上手，但是对于他的成长就不是很好，毕竟这样的参考书，就只有改写的人才有
|
| 返回顶楼 | |
|
发表时间：2007-06-25
恩 这也是在这样的公司的一个坏处
我们公司的框架就是 基于 struts spring的 但是使用的开发人员 几乎感觉不到他们的存在 自己写的代码里 很少能引入 struts spring的类 这样做有好处 也有坏处 |
| 返回顶楼 | |
|
发表时间：2007-06-25
其实大家也可以把这个看作一个 小实验
没必要去深究它的实际意义 因为也许他只对我们公司 我们部门有意义 这篇帖子写成这样,主要目的也是帮助新手明白了一下struts的dispatchaction的简单原理 呵呵 可以看作struts源码的一种解读 当然 仅限于初学者. :D |
| 返回顶楼 | |
|
发表时间：2007-08-22
呵呵.我这阵子在帮公司改造一个项目,struts层的设计跟你类似.呵呵.我一直就觉得action的参数太分散了,就应该当做一个对象来对待.
|
| 返回顶楼 | |
|
发表时间：2007-08-22
意义不如在Struts1中引入OGNL设值来的大。
|
| 返回顶楼 | |
|
发表时间：2007-09-04
粗看题目，很感兴趣，因为在下的框架就是struts1.2.4上修改的，不过本来就可以测试阿，能说明一下你的出发点不？
|
| 返回顶楼 | |
|
发表时间：2007-09-04
Fins为啥不直接用ww2或者struts2？
|
| 返回顶楼 | |
