Java 这是数据库查询的清晰代码吗

我不熟悉java语言。我只是想确定这段代码对于我未来的项目是否足够好和干净；我不希望这会导致我的应用程序内存泄漏

public Response returnAllPcParts() throws Exception {
    PreparedStatement ps = null;
    Connection conn= null;
    ResultSet rs =null;
    String returnString = null;
    Response rb = null;

    try {
        conn = DatabaseConnection.databaseConn().getConnection();
        ps = conn.prepareStatement("SELECT * "+ "FROM r.order");    
        rs = ps.executeQuery();

        ToJSON converter = new ToJSON();
        JSONArray json = new JSONArray();

        json= converter.toJSONArray(rs);
        ps.close();
        returnString = json.toString();
        rb = Response.ok(returnString).build();

    } catch (Exception e) {
        e.printStackTrace();
    } finally{
        if(rs   !=  null) rs.close();
        if(ps   !=  null) ps.close();
        if(conn != null) conn.close();
    }

    return rb;
}

---

如果您将此代码用于一个非常重要的项目或应用程序，我建议您使用类似于或其他帮助程序库的代码。

---

它清除了大部分样板代码（即：获取连接、关闭结果集catch/finally blocks等的代码）

您有Java 7吗？如果是这样的话，我建议使用新的“尝试使用资源”语法。这个问题似乎与主题无关，因为它是关于代码审阅的。这个问题似乎与主题无关，因为它属于codereview.stackexchange.com，看起来不错。如果结果集太大，可能会出现问题。尝试限制resultset-->屏幕模式。但是它看起来真的很好，但是你总是可以对代码进行改进。A

ps.close（）

太多了。