Spring MVC 性能提升

这几天在做公司内部SpringMVC的培训，培训之余看了看大家的代码，总感觉一个很不爽的地方，就是在每一个控制器的执行方法里，总要写这样一段代码：

代码1：

Map model = new HashMap();
model.put("yourKey", "yourObject");
ModelAndView mv = new ModelAndView("yourPage", model);
...
...
return mv;

每个请求就要new 一个Map和一个ModeAndView对象，这样如果请求多，就在不停的做new操作。new操作对JVM来说开销算是比较大的，所以尽量少建新对象。
在上面的例子中我们用ModelAndView的构造函数是public ModelAndView(String viewName, Map model)，打开该构造函数的源码：

代码2

public ModelAndView(String viewName, Map model) {
            this.view = viewName;
                        if (model != null) {
                                                getModelMap().addAllObjects(model);
                        }
}

//getModelMap()的代码：
public ModelMap getModelMap() {
                        if (this.model == null) {
                                                this.model = new ModelMap();
                            }
                        return this.model;
}

可以看出我们在java 代码1中创建的Map，在被提取值以后就没有什么用了。所以在Controller里面创建Map对象是冗余操作。
代码1应该如下写：

代码3

ModelAndView mv = new ModelAndView();
Map model = mv.getModelMap();
model.put("yourKey", "yourObject");
...
...
return vm;

这样一来就少创建了一个对象，节省了一点点开销。
可是还是每次都要创建ModelAndView新实例，如果一个网站的日访问量是10万（当然，访问理大的网站都会考虑用静态页面的方法），那么一天这个JVM就要创建10万个ModelAndView实例。这样的开销就很大了。

一般的解决办法就是让ModelAndView能够循环利用，要想循环利用ModelAndView就要有一个Pool，在我们需要可以通过Pool.get方法来取出ModelAndView实例，在不需要时在调用Pool.release(ModelAndView)把它放回。需要进取出总是很好办的，问题在什么时候可以调用release方法来释放。解决这个问题只能看SpringMVC的源码，看看有没有解决办法。
最笨方法的就是顺藤摸瓜，每一个请求到来都要先到DispatchServlet的doService。打开源码找到这个方法，发现也没有什么了不起的地方，可是发现一个醒目的片段：

try {
                            doDispatch(request, response);
}
finally

看来重要工作还在doDispatch中，那么就再找到doDispatch。
在doDispatch方法的前半部分是得到Controller返回的ModelAndView对象，后半部分是做显示处理，其中有这样一个显赫的代码段：

// Did the handler return a view to render?
if (mv != null && !mv.wasCleared()) {
    render(mv, processedRequest, response);
}

在被调的render方法里，就可以看出当render方法执行完后，ModelAndView对象已经不会再被使用，唯一能用到它的就是垃圾回收器了。（如果不信，你可以继续往下看， 以后工作都在View的子类里跑）
所以在render方法结束以后，再把ModelAndView对象放入缓存是正确的。我们可以继承DispatchServlet，然后把render方法重写，再用一个List作为简单的缓存池，实现如下：

代码4

public class CustomDispatchServlet extends DispatcherServlet {

    private static List<ModelAndView> modeAndViews = new ArrayList<ModelAndView>(20);

    static{
        for(int i = 0; i < 20; i++){  //缓存量为20
            modeAndViews.add(new ModelAndView());
        }
    }

    public void render(ModelAndView mv, HttpServletRequest request, HttpServletResponse response)
    throws Exception{
        super.render(mv, request, response);
        releaseModelAndView(mv);
    }

        //回收ModelAndView
    private void releaseModelAndView(ModelAndView mv){
        mv.clear();
        mv.getModelMap().clear();
        synchronized(modeAndViews){
            modeAndViews.add(mv);
        }
    }

        //取ModelAndView
    public static ModelAndView getModeAndView(){
        synchronized(modeAndViews){
            if(modeAndViews.size() > 0)
                return modeAndViews.remove(0);
            else
                return new ModelAndView();
        }
    }

}

当然你的web.xml的配置应改为：

代码5

<servlet>
    <servlet-name>test</servlet-name>
        <servlet-class>your.package.CustomDispatchServlet</servlet-class>
    <load-on-startup>1</load-on-startup>
</servlet>

然后你Controller里的执行代码也应改成如下：

代码6

ModelAndView mv = CustomDispatchServlet.getModeAndView();
mv.setViewName("yourPage");
mv.getModel().put("attribute", "yourObject");
...
       ...
return mv;

整个完成以后，测试起来屡试不爽。
整个下来，没有做太大的改到，也降低了开销。

还有一点要提醒一下，如果你用JSP做为页面，请不要用在ModelAndView的Map里放什么东西，这样Spring要做很多无用功，如果有东东全放入HttpServletRequest中，要想研究的话可以看看InternalResourceView的方法renderMergedOutputModel。

评论

23 楼 bottom 2008-08-29  

在现在的JVM里，这种 lightweight object 的 pooling是performance killer. 不能用！

22 楼 sunrie 2008-08-27  

就是嘛，楼主测试过new对象的速度吗？

21 楼 Joo 2008-08-27  

同意楼上，最好的优化就是不优化...这废话谁说的来着?
另外各位如何评判一个对象是heavy还是light? Connection相对于Map来说创建比较消耗资源体现在什么地方?

20 楼 javalover 2008-08-26  

这是想当然performance tuning的一个很好的例子。
performance tuning的第一条原则就是不要想当然，通过测试数据来说话。

19 楼 laiseeme 2008-08-08  

呵呵  我看lz整得也挺好 看了一圈了解了下springMVC内部的东西

不过这种优化确实没必要  就像上面人说的  同步开销要大得多

一般对象池用在大开销的重量级对象上
轻量级的给jvm处理就可以了，如果jvm连这点处理能力都没有
哪sun可以关门了

18 楼 dakulaliu 2008-08-07  

魔力猫咪 写道

恩。new并不一定是性能的问题所在。要是按你的说法Struts2这类每次请求都要新建Action的框架要慢死的。

严重同意！！！！！！！！！！！！

17 楼 dengyin2000 2008-06-05  

zgd 写道

在新版本的java里面，同步比gc慢
所以以前很多framework里面都有pool的设定，现在都取消了
现在只有一些数据库连接等IO资源做pool以外，基本的数据class已经都不作pool了

pool 一般用在创建很大消费的对象时才用的。 比如 connection， tapestry的page。 像new HashMap这样的 是没有必要的。

16 楼 xzs 2008-06-05  

zgd 写道

以前很多framework里面都有pool的设定，现在都取消了

很多？可否举些例子？

15 楼 zgd 2008-06-05  

在新版本的java里面，同步比gc慢
所以以前很多framework里面都有pool的设定，现在都取消了
现在只有一些数据库连接等IO资源做pool以外，基本的数据class已经都不作pool了

14 楼 惊鸿逝水 2008-06-05  

LZ,你应该更仔细才对，你会发现有很多很多对象没缓存，这下你会惊出一身的冷汗……
因为你发现你会变得很忙很忙

13 楼 slaser 2008-06-05  

建议lz不要用web框架，直接上servlet,效率最高。

12 楼 lzmhehe 2008-06-05  

downpour 写道

有时间花在这种无关痛痒的事情上面，还不如看看如何通过调整你Web服务器的配置参数来进行优化。

不懂，为什么很多人都无法找到问题的主要方面和次要方面。

在一个项目中没有调整 jvm 和tomcat 的参数
出现莫名其妙的事情，血的教训啊

11 楼 魔力猫咪 2008-06-05  

wangneng_001 写道

魔力猫咪 写道

恩。new并不一定是性能的问题所在。要是按你的说法Struts2这类每次请求都要新建Action的框架要慢死的。

不是每次请求都会创建哦，仅仅是第一次请求吧。

是每次请求都创建新的。Struts2的Action师从Webwork。Action里面可以写私有属性的。如果是单例，那么这些私有属性就会因为并发而混乱，所以必须每次请求一个新Action对象。放心，JVM会马上把它丢掉的。

10 楼 wangneng_001 2008-06-05  

魔力猫咪 写道

恩。new并不一定是性能的问题所在。要是按你的说法Struts2这类每次请求都要新建Action的框架要慢死的。

不是每次请求都会创建哦，仅仅是第一次请求吧。

9 楼 downpour 2008-06-05  

有时间花在这种无关痛痒的事情上面，还不如看看如何通过调整你Web服务器的配置参数来进行优化。

不懂，为什么很多人都无法找到问题的主要方面和次要方面。

8 楼 chaos_1981 2008-06-05  

数据库用连接池有必要，这个没什么必要吧

synchronized 在大并发的情况下可能会引起排队情况

7 楼 dennis_zane 2008-06-05  

这完全是想当然式的优化

6 楼 魔力猫咪 2008-06-05  

恩。new并不一定是性能的问题所在。要是按你的说法Struts2这类每次请求都要新建Action的框架要慢死的。

5 楼 fujohnwang 2008-06-05  

有点儿反其道而行之的感觉了

4 楼 linhong_1001 2008-06-05  

同意楼上的，同步做得不好的话开销更大，不知道本贴的楼主有没有做过真正的压力测试和性能测试，凭什么认为那样“重复利用MaV对象”就高效了，现代垃圾回收器可是比你想象的高效和稳定，我觉得不要凭感觉说话，还是做了测试之后再做性能改善，也学你的项目的性能瓶颈并不是想当然的在某个地方。