不小心走入了cglib的误区

这2天在学习CGLIB，一时兴起用CGLIB实现了一个管理hibernate事务的类，用来在业务层处理事务。

首先来看一下代理类，代码如下

package com.test;

import java.lang.reflect.Method;

import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import org.hibernate.HibernateException;
import org.hibernate.Session;
import org.hibernate.Transaction;

import com.linbs.core.common.hibernate.HibernateUtil;

import net.sf.cglib.proxy.Enhancer;
import net.sf.cglib.proxy.MethodInterceptor;
import net.sf.cglib.proxy.MethodProxy;

/**
 * 使用CGLIB进行事务处理设置
 * @author linbs
 */
public class TransactionDAOProxy implements MethodInterceptor {

	private static Log logger = LogFactory.getLog(TransactionDAOProxy.class);

	private Enhancer enhancer=new Enhancer();

	public Object getDAO(Class clz){
        enhancer.setSuperclass(clz);
        enhancer.setCallback(this);
        return enhancer.create();
    }
	public Object intercept(Object obj, Method method, Object[] args,
			MethodProxy proxy) throws Throwable {
		Session session = null;
		Transaction tx = null;
		Object result = null;

		try{
			session = HibernateUtil.getSession();
			tx = session.beginTransaction();
			result = proxy.invokeSuper(obj, args);
			tx.commit();
			session.flush();
		}catch(HibernateException e){
			try {
				tx.rollback();
			} catch (HibernateException e1) {
				logger.error(e1.getMessage());
				throw e;
			}
		}finally{
			try {
				HibernateUtil.closeSession();
			} catch (HibernateException e) {
				logger.error(e.getMessage());
				throw new Throwable("关闭session时出错！");
			}
		}
		return result;
	}

}

在result = proxy.invokeSuper(obj, args);这个地方调用了未进行代理的dao类的方法；

假设有个BaseHibernateDAO类 里面有这2个方法：

/**
	 * 取得当期进程的session对象
	 * @return
	 */
	public Session getSession() {
		return HibernateUtil.getSession();
	}

	/**
	 * 保存一个新的实体类的实例，并返回标识号
	 * @param obj
	 * @return
	 */
	public Serializable save(final Object obj) {
		return this.getSession().save(obj);
	}

注意在save 方法中又调用了getSession()这个方法；

现在来写一个测试类；

package com.test;

import com.linbs.core.common.hibernate.BaseHibernateDAO;
import com.linbs.usermanage.model.User;

public class Test {
	public static void main(String[] args){
		TransactionDAOProxy proxy = new TransactionDAOProxy();
		BaseHibernateDAO baseDAO = (BaseHibernateDAO)proxy.getDAO(BaseHibernateDAO.class);
		User user = new User();
		user.setUserName("abcProxy");
		user.setUserPwd("abcProxy");
		user.setEmail("abc@126.cn");
		baseDAO.save(user);
	}
}

该测试类对BaseHibernateDAO的save方法进行了测试。
这个代码貌似没错，但是在运行的时候抛出了如下错误：
Exception in thread "main" org.hibernate.SessionException: Session is closed!
at org.hibernate.impl.AbstractSessionImpl.errorIfClosed(AbstractSessionImpl.java:49)
at org.hibernate.impl.SessionImpl.fireSave(SessionImpl.java:526)
at org.hibernate.impl.SessionImpl.save(SessionImpl.java:518)
at org.hibernate.impl.SessionImpl.save(SessionImpl.java:514)
at com.linbs.core.common.hibernate.BaseHibernateDAO.save(BaseHibernateDAO.java:31)
at com.linbs.core.common.hibernate.BaseHibernateDAO$$EnhancerByCGLIB$$c88d447e.CGLIB$save$3(<generated>)
at com.linbs.core.common.hibernate.BaseHibernateDAO$$EnhancerByCGLIB$$c88d447e$$FastClassByCGLIB$$e5947bbb.invoke(<generated>)
at net.sf.cglib.proxy.MethodProxy.invokeSuper(MethodProxy.java:167)
at com.test.TransactionDAOProxy.intercept(TransactionDAOProxy.java:49)
at com.linbs.core.common.hibernate.BaseHibernateDAO$$EnhancerByCGLIB$$c88d447e.save(<generated>)
at com.test.Test.main(Test.java:21)

为什么？ 经过测试，以及朋友的提醒，发现在BaseHibernateDAO的save()方法里调用了getSession()方法，而在运行的时候CGLIB会对getSession()和save()这2个方法都进行代理，因此在对save()进行tx.commit()和session.flush();的时候，在getSession()方法中已经进行了事务管理并关闭了session。

唉！一不小心就踏入了陷阱。