मैं नहीं होता इस दृष्टिकोण की सिफारिश करें। आप मूल रूप से एक कनेक्शन बना रहे हैं और उस पर लटक रहे हैं। मुझे आपके पैटर्न से प्यार नहीं है, लेकिन ऐसा कुछ बेहतर होगा:
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;
}
}
इसके अलावा, मैं किसी भी डेटा स्रोत पैरामीटर को हार्डकोड नहीं करूंगा, बल्कि गुण फ़ाइल से डेटासोर्स को प्रारंभ करूंगा।