慎用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。

你可能感兴趣的:(慎用OGNL)