jdbc 一个类中方法的优化

[zhushoujun]

public Article queryOneArticleByArticleId(String articleId) {
Article article=null;

DBManager dbc=Factory.getDBManagerInstance();

Connection conn=null;
PreparedStatement pstmt=null;
ResultSet rs=null;
/****还得使readCount加1******/
String sql0="update article set readCount=readCount+1 where articleId=?";

String sql="select * from article where articleid=?";
try {
conn=dbc.getConn();
pstmt=conn.prepareStatement(sql0);
pstmt.setString(1, articleId);
pstmt.executeUpdate();
/******问题在这句*********************/
pstmt.close();//这里到底要不要关嘛

pstmt=conn.prepareStatement(sql);
pstmt.setString(1, articleId);
rs=pstmt.executeQuery();
while(rs.next()){

·····略





问题：我想问的是，当执行两句sql语句时候，执行完第一句sql语句后以后，要不要pstmt.close()这句呢，

不要会不会不安全？或者无所谓呢？（实验的时候要不要这句都正常运行的，但是我想选择一种好的方法）

[sl514]

关吧。虽然不关问题也不大。

[sl514]

好点的方法就是 使用数据库连接池。

[zhushoujun]

我知道可以用连接池（不过只会一点点哈），但是我想搞清这些细节呀，呵呵

[zhushoujun]

[Quote=引用 1 楼 sl514 的回复:]
关吧。虽然不关问题也不大。
[/Quote]

能不能解释一下不关会有什么问题，呵呵

[龙四]

一般是一下形式：
Connection conn = null;
PreparedStatement ps = null;
ResultSet rs = null;

//init

try {
//
} finally {
try {
if(rs != null) {
rs.close();
}catch (Exception e){
//ignore
} finally {
try {
if(ps != null) {
ps.close();
}
} catch (Exception e) {
//ignore
} finally {
if(conn != null) {
conn.close();
}
}
}
}
}

[sl514]

那你的网站是不是只有一个人用啊？
如果1000个人，会生成1000个session，都不关闭，会宕机的


使用数据库连接池会保持一部分连接，不管你连不连，连了话，连接池给你一个，不连了，还给他就可以了

[宅男小何]

在一个方法内，我建议先不关，这样可以可以提高效率，操作完了在方法后面的finally里面关掉，注意你的update最好放在事务里面！

[zhushoujun]

[Quote=引用 7 楼 lazy_p 的回复:]
在一个方法内，我建议先不关，这样可以可以提高效率，操作完了在方法后面的finally里面关掉，注意你的update最好放在事务里面！
[/Quote]

恩，
我是用了setAutoCommit（false）方法试过的，可惜mysql不支持，，还没有sqlserver试过，呵呵

[zhushoujun]

当然，finally 里面我会关掉所有的该关的东西的，只是想问一下要不要提前先关闭一下这里的pstmt，