18

注意：我们重用单个连接。

************************************************
public Connection connection() {        
    try {
        if ((connection == null) || (connection.isClosed()))
        {
            if (connection!=null)
                log.severe("Connection was closed !");
            connection = DriverManager.getConnection(jdbcURL, username, password);
        }
    } catch (SQLException e) {
        log.severe("can't connect: " + e.getMessage());
    }
    return connection;        
}
**************************************************

public IngisObject[] select(String query, String idColumnName, String[] columns) {
    Connection con = connection();

    Vector<IngisObject> objects = new Vector<IngisObject>();
    try {
        Statement stmt = con.createStatement();

        String sql = query;
        ResultSet rs =stmt.executeQuery(sql);//oracle increases cursors count here
        while(rs.next()) {
            IngisObject o = new IngisObject("New Result");
            o.setIdColumnName(idColumnName);            
            o.setDatabase(this);
            for(String column: columns)
                o.attrs().put(column, rs.getObject(column));
            objects.add(o);
        }

        rs.close();// oracle don't decrease cursor count here, while it's expected
        stmt.close();
    } 
    catch (SQLException ex) {
        System.out.println(query);
        ex.printStackTrace();
    }
4

4 回答 4

26

init.ora 参数open_cursors定义了一个会话一次可以打开的最大游标数。它的默认值为 50。如果应用程序超过此数字，则会引发错误“ORA-01000：超出最大打开游标”。

因此，当不再需要 JDBC 资源时，必须关闭它们，尤其是 java.sql.ResultSet 和 java.sql.Statement。如果它们没有关闭，则应用程序存在资源泄漏。

在重用 Connection 对象的情况下，您必须注意，只要连接存在且事务尚未结束，打开的 oracle 游标就会保持打开和使用状态。当应用程序提交时，打开的游标被释放。

因此，作为应用程序设计人员，您需要知道对最复杂事务所需的打开游标的粗略估计。

困难在于 oracle 的内部参数视图（v$open_cursor、v$sesstat 等）无法显示可重用的已打开游标和仍被阻止（不可重用！）的已打开游标之间的区别一个未封闭的 ResulSet 或 Statement。如果您关闭 finally 块中的所有 Statement 和 ResultSet 对象，则您的应用程序非常好。

调整 init.ora 参数的工作方式如下（我们的应用程序最多需要 800 个游标）

ALTER SYSTEM SET open_cursors = 800 SCOPE=BOTH;
于 2010-04-06T07:33:00.883 回答
7

通常，您会将 ResultSet 和 Statement 的关闭语句放入一个finally块中，以确保即使发生异常也能调用它们（可能是您在此处遇到的问题）。在您当前的代码中，如果发生 SQLException，则两个 close() 方法调用将永远不会发生，并且游标将保持打开状态。

另外，您在 Oracle 中使用什么查询来查看打开游标的计数？

编辑：
该代码应该关闭光标。如果不是，那么您应该能够看到调用您的方法和游标计数增加 1 的 1 对 1 相关性。确保没有导致游标计数增加的意外过程。

如果您有权限，您可以对数据库运行此查询以通过 sid 查看打开的游标计数，以查看是否是其他进程增加了游标而不是您的特定进程。它会拉回任何打开超过 10 个光标的内容，您可以提高它以过滤掉噪音或通过用户名或 osuser 专门缩小噪音：

select oc.sid,
       count(*) numCur,
       s.username username,
       s.osuser osuser,
       oc.sql_text,
       s.program
  from v$open_cursor oc,
       v$session s
 where s.sid = oc.sid
group by oc.sid, 
         oc.sql_text, 
         s.username, 
         s.osuser, 
         s.program
having count(*) > 10
order by oc.sid;

另一个可能有用的查询，以防多个 sid 使用相同的查询字符串，因此上述内容不能很好地揭示罪犯：

 select oc.sql_text, count(*) 
   from v$open_cursor oc 
   group by oc.sql_text 
   having count(*) > 10 
   order by count(*) desc;
于 2010-04-01T12:52:07.010 回答
7

正确的做法是在它自己的 try/catch 块中关闭 finally 块中的每个资源。我通常使用这样的静态实用程序类：

public class DatabaseUtils
{
    public static void close(Connection connection)
    {
        try
        {
            if (connection != null)
            {
                connection.close();
            }
        }
        catch (SQLException e)
        {
            // log exception here.
        }
    }

    // similar methods for ResultSet and Statement
}

所以我会这样写你的代码：

public IngisObject[] select(String query, String idColumnName, String[] columns) {

Vector<IngisObject> objects = new Vector<IngisObject>();

Connection con = null;
Statement stmt = null;
ResultSet rs = null;

try 
{
    connection = connection();
    stmt = con.createStatement();

    // This is a SQL injection attack waiting to happen; I'd recommend PreparedStatemen
    String sql = query;
    rs =stmt.executeQuery(sql);//oracle increases cursors count here
    while(rs.next()) 
    {
       IngisObject o = new IngisObject("New Result");
       o.setIdColumnName(idColumnName);            
       o.setDatabase(this);
       for(String column: columns) o.attrs().put(column, rs.getObject(column));
       objects.add(o);
    }

} 
catch (SQLException ex) 
{
    System.out.println(query);
    ex.printStackTrace();
}
finally
{
    DatabaseUtils.close(rs);
    DatabaseUtils.close(stmt);
    DatabaseUtils.close(con);
}
于 2010-04-01T12:53:24.693 回答
6

我刚刚遇到了同样的问题，发现 - 如果你不关闭连接（因为你以后可能会重用它） - 你至少必须做一个connection.rollback()或connection.commit()来释放打开游标与关闭 ResultSet 和 Statements 一起。

于 2012-06-19T15:37:38.263 回答