Безопасность потоков констант перечисления, используемых для подключения к различным базам данных

Question

Следует ли синхронизировать getConnection() внутри Enum?

Какая из них защищена потоком?

Первое:

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; 
}} 

или второй, синхронизированный вызов и изменчивой переменной:

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

Какой потокобезопасен?

Первый вариант не поточно-, потому что у вас есть чек-то одноактный ситуации, которая может привести к состоянию гонки. Второй вариант: thread safe, но использование volatile является избыточным, если вы синхронизируете весь метод getConnection.

Тем не менее, вы лучше использовать conneciton бассейн вместо Singleton DB соединение.