在多线程系统中使用静态java.sql.Connection实例是否安全？

Question

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

这是连接对象:

private static Connection conn = null;

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

另外,我有执行查询的方法,例如在db中搜索用户:

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

此方法使用静态Connection对象.我的问题是,我在静态Connection对象线程中的使用是否安全？或者当很多用户调用该searchUser方法时会引起问题吗？

Answer 1

我在静态连接对象线程中使用安全吗？

绝对不!

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

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

您应始终在尽可能短的范围内获取和关闭连接,语句和结果集,最好在与执行查询的try-with-resources块完全相同的块中,如下所示: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;
}

请注意,您应该不返回ResultSet这里.您应该立即读取它并将其映射到非JDBC类,然后将其返回,以便ResultSet可以安全地关闭它.

如果您还没有使用Java 7,那么请使用一个try-finally块,在该块中,您可以按照相反的顺序手动关闭可关闭资源.您可以在此处找到一个示例:在JDBC中,Connection,Statement和ResultSet应该多久关闭一次？

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

也可以看看:

• 我应该如何在基于servlet的应用程序中连接到JDBC数据库/数据源？
• 当我的应用程序失去连接时,我应该如何恢复它？
• 我使用JDBC连接池吗？
• 使用MVC和DAO模式在JSP页面中以HTML格式显示JDBC ResultSet
• 使用JDBC的DAO教程