java数据访问：这是java数据访问代码的良好风格，还是它太多了？

Question

这是java数据访问代码的良好风格，还是尝试最后太多了？

public List<Item> getItems() throws ItemException {
    ArrayList<Item> items = new ArrayList<Item>();
    try {
        Connection con = ds.getConnection();
        try {
            PreparedStatement pStmt = con.prepareStatement("SELECT ....");
            try {
                ResultSet rs = pStmt.executeQuery();
                try {
                    while (rs.next()) {
                        Item item = new Item();
                        item.setItemNo(rs.getString("item_id"));
                        // ...
                        items.add(item);
                    }
                } finally {
                    rs.close();
                }
            } finally {
                pStmt.close();
            }
        } finally {
            con.close();
        }
    } catch (SQLException e) {
        throw new ItemException(e);
    }
    return items;
}

Answer 1

将其与我的代码进行比较：

Connection con = null;
PreparedStatement pStmt = null;
ResultSet rs = null;
try
{
    con = ds.getConnection();
    pStmt = con.prepareStatement("SELECT ....");
    rs = pStmt.executeQuery();
    while (rs.next()) {
        Item item = new Item();

        item.setItemNo(rs.getString("item_id"));
        ...

        items.add(item);
    }
}
finally {
    rs = DBUtil.close (rs);
    pStmt = DBUtil.close (rs);
    con = DBUtil.close (rs);
}

close() 如下所示：

public static ResultSet close (ResultSet rs) {
    try {
        if (rs != null)
            rs.close ();
    } catch (SQLException e) {
        e.printStackTrace (); 
        // Or use your favorite logging framework.
        // DO NOT THROW THIS EXCEPTION OR IT WILL
        // HIDE EXCEPTIONS IN THE CALLING METHOD!!
    }
    return null; // Make sure no one uses this anymore
}

[编辑] 您需要为其他类型复制此代码。

我还将所有这些移至名为 DBOp 的帮助程序类中，因此我只需重写 processRow(ResultSet row) 即可进行实际处理，并且可以省略所有样板代码。在 Java 5 中，DBOp 的构造函数如下：

public DBOp (Logger log, DataSource ds, String sql, Object... param)

我传入记录器，以便可以显示哪个实例实际上正在轮询数据。

Answer 2

根据 API 文档，语句< /a> 和 结果集 是通过 Connection 隐式关闭的，所以是的 - 其中 2 个 try/finally 块是不必要的（而且非常难看）。当您为大量查询保留单个连接时，显式关闭语句可能会很有用，以减少内存使用。

Answer 3

...并且您不应该在该 getItems() 方法中关闭连接。看起来，您将 Connection 对象存储在该 ds 对象中。因此您应该将其留给 ds 来关闭它。否则，存在 ds 使用 getConnection() 方法返回已经关闭的连接的风险，我认为这是一种不需要的和意外的行为；）