java.sql.SQLException: You can't operate on a closed Connection!!!

我的JavaWeb项目中应用了Apache DbUtils 还有 C3p0连接池

项目结构如下

 

我的JDBCUtil如下

import java.sql.Connection;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
import javax.sql.DataSource;
import com.mchange.v2.c3p0.ComboPooledDataSource;


/**
 * 
 * JDBC 的工具类
 *
 */
public class JDBCUtil {
/**通过数据源获得连接对象
 * 多用在WEB项目中
 */
   private static ComboPooledDataSource dataSource = null;
	
	private static ThreadLocal tl = new ThreadLocal();
	
	static {
		dataSource = new ComboPooledDataSource();
	}
	
	public static DataSource getDataSource(){
		return dataSource;
	}
	
	public static Connection getConnection(){
		Connection conn = tl.get();//首先从ThreadLocal中获得连接对象
		try {
			if(conn == null){
				conn = dataSource.getConnection();
			}
		} catch (SQLException e) {
			e.printStackTrace();
		}
		tl.set(conn);
		return conn;
	}
	
    
	/**
	 * 开始事务
	 * @throws SQLException
	 */
	public static void startTransaction(){
		Connection conn = getConnection();
		try {
			conn.setAutoCommit(false);
		} catch (SQLException e) {
			e.printStackTrace();
		}
	}
	
	/**
	 * 回滚事务
	 * @throws SQLException
	 */
	public static void rollback(){
		Connection conn = getConnection();
		try {
			conn.rollback();
		} catch (SQLException e) {
			e.printStackTrace();
		}
		
	}
	
	/**
	 * 提交事务
	 * @throws SQLException
	 */
	public static void commit(){
		Connection conn = getConnection();
		try {
			conn.commit();
		} catch (SQLException e) {
			e.printStackTrace();
		}
	}
	
	/**
	 * 关闭Connection,并移除线程中的连接
	 * @throws SQLException
	 */
	public static void closeConnection(){
		try {
			getConnection().close();
			tl.remove();
		} catch (SQLException e) {
			e.printStackTrace();
		};
		
	}
	
	
	public static void close(Statement stm){
		try {
			if(stm != null){
				stm.close();
			}
		} catch (SQLException e) {
			e.printStackTrace();
		}
	}
	
	public static void close(ResultSet rs){
		try {
			if(rs != null){
				rs.close();
			}
		} catch (SQLException e) {
			e.printStackTrace();
		}
	}
	/**
	 * 测试
	 * @param args
	 */
//	public static void main(String[] args) {
//		System.out.println(getConnection());
//	}
}

在JDBCUtil中有关闭连接的方法，而且Apache Commons DbUtils中自带了关闭连接的方法如下。

public final class DbUtils
 {
  public static void close(Connection conn)
    throws SQLException
   {
     if (conn != null) {
       conn.close();
    }
  }

显然这两个关闭连接的方法是不同。

我在代码中用了  DbUtils.close(conn); 就报了这个错误，因为当时我并没有注意到这两个关闭连接方法的不同。

    /**
      * 修改员工部门，电话，地址，薪水
     * @throws SQLException 
      */
	@Override
	public boolean updateEmployee(Employee emp,History history) throws SQLException {
		
		Connection conn = JDBCUtil.getConnection();
		boolean result = true;
		try {
			//开启事务
			JDBCUtil.startTransaction();
			String sql1 = "update employee set deptno=?,empphone=?,empaddr=?,salary=? where empno=?";
			Object[] params1 ={emp.getDeptno(),
					          emp.getEmpphone(),
					          emp.getEmpaddr(),
					          emp.getSalary(),
					          emp.getEmpno()};
		   empdao.update(conn,sql1, params1);
		   //历史记录的添加
		   //changedate changereason 必有-history
		   //
		   
		   String sql2 = "insert into history(empno,deptno,salary,changedate,changereason) values(?,?,?,?,?)"; 
		   Object[] params2 = {emp.getEmpno(),
				              emp.getDeptno(),
				              emp.getSalary(),
				              history.getChangedate(),
				              history.getChangereason(),};
		   hisdao.update(conn, sql2, params2);
		   JDBCUtil.commit();
		} catch (Exception e) {
			e.printStackTrace();
			System.out.println("失败---------");
			JDBCUtil.rollback();
			result = false;
		}finally{
//			JDBCUtil.closeConnection();
			DbUtils.close(conn);
		}
		return result;
		
	}

因为应用了线程池，不仅要关闭连接，关闭连接后还需要在线程池中移除这个连接。