高手!这样连接数据库的javabean有没有错?

[yinliangzhi]

连接数据库的javabean如下:

package quanxian_db;
import java.sql.*;

public class ConnOracle {
String sDBDiver="sun.jdbc.odbc.JdbcOdbcDriver";
Connection connect=null;
//Statement stmt;
ResultSet rs=null;

public ConnOracle(){
try{
Class.forName(sDBDiver);
}
catch(java.lang.ClassNotFoundException e){
System.err.println(e.getMessage());
}
}

public ResultSet executeQuery(String sql){

try{
connect=DriverManager.getConnection("jdbc:odbc:XinHaoFenxi","system","pass");
Statement stmt=connect.createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE,ResultSet.CONCUR_READ_ONLY);
rs=stmt.executeQuery(sql);

}

catch(SQLException ex){
System.err.println(ex.getMessage());
}
return rs;
}

public int executeUpdate(String sql){
int result=0;
try{
connect=DriverManager.getConnection("jdbc:odbc:XinHaoFenxi","system","pass");
Statement stmt=connect.createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE,ResultSet.CONCUR_READ_ONLY);
result=stmt.executeUpdate(sql);
connect.commit();//新增加的语句
}

catch(SQLException ex){
// connect.rollback();//新增加的语句
System.err.println(ex.getMessage());
}
return result;
}
}


调用javabean的jsp如下:
<%@ page contentType="text/html;charset=gb2312" %>
<%@ page language="java" import="java.sql.*" %>

<jsp:useBean id="connDb" scope="page" class="quanxian_db.ConnOracle"/>

<%
String sql ="select * from qx";
connDb.executeQuery(sql);
……
%>

后来既没有关闭连接，又没有关闭数据集．请问在哪关闭连接？书上的程序也没有关闭．会不会报错？

如果有错，正确的写法应该是怎样的？

[MARS.nEIL]

当你觉得不会用到的时候,就可以把它关闭掉啊,不关不会报错.

[seanzhu2004]

在 return 语句之后关掉。
好象你的查询语句没有把返回值赋给一个ResultSet.

[hbuzhang]

你只要在你的bean中写上下面的函数
public void destroy() {
try {
stmt.close();
connect.close();
}
catch(Exception e) {
System.err.println("DBOperate.destroy():"+e.getMessage());
}
}
然后再jsp页面中写上：
connDb.destroy();
就可以了。这样就将数据库连接所占用的资源关闭掉了，你的程序没有关闭，虽然可能不会报错，但太浪费资源。

[angelheart]

这个是不用连接池的写法
我建议不在在构造函数里面加载驱动，写一个加载数据库驱动的方法，然后在executeQuery等操作里面判断connect，如果为NULL，则再去调用加载数据库驱动的方法

[yinliangzhi]

再来看看.欢迎高手发表意见.