Возможный дубликат:
когда нужно закрыть Connection, Statement, PreparedStatement и ResultSet в JDBC
Я написал простую оболочку для подключения JDBC, и она работает, но я хочу улучшить ее с помощью лучших практик, насколько это возможно. В основном он имеет такие методы, как open()
, close()
, isOpened()
, select()
, insert()
, update()
, delete()
и batch()
. Для простоты я отправлю здесь первые 4 метода.
public class Query{
private Connection con;
private PreparedStatement ps;
private ResultSet rs;
//Database.open() returns a Connection ready to use
public void open (Database database) throws DatabaseException, SQLException{
if (!isOpened ()){
con = database.open ();
}
}
public void close () throws SQLException{
if (isOpened ()){
if (ps != null) ps.close ();
con.close ();
con = null;
}
}
public boolean isOpened (){
return con != null;
}
//The query string is the query without the word "select" and can use placeholders (?)
//The args param it just an array owith the values of this placeholders
public ResultSet select (String query, Object[] args) throws SQLException{
if (ps != null) ps.close ();
if (isOpened ()){
ps = con.prepareStatement ("select " + query);
if (args != null){
for (int i=0; i<args.length; i++){
ps.setObject (i+1, args[i]);
}
}
rs = ps.executeQuery ();
}
return rs;
}
}
Примечания:
- Тот же объект запроса может быть повторно использован, например, открытие и закрытие он, и после открытия снова.
- Я не закрываю соединение для каждого запроса, я просто закрываю подготовленное выражение (это правильно или я могу оставить подготовленный оператор открыт, потому что объект Connection закроет его?)
- Когда я закрываю
Connection
, всеPreparedStatement
и ихResultSet
также замкнуты, правильно?
Использование:
Database database;
//Database initialization
Query query = new Query ();
query.open (database);
ResultSet rs = query.select ("* from user where name=?", new String[]{ "MyName" });
doSomethingWithResult1 (rs);
//Connection is not closed here
ResultSet rs = query.select ("coordx from point where coordy=? and coordz=?", new Float[]{ 0.1, 0.2 });
doSomethingWithResult2 (rs);
query.close ();
query.open (database);
ResultSet rs = query.select ("* from user where name=?", new String[]{ "MyName" });
doSomethingWithResult1 (rs);
//Connection is not closed here
ResultSet rs = query.select ("coordx from point where coordy=? and coordz=?", new Float[]{ 0.1, 0.2 });
doSomethingWithResult2 (rs);
query.close ();
Как вы думаете? Должен ли я закрывать и открывать соединение после каждого запроса? Могу ли я оставить открытое PreparedStatement после каждого запроса в том же соединении? Это хороший дизайн?