Question

我在java应用程序中使用单例数据库连接，这是我的连接管理器类的代码：

public abstract class DatabaseManager {
    //Static instance of connection, only one will ever exist
        private static Connection connection = null;    
        private static String dbName="SNfinal";
        //Returns single instance of connection
        public static Connection getConnection(){       
            //If instance has not been created yet, create it
            if(DatabaseManager.connection == null){
                initConnection();
            }
            return DatabaseManager.connection;
        }   
        //Gets JDBC connection instance
        private static void initConnection(){           
            try{        
                Class.forName("com.microsoft.sqlserver.jdbc.SQLServerDriver");
                   String connectionUrl = "jdbc:sqlserver://localhost:1433;" +
                      "databaseName="+dbName+";integratedSecurity=true";

                DatabaseManager.connection =
                             DriverManager.getConnection(connectionUrl);        
            }
            catch (ClassNotFoundException e){       
                System.out.println(e.getMessage());
                System.exit(0);
            }
            catch (SQLException e){         
                System.out.println(e.getMessage());
                System.exit(0);
            }
            catch (Exception e){        
            }       
        }
    public static ResultSet executeQuery(String SQL, String dbName)
    {
        ResultSet rset = null ;
        try {
               Statement st = DatabaseManager.getConnection().createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE,ResultSet.CONCUR_READ_ONLY);
               rset = st.executeQuery(SQL);
               //st.close();
        }
        catch (SQLException e) {
            System.out.println(e.getMessage());
            System.exit(0);
        }
        return rset;
     }

    public static void executeUpdate(String SQL, String dbName)
    {
        try {
               Statement st = DatabaseManager.getConnection().createStatement();
               st.executeUpdate(SQL);
               st.close();
        }

        catch (SQLException e) {
            System.out.println(e.getMessage());
            System.exit(0);
        }
     }
}

问题是我的代码在开始时工作完美，但是当时间过去变得非常慢。是什么导致了这个问题，我该如何解决？在启动时，我的应用程序每秒处理大约20个查询，运行1小时后达到每秒10个查询，运行3天后每10秒达到1个查询！ P.S：我的应用程序是一个单用户应用程序，通过数据库进行许多查询。 P.S：这是我在eclipse.ini中的JVM参数：

--launcher.XXMaxPermSize
512M
-showsplash
org.eclipse.platform
--launcher.XXMaxPermSize
512m
--launcher.defaultAction
openFile
--launcher.appendVmargs
-vmargs
-Dosgi.requiredJavaVersion=1.6
-Xms500m
-Xmx4G
-XX:MaxHeapSize=4500m

不幸的是数据库是远程的，我没有任何监控访问权限来查找那里发生的事情。

以下是我的使用示例：

String count="select count(*) as counter from TSN";
ResultSet rscount=DatabaseManager.executeQuery(count, "SNfinal");
if(rscount.next()) {
    numberofNodes=rscount.getInt("counter");
}

Answer 1

是什么导致了这个问题，我该如何解决？

您遇到的主要问题是executeQuery()方法。您没有关闭Statement，我认为您已对st.close()行进行了评论，因为您需要ResultSet开放进一步处理。我可以看到你的想法是避免在应用程序中看到重复的JDBC代码，但这不是正确的方法。

规则是：关闭ResultSet，然后关闭Statement，否则你没有正确地释放资源，而是暴露于你所描述的那种问题。

Here您可以找到关于如何正确关闭资源的良好解释（请记住，在您的情况下，您不需要关闭连接）

修改一个例子可能是

try{ Statement st = DatabaseManager.getConnection().createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE,ResultSet.CONCUR_READ_ONLY); ResultSet rsCount = st.executeQuery(count); //count="select count(*) as counter from TSN"; if(rsCount.next()) { numberofNodes=rscount.getInt("counter"); } } catch (SQLException e) { //log exception } finally { rsCount.close(); st.close(); }

Answer 2

虽然Connection Manager会自动关闭Statement和Resultset，但如果您立即关闭它们会更好。
你的代码中没有别的东西会影响你的单线程任务，所以我敢打赌你的数据库肯定有问题。尝试找出是否有任何数据库锁定或错误的列索引。并且还要看看数据库查询状态，找出瓶颈所在。

Answer 3

您应该考虑使用类似CachedRowSet http://docs.oracle.com/javase/1.5.0/docs/api/javax/sql/rowset/CachedRowSet.html

的断开连接的结果集

public static ResultSet executeQuery(String SQL, String dbName)
{
    CachedRowSetImpl crs = new CachedRowSetImpl();
    ResultSet rset = null ;
    Statement st = null;
    try {
           st = DatabaseManager.getConnection().createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE,ResultSet.CONCUR_READ_ONLY);
           rset = st.executeQuery(SQL);
           crs.populate(rset);
    }
    catch (SQLException e) {
        System.out.println(e.getMessage());
        System.exit(0);
    }finally{
        rset.close();
        st.close();
    }
    return crs;
 }

CachedRowSet实现ResultSet，因此它应该像ResultSet一样。

http://www.onjava.com/pub/a/onjava/2004/06/23/cachedrowset.html

除了这些更改之外，我还建议您使用池数据源来获取连接并关闭它们，而不是保持一个打开的连接。

http://brettwooldridge.github.io/HikariCP/

或者如果你不是java7，bonecp或c3po。

编辑：

要回答您的问题，这可以解决您的问题，因为CachedRowSetImpl在使用时不会保持与数据库的连接。这样，您就可以在填充Resultset后关闭Statement和CachedRowSetImpl。

希望能回答你的问题。

传统的DB单例连接效果很差

3 个答案: