我有一种使用JDBC从数据库中获取用户的方法:
public List<User> getUser(int userId) {
String sql = "SELECT id, name FROM users WHERE id = ?";
List<User> users = new ArrayList<User>();
try {
Connection con = DriverManager.getConnection(myConnectionURL);
PreparedStatement ps = con.prepareStatement(sql);
ps.setInt(1, userId);
ResultSet rs = ps.executeQuery();
while(rs.next()) {
users.add(new User(rs.getInt("id"), rs.getString("name")));
}
rs.close();
ps.close();
con.close();
} catch (SQLException e) {
e.printStackTrace();
}
return users;
}
我应该如何使用Java 7 try-with-resources来改进此代码?
我已经尝试使用下面的代码,但是它使用了很多try
块,并且并没有太大地提高可读性。我应该try-with-resources
以其他方式使用吗?
public List<User> getUser(int userId) {
String sql = "SELECT id, name FROM users WHERE id = ?";
List<User> users = new ArrayList<>();
try {
try (Connection con = DriverManager.getConnection(myConnectionURL);
PreparedStatement ps = con.prepareStatement(sql);) {
ps.setInt(1, userId);
try (ResultSet rs = ps.executeQuery();) {
while(rs.next()) {
users.add(new User(rs.getInt("id"), rs.getString("name")));
}
}
}
} catch (SQLException e) {
e.printStackTrace();
}
return users;
}
Connection
,PreparedStatement
和ResultSet
也。没有理由不这样做,因为try-with-resources使它变得如此简单,并使我们的代码更符合我们的意图。
try (ResultSet rs = ps.executeQuery()) {
因为ResultSet对象会自动由生成该对象的Statement对象关闭