java - MySql连接太多

Question

我讨厌提出一个在网络上被广泛询问的问题，但我似乎无法解决它。

我前段时间开始了一个项目，经过一个月的测试，我遇到了“连接太多”的错误。我调查了它，并通过增加 max_connections 来“解决”了它。这随后奏效了。

从那时起，越来越多的人开始使用它，它又火了起来。当我是站点上的唯一用户时，我键入“show processlist”，它会显示大约 50 个仍处于打开状态的连接(在命令中说“Sleep”)。现在，我不知道为什么这些是打开的，但在我的代码中我三重检查并且我打开的每个连接，我关闭。

即。

public int getSiteIdFromName(String name, String company)throws DataAccessException,java.sql.SQLException{

Connection conn = this.getSession().connection();
Statement smt = conn.createStatement();
ResultSet rs=null;
String query="SELECT id FROM site WHERE name='"+name+"' and company_id='"+company+"'";

rs=smt.executeQuery(query);
rs.next();

int id=rs.getInt("id");

rs.close();
smt.close();
conn.close();
return id;
}

每次我在网站上做其他事情时，都会打开而不是关闭另一负载的连接。我的代码有问题吗？如果不是，可能是什么问题？

Answer 1

使用您的方法，如果在 调用 conn.close() 之前抛出任何异常，连接将永远不会关闭。您需要在 try block 中获取它(以及语句和结果集)并在 finally block 中关闭它。无论是否抛出异常，finally 中的任何代码都将始终 执行。有了它，您可以确保昂贵的资源将被关闭。

这里是一个重写:

public int getSiteIdFromName(String name, String company) throws DataAccessException, java.sql.SQLException {
    Connection conn = null;
    Statement smt = null;
    ResultSet rs = null;
    int id = 0;
    try {
        conn = this.getSession().connection();
        smt = conn.createStatement();
        String query = "SELECT id FROM site WHERE name='" + name + "' and company_id='" + company + "'";
        rs = smt.executeQuery(query);
        rs.next();
        id = rs.getInt("id");
    } finally {
        if (rs != null) try { rs.close(); } catch (SQLException logOrIgnore) {}
        if (smt != null) try { smt.close(); } catch (SQLException logOrIgnore) {}
        if (conn != null) try { conn.close(); } catch (SQLException logOrIgnore) {}
    }
    return id;
}

也就是说，此代码对 SQL injection 敏感attacks .使用 PreparedStatement 而不是 Statement。

另见:

• Sun Exceptions tutorial: The Finally block
• Sun JDBC tutorial: introduction
• Sun JDBC tutorial: how to use PreparedStatement
• DAO tutorial: how to use basic JDBC code properly