在多线程系统中使用静态 java.sql.Connection 实例是否安全?绝对不行!

我正在Tomcat上运行一个Web应用程序。我有一个处理所有数据库查询的类。此类包含返回查询结果的对象和方法。Connection

这是连接对象:

private static Connection conn = null;

它只有一个实例(单例)。

此外,我还有执行查询的方法,例如在数据库中搜索用户:

public static ResultSet searchUser(String user, String pass) throws SQLException

此方法使用静态对象。我的问题是,我在静态对象线程中的使用是安全的吗?或者当很多用户调用该方法时,它是否会导致问题?ConnectionConnectionsearchUser


答案 1

我在静态连接对象线程中的使用是否安全?

绝对不行!

这样,连接将在所有用户发送的所有请求之间共享,因此所有查询将相互干扰。但是线程安全不是你唯一的问题,资源泄漏也是你的另一个问题。在整个应用程序的生存期内,您将保持单个连接处于打开状态。平均数据库会在连接打开时间过长时(通常介于 30 分钟到 8 小时之间)回收连接,具体取决于数据库的配置。因此,如果您的Web应用程序的运行时间超过此时间,则连接将丢失,您将无法再执行查询。

当这些资源被保存为多次重用的类实例的非实例变量时,此问题也适用。static

您应该始终尽可能短的范围内获取并关闭连接、语句和结果集,最好是在根据以下 JDBC 习语执行查询的完全相同的资源试用内:

public User find(String username, String password) throws SQLException {
    User user = null;

    try (
        Connection connection = dataSource.getConnection();
        PreparedStatement statement = connection.prepareStatement("SELECT id, username, email FROM user WHERE username=? AND password=md5(?)");
    ) {
        statement.setString(1, username);
        statement.setString(2, password);

        try (ResultSet resultSet = statement.executeQuery()) {
            if (resultSet.next()) {
                user = new User();
                user.setId(resultSet.getLong("id"));
                user.setUsername(resultSet.getString("username"));
                user.setEmail(resultSet.getString("email"));
            }
        }
    }       

    return user;
}

请注意,您不应在此处返回 。您应该立即读取它并将其映射到非 JDBC 类,然后返回它,以便可以安全地关闭 。ResultSetResultSet

如果您还没有使用Java 7,那么请使用一个块,其中您手动关闭可关闭的资源,顺序与获取它们的顺序相反。您可以在这里找到一个示例:在 JDBC 中,连接、语句和结果集应该多久关闭一次?try-finally

如果担心连接性能,则应改用连接池。这内置于许多Java EE应用程序服务器中,甚至像Tomcat这样的准系统servletcontainers也支持它。只需在服务器本身中创建一个JNDI数据源,然后让您的Web应用程序将其抓取为.它透明地已经是一个连接池。您可以在下面列表的第一个链接中找到一个示例。DataSource

另请参阅:


答案 2

如果您只运行查询(听起来像只是选择数据),除了线程争用之外,不会有任何问题。SelectsearchUser

据我所知,a 一次只能处理一个查询,因此通过使用单个实例,您基本上将序列化数据库访问。但这并不一定意味着,在多线程环境中访问这样的数据库总是安全的。如果并发访问是交错的,则可能仍然存在问题。Connection