Java返回字符串对象

Question

我有一个数据库函数，该函数检索数据，然后填充一个jtable。 我需要更改此函数，以便它从Object[][]的数据库中返回数据。 如何才能做到这一点？ （我不确定如何在每次行迭代中存储数据-以下代码中的while循环部分）。

public void data() {
    // clear table then load information 
    DefaultTableModel model=(DefaultTableModel)table.getModel();  
         model.getDataVector().removeAllElements();
  table.repaint(); 
  ResultSet rs=null;
  Statement st=null;

    try { 
        Class.forName("java.sql.Driver"); 
        _con = DriverManager.getConnection(_url,_user,_pwd);
        st = _con.createStatement(); 
        String query = "SELECT * FROM table";

        rs = st.executeQuery(query); 

        while (rs.next()) { 
            String d1 = rs.getString("record1"); 
            String d2 = rs.getString("record2");
            model.addRow(new Object[]{d1,d2}); 
        }        
    } catch (Exception e) { 
    } finally {
        try {
            if (rs != null) {
                rs.close();
            }
            if (st != null) {
                st.close();
            }
            if (_con != null) {
                _con.close();
            }
        } catch (SQLException ex) {
        }
    }
}

Answer 1

我认为此方法不是一个好主意，原因有几个：

1. 一种方法应该做的很好。 您已经用这种方法将UI，连接获取和查询混合在一起。 我将开始分解它们。 持久层不应该知道或关心您正在使用Swing。
2. 在finally块的单独try / catch块中关闭这些资源。 如果有人抛出异常，您仍然希望其他人关闭。
3. 空的捕获块是一个可怕的想法。 打印或记录堆栈跟踪。
4. 在此方法之外创建Connection并将其传入。
5. ResultSet和Statement不应是该类的成员变量。 使它们在方法本地。

只需更改方法以返回所需内容即可，而不是void 。 我不会使用Object [][] ； 我更喜欢地图列表或封装行的其他类型。 使用列名称作为Map中的键。

Answer 2

这是OP要求的一些代码，是的，OP应该注意duffymo的建议-我完全同意。

  Object[] result = null;

  try{ 

   Class.forName("java.sql.Driver"); 
   _con = DriverManager.getConnection(_url,_user,_pwd);
   st=_con.createStatement(); 

   String query="SELECT COUNT(*) cnt FROM table";
   rs=st.executeQuery(query); 
   Integer size = null;
   if (rs.next()){
       size = rs.getInteger("cnt");
   }

   if (size != null){
     Object[] result = new Object[size];
     query="SELECT * FROM table";    
     rs=st.executeQuery(query); 
     int i = 0;
     while(rs.next())
     { 
         String d1=rs.getString("record1"); 
         String d2=rs.getString("record2");
         result[i++] = new Object[]{d1,d2}; 
     }        
   }
   } 
   catch(Exception e)  
   { 
      //TODO handle exceptions, e.g. rethrow 
      //throw new RuntimeException(e);
   }