java - 用于连接不同数据库的枚举常量的线程安全性

Question

Enum 中的 getConnection() 应该同步吗？

哪一个是线程安全的？

第一:

public enum DBConnection {

POSTGRESQL("jdbc:postgresql://localhost:5432/mydb", "vagrant", "vagrant"),
MYSQL("jdbc:mysql://localhost:3306/mydb", "vagrant", "vagrant");

DBConnection(String host, String user, String password) {

    this.host = host;
    this.user = user;
    this.password = password;
}

private final String host;
private final String user;
private final String password;

private Connection connection;

public Connection getConnection() throws SQLException {

    if (connection == null || connection.isClosed()) {
        connection = DriverManager.getConnection(host, user, password);
    }

    return connection;
}}

或者第二个，同步调用和 volatile 变量:

private volatile Connection connection;

public synchronized Connection getConnection() throws SQLException {

    if (connection == null || connection.isClosed()) {
        connection = DriverManager.getConnection(host, user, password);
    }

    return connection;
}

调用连接，例如:

      try (Connection connection = DBConnection.POSTGRESQL.getConnection()) {
       // Prepared statement etc
        }
    } catch (SQLException ignored) {}

Answer 1

Which one is thread safe?

第一个选项不是线程安全的，因为您遇到的情况可能会导致竞争条件。第二个选项是线程安全，但如果您要同步整个 getConnection 方法，则使用 volatile 是多余的。

也就是说，您最好使用连接池而不是单例数据库连接。