JDBC连接太多 - 连接已正确关闭

时间:2014-12-24 21:24:38

标签: java mysql jdbc connection

我讨厌提出一个在网络上广泛提出的问题,但我似乎无法解决它。 我正在关闭所有的连接,但程序正在抛出“太多连接异常”

这是我的Connectionhelper

 public MySQLConnection() {
    try {

      // Datenbanktreiber für ODBC Schnittstellen laden.
      // Für verschiedene ODBC-Datenbanken muss dieser Treiber
      // nur einmal geladen werden.
      Class.forName("com.mysql.jdbc.Driver");

      // Verbindung zur ODBC-Datenbank 'sakila' herstellen.
      // Es wird die JDBC-ODBC-Brücke verwendet.
      conn = DriverManager.getConnection("jdbc:mysql://" + dbHost + ":"
          + dbPort + "/" + database + "?" + "user=" + dbUser + "&"
          + "password=" + dbPassword);
    } catch (ClassNotFoundException e) {
      System.out.println("Treiber nicht gefunden");
    } catch (SQLException e) {
      System.out.println("Connect nicht moeglich"+e);
    }
  }

  public Connection getInstance()
  {

    if(conn == null)
      new MySQLConnection();
    return conn;
  }


  public final void close_multiple( final Connection con, final ArrayList<PreparedStatement> listPreparedstatments, final ArrayList<ResultSet> listResultsets )
  {
      try
      {
          if ( con != null && con.isClosed())
          {
              con.close();

          }
          for(PreparedStatement pstm : listPreparedstatments)
              if ( pstm != null && pstm.isClosed())
              {
                  pstm.close();
              }
          for(ResultSet rs : listResultsets){
              if ( rs != null && rs.isClosed())
              {
                  rs.close();
              }
          }

      }
      catch ( SQLException e )
      {
          e.printStackTrace();
      }
  }

  public final void close_single( final Connection con, final PreparedStatement stmt, final ResultSet rs )
  {
      try
      {
          if( con != null && con.isClosed())
          {
              con.close();
          }
          if ( stmt != null && stmt.isClosed())
          {
              stmt.close();
          }
          if ( rs != null && rs.isClosed())
          {

            rs.close();
          }

      }
      catch ( SQLException e )
      {
          e.printStackTrace();
      }
  }

以下是一些示例代码

 public ArrayList<Product_Type> getProduct_Types() throws JSONException {
      conn = mysqlconnection.getInstance();
      Product_Type product_type;
      ArrayList<Product_Type> listProduct_Types = new ArrayList<Product_Type>();
      if(conn != null){
          //MYSQL Resultsets and Statements
          ResultSet rst_product_type=null;
          PreparedStatement pstmt_product_type=null;
          try{

            String sql = "SELECT * FROM `product_type`";
            pstmt_product_type = conn.prepareStatement(sql);
            rst_product_type = pstmt_product_type.executeQuery();

            while(rst_product_type.next()) { 
                product_type = new Product_Type();
                product_type.setProduct_Type_ID(rst_product_type.getInt("Product_Type_ID"));
                product_type.setProduct_Type_Name(rst_product_type.getString("Product_Type_Name")); 
                listProduct_Types.add(product_type);
            }
         } catch(Exception e){
            System.out.println("Unable to connect to product_type Table.");
         } finally {
             mysqlconnection.close_single(conn, pstmt_product_type, rst_product_type);
         }
      }
      return listProduct_Types; 
  }

  public ArrayList<Product_Type> getProduct_Type_by_Manufacturer_ID(int manufacturer_id) throws JSONException, SQLException {
        conn = mysqlconnection.getInstance();
        Product_Type product_type;
        ArrayList<Product_Type> listProduct_Types = new ArrayList<Product_Type>();
        ArrayList<ResultSet> listResultsets = new ArrayList<ResultSet>();
        ArrayList<PreparedStatement> listPreparedstatments = new ArrayList<PreparedStatement>();

         if(conn != null){
             PreparedStatement pstmt_product_detail = null;
             PreparedStatement pstmt_product_type = null;

             ResultSet rst_product_type = null;
             ResultSet rst_product_detail = null;
             try{                       
                    String sql_product_detail = "SELECT DISTINCT(Product_Type_ID) FROM `product_details` WHERE Manufacturer_ID=?"; 
                    pstmt_product_detail = conn.prepareStatement(sql_product_detail);
                    pstmt_product_detail.setInt(1, manufacturer_id);
                    rst_product_detail = pstmt_product_detail.executeQuery();

                    while(rst_product_detail.next()) { 
                         String sql_product_type = "SELECT Product_Type_ID, Product_Type_Name FROM `product_type` WHERE Product_Type_ID=?";
                         pstmt_product_type = conn.prepareStatement(sql_product_type);
                         pstmt_product_type.setInt(1, rst_product_detail.getInt("Product_Type_ID"));
                         rst_product_type = pstmt_product_type.executeQuery();

                         while(rst_product_type.next()) { 
                             product_type = new Product_Type();
                             product_type.setProduct_Type_ID(rst_product_type.getInt("Product_Type_ID"));
                             product_type.setProduct_Type_Name(rst_product_type.getString("Product_Type_Name"));    
                             listProduct_Types.add(product_type);
                        }
                     }
                    listPreparedstatments.add(pstmt_product_type);
                    listPreparedstatments.add(pstmt_product_detail);
                    listResultsets.add(rst_product_detail);
                    listResultsets.add(rst_product_type);
             } catch(Exception e){
                System.out.println("Unable to connect to pr Table.");
             } finally {
                 //Close MYSQL-Connection
                 mysqlconnection.close_multiple(conn, listPreparedstatments, listResultsets);
             }
         }

         return listProduct_Types;
  }

我不知道问题出在哪里?我还尝试增加mysql max连接,但它仍然无法正常工作

2 个答案:

答案 0 :(得分:4)

您实际上根本没有关闭连接(以及ResultSet和PreparedStatements),只有在连接已经关闭时才会调用此代码中的con.close()

if (con != null && con.isClosed()) {
    con.close();
}

应该是!con.isClosed()

这似乎是主要问题。但是,John Skeet在问题评论中提到的代码中还存在其他问题。

答案 1 :(得分:2)

从Java7开始,您可以利用try-with-resource来处理自动关闭资源(资源是一个在程序完成后必须关闭的对象)。此外,您的代码将被简化(通过删除您只是检查和关闭连接,语句,结果集的行),易于阅读和维护例如。

try (Connection conn = datasource.getConnection();
    Statement stmt = conn.createStatement()) {// you can use all other autocloseable objects in try clause

 //your code here
 }