慎用OGNL

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

慎用OGNL

博客分类： OGNL&MVEL

框架

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

Java代码

1. publicstaticObjectinvokeMethod(Objecttarget,Methodmethod,Object[]argsArray)throwsInvocationTargetException,IllegalAccessException
2. {
3. Objectresult;
4. booleanwasAccessible=true;
6. if(securityManager!=null){
7. try{
8. securityManager.checkPermission(getPermission(method));
9. }catch(SecurityExceptionex){
10. thrownewIllegalAccessException("Method["+method+"]cannotbeaccessed.");
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. returnresult;
25. }

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

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

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

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