慎用OGNL

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

慎用OGNL

• 博客分类：
• OGNL&MVEL

框架

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

 

Java代码  

1. public static Object invokeMethod( Object target, Method method, Object[] argsArray ) throws InvocationTargetException, IllegalAccessException  
2.     {  
3.         Object      result;  
4.         boolean     wasAccessible = true;  
5.   
6.         if (securityManager != null) {  
7.             try {  
8.                 securityManager.checkPermission(getPermission(method));  
9.             } catch (SecurityException ex) {  
10.                 throw new IllegalAccessException("Method [" + method + "] cannot be accessed.");  
11.             }  
12.         }  
13.         synchronized (method) {  
14.             if (!Modifier.isPublic(method.getModifiers()) || !Modifier.isPublic(method.getDeclaringClass().getModifiers())) {  
15.                 if (!(wasAccessible = ((AccessibleObject)method).isAccessible())) {  
16.                     ((AccessibleObject)method).setAccessible(true);  
17.                 }  
18.             }  
19.             result = method.invoke( target, argsArray );  
20.             if (!wasAccessible) {  
21.                 ((AccessibleObject)method).setAccessible(false);  
22.             }  
23.         }  
24.         return result;  
25.     }  

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

 

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

 

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

 

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