慎用OGNL

原文链接：http://jansel.iteye.com/blog/538796

慎用OGNL

博客分类：
• OGNL&MVEL

框架

在最近的项目中使用了Xwork作为基础框架，发现了致命的性能问题。 大家都知道Xwork基于OGNL来的，在压力测试的过程中，发现大量线程堵塞在OGNLRuntime这个类中，请看代码：

 

public static Object invokeMethod( Object target, Method method, Object[] argsArray ) throws InvocationTargetException, IllegalAccessException
    {
        Object      result;
        boolean     wasAccessible = true;

        if (securityManager != null) {
            try {
                securityManager.checkPermission(getPermission(method));
            } catch (SecurityException ex) {
                throw new IllegalAccessException("Method [" + method + "] cannot be accessed.");
            }
        }
        synchronized (method) {
            if (!Modifier.isPublic(method.getModifiers()) || !Modifier.isPublic(method.getDeclaringClass().getModifiers())) {
                if (!(wasAccessible = ((AccessibleObject)method).isAccessible())) {
                    ((AccessibleObject)method).setAccessible(true);
                }
            }
            result = method.invoke( target, argsArray );
            if (!wasAccessible) {
                ((AccessibleObject)method).setAccessible(false);
            }
        }
        return result;
    }

 OGNL把Method对象加锁，这个开销可不小啊。这个有点想不通OGNL为啥要把Method加锁。只有当该Method是非线程安全才有必要加锁啊，但是OGNL应该理会这些吗？安全与否应该是Method考虑的，而不是OGNL认为的。

 

还有一个地方：public static final Field getField(Class inClass, String name)

 

这个方法在Xwork中调用的频率很高，在CompoundRootAccessor中调用频率很高，相关代码大家可以去看一下。

 

由此可以推出，Struts2的性能也堪忧啊，尤其是用了ParametersInterceptor。