Jag skulle inte rekommenderar detta tillvägagångssätt. Du skapar i princip en anslutning och hänger på den. Jag är inte kär i ditt mönster, men något sådant här skulle vara bättre:
public class DataTransaction {
private final static BasicDataSource dataSource;
static {
dataSource = new BasicDataSource();
dataSource.setDriverClassName("com.mysql.jdbc.Driver");
dataSource.setUrl("jdbc:mysql://127.0.0.1:3306/registrationtest");
dataSource.setUsername("root");
dataSource.setPassword("root");
dataSource.setMaxActive(100);
dataSource.setMaxWait(10000);
dataSource.setMaxIdle(10);
}
private DataTransaction() {
}
public static DataSource getDataSource() {
return dataSource;
}
}
Vidare skulle jag inte hårdkoda någon av DataSource-parametrarna, utan snarare initiera DataSource från en egenskapsfil.