这不是检索连接的正确方法。您正在检索连接并将其分配为该类的实例变量(或更糟糕的是,静态变量)。基本上,您将使连接永远保持打开状态,并对所有查询重新使用单个连接。如果查询由不同的线程执行,则可能会导致灾难。另外,当它保持打开状态太长时间时,DB会回收它,因为它假定它已死/泄漏。
您应该在 尽可能短的范围内 获取 并关闭 连接。即与执行查询的位置相同。像这样: __
try
public Entity find(Long id) throws SQLException { Entity entity = null; try ( Connection connection = dataSource.getConnection(); // This should return a NEW connection! PreparedStatement statement = connection.prepareStatement(SQL_FIND); ) { statement.setLong(1, id); try (ResultSet resultSet = preparedStatement.executeQuery()) { if (resultSet.next()) { entity = new Entity( resultSet.getLong("id"), resultSet.getString("name"), resultSet.getInt("value") ); } } } return entity;}如果您担心连接性能并想重用连接,则应该使用连接池。您可以自己种一个,但是我强烈不建议这样做,因为您似乎还很陌生。只需使用现有的连接池,例如BoneCP,C3P0或DBCP即可。请注意,您应该
不会 如在上面的例子改变JDBC成语。您仍然需要在尽可能短的范围内获取并关闭连接。连接池本身将担心实际重用,测试和/或关闭连接。



