需要有关修复JAVA查询语句的建议吗?

时间:2011-11-25 07:54:59

标签: java oracle rft

我的JAVA脚本包含2个JAVA类:RMSqueryRMSRMS类中,我在queryRMS

中调用该方法

RMS Java Class(我省略了开始执行部分,下面只是方法)

    for (int i = 1; i <= itemCount; i++) {
        GlobalVariables.numberRow = i;
        JavaDatapool.settings();

        String item = queryRPM.connectDB_Multi(configFile,"SELECT ITEM FROM ORDSKU WHERE ORDER_NO  = '" + orderNo + "' ORDER BY ITEM ASC",i);
        JavaDatapool.writeXLS("item",item,GlobalVariables.sheetXLS);
        sleep(1);

    }

queryRMS JAVA类

public static String connectDB_Multi(String configFile, String query, int i) throws FileNotFoundException, IOException, SQLException, ClassNotFoundException{
    Properties p = new Properties();
    p.load(new FileInputStream(configFile));

    String serverName = (p.getProperty("RMS_DBServerName"));
    String portNumber = (p.getProperty("RMS_PortNumber"));
    String sid = (p.getProperty("RMS_SID"));
    String url = "jdbc:oracle:thin:@//" + serverName + ":" + portNumber + "/" + sid;
    String username = (p.getProperty("RMS_Username"));
    String password = (p.getProperty("RMS_Password"));
    //  jdbc:oracle:thin:@//localhost:1521/orcl

    Class.forName("oracle.jdbc.driver.OracleDriver");
    Connection connection = DriverManager.getConnection(url,username,password);     
    String setr = null;
    try {      
        Statement stmt = connection.createStatement();

        try {ResultSet rset = stmt.executeQuery(query);
            try {
                while(rset.absolute(i))   
                    setr = rset.getString(1);
                    return setr;  
            }        
            finally {
                try { rset.close(); 
                } 
                catch (Exception ignore) {}

            }
        } 
        finally {
            try { stmt.close(); 
            } 
            catch (Exception ignore) {}
        }
    } 
    finally {
        try { connection.close(); 
        } 
        catch (Exception ignore) {}

    }
}

所以它的作用是调用connectDB_multi类,然后返回String,下一部分将其保存在Excel工作表中。

循环应该一次返回所有行,然后将其保存在Excel工作表中。

在第二次循环中,查询出现故障,尽管查询应返回包含2行的1列。

原件包含while(rset.next())部件而不是while(rset.absolute(i)) 但接下来每次只返回第一行。所以当从数据库中只检索到一列和一行时,脚本才有效。

3 个答案:

答案 0 :(得分:5)

你的逻辑看起来有些混乱。

查看您发布的第一个循环。你有效地执行:

SELECT ITEM FROM ORDSKU WHERE ORDER_NO = '" + orderNo + "' ORDER BY ITEM ASC

itemCount次。每次执行它时,您都试图访问 n :第n行,n是循环计数器。你觉得那里有问题吗?您如何知道查询将返回 itemCount 行数?因为如果没有,它将失败,因为您试图访问不存在的行。

我怀疑你想做的事情是这样的

Statement stmt = connection.createStatement();
ResultSet rset = stmt.executeQuery(query);
while(rset.next()) {
    JavaDatapool.writeXLS("item",rset.getString(1),GlobalVariables.sheetXLS);
}

您还应该认真考虑使用某种形式的连接池,以避免不得不一直重新打开新连接,因为这是非常耗时的操作。

答案 1 :(得分:1)

这段代码看起来非常低效,对于你想要从数据库中获取的每一行,你读取一个属性文件,创建一个连接,选择所有匹配的行,跳到你想要的行并返回该行。 (或者至少我认为这就是你要做的事情)。

您的代码

while(rset.absolute(i))   
    setr = rset.getString(1);

可能是一个无限循环,因为它可以继续转到同一行,只要它可以转到该行,因此该行不存在(并且存在时)或该行确实存在(并且永远地继续)。

您可能应该重新构建程序,只进行一次选择并读取所需的所有行并将它们存储在Excel文件中。执行此操作时,您可以进行调试以查看是否确实获得了所需的数据。

除了为每行创建新连接和查询一次效率低下的代码之外,你怎么知道你想要多少行?

我想最终你想要这样的东西

....
while(rset.next()) {
    JavaDatapool.writeXLS("item",item,GlobalVariables.sheetXLS);
}

什么是睡眠(1)支持完成?

答案 2 :(得分:0)

仅供参考:如果您经常打开和关闭语句作为逻辑或pap的解决方案,则可以获得“java.sql.SQLException:ORA-01000:超出最大打开游标数”错误消息。

我建议你不要'太多概括'。我看到很多OOP程序员都过度泛化,这很痛苦。你应该按目标设计,目标不应该是“只是对齐”,也不应该是“代码看起来很漂亮”,它必须有设计目的。