难经3：Struts2，拦截器拦不住Result?

[问题]

使用Struts2作为web框架，知道它的拦截器（Interceptor）机制，类似与Filter和Spring的AOP，于是实现了一个为Action增加自定义前置(before)动作和后置动作(after)的拦截器（曰：WInterceptor），不过用一段时间发现，在WInterceptor的after中，对Action对象的属性修改在页面看不到，对请求对象的属性设置也无效。为什么在调用了Action之后(invokeAction())之后，request就不能使用了呢，拦截器不能改变Action的Result么?

 

[探幽]

 

在重看了Struts2的拦截器的官方文档以后，还是不明白上面的问题是为什么。地球人都知道，Struts2其实就是Webwork2，而拦截器的核心实现在XWork，利用XWork的拦截器框架，Struts2在外围通过线程上下文，绑定了Request和Response对象的包装类，哪问题到底在Struts2，还是在XWork？

 

在看到下面这张调用图，我才突然反应过来，“我真笨，真的，我只知道拦截器调用栈的最底层，是Action方法的调用，却不知道Result的调用也是在栈底调用，之后才返回给上一个拦截器，层层退出”：

        感谢这张图的作者，它简单，但有效。

 

 问题的关键在于，在调用actionInvocation.invoke()的之后，不仅执行类Action，也执行类Result。因而，等退回到拦截器的调用代码时，Result已经生成，View已经确定，这时你再修改模型（Action的属性）或请求对象的属性，对视图不会有任何影响。

 

另，为什么Result的执行不放到拦截器链的外面呢？这是我开始的直觉，有知道的朋友烦告知一声。

[解难]

 方法一：使用现成的PreResultListener监听器事件

搞清楚原因，卷起袖子干吧，只要让WInterpretor的after事件，放在Result的生成之前就行了。

看看XWork的拦截器接口注入的actionInvocation，其实就提供增加Result执行的前置监听事件-PreResultListener：

    /**
     * Register a {@link PreResultListener} to be notified after the Action is executed and
     * before the Result is executed.
     * <p/>
     * The ActionInvocation implementation must guarantee that listeners will be called in
     * the order in which they are registered.
     * <p/>
     * Listener registration and execution does not need to be thread-safe.
     *
     * @param listener the listener to add.
     */
    void addPreResultListener(PreResultListener listener);

 

因此，让拦截器实现这个接口，就可以自然实现Action执行after事件了。

 

 

方法二，实现自己的 ActionInvocation ，手动分离Action和Result的执行

本来前面的方法已经很好了，可是，可是啊，在addPreResultListener里的异常，不会被Struts的框架捕获，而且，addPreResultListener接口不能传递自己的上下文参数，难道动用ThreadLocal传参？

 

研究了一下XWork的ActionInvocation 接口默认实现类DefaultActionInvocation， 写了一个包装类，将Action的执行和Result的生成完全分开，或许有人用的着，放上来，见附件(ActionInvocationWrapper)，如有不妥之处请告知。

 

exeucteAction是执行Action，executeResult是执行Result

 

评论

14 楼 yanglphf 2014-01-17  

长知识了，谢谢

13 楼 buddie 2011-11-01  

为什么，执行Action和生成Result分开后，生成的页面中内容会出现两次。就是一个页面中，页面内容在上面显示完后，下面又重复显示了一次？

12 楼 buddie 2011-10-31  

非常感谢ixu的这篇文章，让我成功的解决我所遇到了问题，并让我有了想去看Struts2源码的想法（对于我这种菜鸟来说，过去从没想过！）。

11 楼 yudylaw 2009-04-15  

看了你的两篇 文章，收获不少。

10 楼 downpour 2009-02-16  

ixu 写道

在“[url=http://liuu.iteye.com/blog/333317]难经4：Struts2，拦截器拦不住的异常?![url]”中，我对这个问题做了更清晰详尽的分析和描述，也有了更好的解决办法。

我看了一下，暂时好像没看出漏洞来，应该是ActionInvocation的代码的逻辑问题。我还需要搭一个环境试试看这个问题。

不过实际上，这样的话，拦截器还是能够拦截到exception，ActionInvocation所控制的执行顺序出了问题。

可能是我从来不用struts2自带的Exception的拦截器，我基本上会自己写一个用于拦截自己的BusinessException，并做一些业务判断，所以也用不上<global-exception-mappings>的配置。

谢谢你的分析，让我学到很多东西。

9 楼 ixu 2009-02-16  

在“[url=http://liuu.iteye.com/blog/333317]难经4：Struts2，拦截器拦不住的异常?![url]”中，我对这个问题做了更清晰详尽的分析和描述，也有了更好的解决办法。

8 楼 downpour 2009-01-17  

如果我的回复还不能足以让你相信的话，请看一下struts2的reference：

thisWillRunFirstInterceptor
  thisWillRunNextInterceptor
    followedByThisInterceptor
      thisWillRunLastInterceptor
        MyAction1
        MyAction2 (chain)
        MyPreResultListener
        MyResult (result)
      thisWillRunLastInterceptor
    followedByThisInterceptor
  thisWillRunNextInterceptor
thisWillRunFirstInterceptor

如果你的thisWillRunFirstInterceptor写了一个try catch，你认为MyPreResultListener不会被捕获到？

7 楼 downpour 2009-01-17  

你的态度过激了，我不和你继续讨论下去了。

我所说的所有的东西全部都有代码的支持。我从Webwork开始一直到Struts2已经用了快4年了，从来没有听说过PreResultListener的异常不能被上层的拦截器拦截。在我自己编写的拦截器中，我也不止一次的抛出过RuntimeException，从来没有出现过异常无法被拦截。所以按照我的经验和实践，你的结论是错的。希望你仔细验证你的代码的问题。

6 楼 ixu 2009-01-16  

如果你想继续探讨这个问题，请：
1、真正写一个拦截器，实现PreResultListener
2、  A.在intercept方法中直接抛出异常（RuntimeException就行）
或者 B.在beforeResult方法中抛出异常
这两个可以用请求参数控制
3、写一个TestAction配上默认拦截器栈和这个拦截器，再配上一个全局的异常映射
4、在Servlet服务器(如Tomcat)下运行，访问TestAction

请观察2.A和2.B的响应结果的不同...

我说自己没有仔细去看源码，请正确理解“仔细”的含义，正因为看过源码，才奇怪在PreResultListener的异常应该是被上层拦截器捕捉的，但是最终却导致了Dispatcher抛出ServletException，对于这一点，我还没有“仔细”看，因而“不清楚原因”。这并不影响我说的现象和结论，在研究代码行为时，最好先把源码当黑箱，不要对源码想当然。

如果你宁愿一直看着源码，做一天的争论，也不愿拿十分钟来写一个测试验证你的猜疑，那么我可以严重怀疑你真的看清了、看懂了源码么？

还是那句话，实践是检验真理的唯一标准。

另，对于参数，确实有很多你想象不到的场景，你说的用ActionContext，其实不就是被包装了的线程上下文么？！

5 楼 downpour 2009-01-16  

所以说嘛，我最近正好在写Struts2的文章，至于Dispatcher，DefaultActionInvocation这些关键类更是读了好多遍，按照我的理解，是不可能发生在PreResultListener里抛出异常，而拦截器无法拦截到的情况的。因为DefaultActionInvocation中的invoke是递归调用，所以在上层的拦截器会包住整个执行过程。所以，既然你还没有研究过源码，不妨去看一下，我对你的这个结论是表示怀疑的。

至于第二个问题，我比较困惑的是，你究竟需要传递什么参数进去？

public void beforeResult(ActionInvocation invocation, String resultCode)

在这个接口中有ActionInvocation，而在ActionInvocation中，我们可以获取ActionContext，从而可以获取一切Action执行过程中的数据。

如果你需要传递额外的参数（恕我实在无法想到这种场景的存在性），你还可以通过ActionInvocation.getInvocationContext().put方法把变量放到ActionContext中进行传递。这个线程总是安全了的吧？

4 楼 ixu 2009-01-16  

downpour 同学，第一个问题已经说了，你完全可以自测试一下嘛，至于源码，你可以去看Struts2的Dispaatcher、XWork的ExceptionMappingInterceptor还有DefaultActionInvocation的代码，不用我在这里贴出来吧。至于为什么会这样，我也还没有仔细看源码，所有暂时还不是很清楚，你如果找出原因了，麻烦告知一下。

如果要用拦截器同时实现PreResultListener接口，第二个问题确实存在，因为拦截器一般都是单例，不能把在拦截过程中用到的数据用属性保存，否则线程不安全，所有不能直接通过属性字段传东西给实现PreResultListener接口的方法，而这个方法的参数又是固定的，因此或许只有用线程上下文了，不知我说明白没有。

3 楼 downpour 2009-01-16  

关于第一个问题，我还是深表怀疑，请用源码证明一下。

关于第二个问题，我认为你的理解也有问题，不用匿名类就不能传参数到PreResultListener的实现类了嘛？也请贴出代码。

2 楼 ixu 2009-01-16  

引用

1. addPreResultListener里的异常，不会被Struts的框架捕获

这个结论怎么得到的？

在使用全局异常映射配置<global-exception-mappings>时，如果Action执行有异常的话，会跳转到指定的异常结果页面。不过很遗憾，如果你在PreResultListener里抛出异常，Struts2的Dispatcher类会直接当成ServletException抛出给Servlet容器，去不到全局异常页面。你可以试一下。

引用

2. addPreResultListener接口不能传递自己的上下文参数，难道动用ThreadLocal传参

什么叫自己的上下文参数？不太明白

其实就是想传自己的参数给接口的实现代码，其实这个好解决，不用拦截器实现接口，直接用匿名内部类就好了，但是第一个问题不好办。

1 楼 downpour 2009-01-16  

引用

在addPreResultListener里的异常，不会被Struts的框架捕获，而且，addPreResultListener接口不能传递自己的上下文参数，难道动用ThreadLocal传参？

2点疑问。

1. addPreResultListener里的异常，不会被Struts的框架捕获

这个结论怎么得到的？

2. addPreResultListener接口不能传递自己的上下文参数，难道动用ThreadLocal传参

什么叫自己的上下文参数？不太明白