Java 和 SQL：返回 null 还是抛出异常？

Question

这是另一个有争议的主题，但这次我只寻找简单且有记录的答案。场景：

让我们假设以下方法：

 public static Hashtable<Long, Dog> getSomeDogs(String colName, String colValue) {
  Hashtable<Long, Dog> result = new Hashtable<Long, Dog>();
  StringBuffer sql = null;
  Dog dog = null;
  ResultSet rs = null;
      try {
          sql = new StringBuffer();
          sql.append("SELECT * FROM ").append("dogs_table");
          sql.append(" WHERE ").append(colName).append("='");
          sql.append(colValue).append("'");
          rs = executeQuery(sql.toString());
              while (rs.next()) {
                  dog= new Dog();
                  //...initialize the dog from the current resultSet row
              result.put(new Long(dog.getId()), dog);
              }
          }
     catch (Exception e) {
         createErrorMsg(e);
         result = null; //i wonder....
         }
     finally {
         closeResultSet(rs); //this method tests for null rs and other stuff when closing the rs.
     }
   return result;
 }

问题：

1。你有什么建议来改进这种归还一些狗的技术，有一些属性？

2。 rs.next() 将为 null ResultSet 返回 false，或者将生成异常，如下所示：

字符串 str = null; System.out.println(str.toString());

3。如果在从 ResultSet 的当前行初始化 dog 对象时，发生了一些不好的事情，例如：连接失败、不兼容的值已传递给 dog 属性设置器等，该怎么办？我现在可能在哈希表中有 10 个元素，或者没有（第一行）。下一步将是什么： a) 返回 null 哈希表； b) 返回结果哈希表，就是这个阶段的样子； c) 抛出异常：这里的异常类型是什么？

4。我想你们都会同意这一点：没有什么不好的事情发生，审讯中没有行，将返回一个空值。但是，@Thorbjørn Ravn Andersen 说 here 我应该返回 NullObject 而不是 null 值。我想知道那是什么。

5。我注意到人们和一群人说应该将应用程序分成层或某种级别。考虑到上面的例子，这里有哪些层，除了我能想到的这些：

Layer1 :: 执行操作的数据库层：此方法。

第二层 :: ??? : 构造新 Dog 对象的某个层：我的 Dog 对象。

第三层 :: ? ：我打算对狗的集合做某事的某个层：主要是 GUI 层，或用户界面的子层。

按照应用流程，如果第一层发生异常，最好的办法是什么？我的想法：捕获异常，记录异常，返回一些值。这是最佳做法吗？

曼尼感谢您的回答，我期待看到其他人对这些问题的看法。

Answer 1

我会避免以下情况

   sql.append("SELECT * FROM ").append("dogs_table");
   sql.append(" WHERE ").append(colName).append("='");
                        sql.append(colValue).append("'");

而是使用 PreparedStatement 及其相关的参数设置方法 (setString()) 等。这将防止 colValue 的值出现问题，以及 SQL 注入攻击（或更一般地说，colValue 形成一些SQL 语法）。

如果集合只是空的，我将永远返回 null。从客户的角度来看，这似乎非常违反直觉，并且完全出乎意料。

我不建议在错误条件下返回 null，因为您的客户必须明确检查这一点（并且可能会忘记）。如果需要，我会返回一个空集合（这可能类似于您对空对象的评论），或者更有可能抛出异常（取决于情况和严重性）。该异常很有用，因为它将携带一些与遇到的错误相关的信息。 Null 什么也不告诉你。

如果在构建Dog 对象时遇到问题该怎么办？我认为这取决于你希望你的应用程序有多健壮和有弹性。返回Dogs 的子集是否有问题，还是完全是灾难性的，您需要报告这个？这是一个应用程序要求（过去我必须满足两种情况 - best-effort 或 all-or-nothing）。

几个观察。我会使用 HashMap 而不是旧的 Hashtable （同步所有访问，更重要的是，不是正确的 Collection - 如果你有一个 Collection 你可以将它传递给任何其他期望 的方法任何 Collection) 和StringBuilder 超过StringBuffer 出于类似原因。不是大问题，但值得了解。

Answer 2

你问了五个问题

1.你有什么方法可以改进这种返回一些具有某些属性的狗的技术？

实际上有几个。

• 您的方法是静态的 - 这并不可怕，但会导致您使用另一个静态“executeQuery”，这对我来说有点单例的味道...
• “Dogs”类违反了 OO 命名惯例 - 复数名词不会成为好的类名，除非该类的一个实例包含一组事物 - 而且 Dogs 实际上是“Dog”。
• HashTable 几乎已被弃用。 HashMap 或 ConcurrentHashMap 提供更好的性能。
• 我看不出有理由使用多个追加来创建查询的第一部分 - 这还不错，但它的可读性不如预期，所以 sql.append ("SELECT * FROM dogs_table WHERE ");如果您只是要硬编码选定的列 (*) 和表名 (dogs_table)，那么这是一个更明智的开始。

2. rs.next() 将为 null ResultSet 返回 false，或者会生成异常

这似乎不是一个问题，但是是的，一旦不再有任何行要处理，rs.next() 就会返回 false。

3.如果在从 ResultSet 的当前行初始化狗对象时，发生了一些不好的事情怎么办​​

如果“发生了不好的事情”，接下来要做什么取决于您和您的设计。有宽容的方法（尽可能返回所有行）和不宽容的方法（抛出异常）。我倾向于倾向于“不宽容”的方法，因为使用“宽容”的方法，用户不会知道您没有返回所有存在的行 - 只是您在错误之前得到的所有行。但也可能存在宽恕方法的情况。

4.我想你们都会同意这一点：没有什么不好的事情发生，审讯中没有行，将返回一个空值。

这不是一个明显的正确答案。首先，这不是书面方法中发生的事情。它将返回一个空的 HashTable（这就是“空对象”的含义）。其次，在“未找到结果”的情况下，null 并不总是答案。

我看到了 null，但我也看到了一个空的结果变量。我声称它们都是正确的方法，但我更喜欢空的结果变量。但是，始终最好保持一致，因此请选择一种返回“无结果”的方法并坚持下去。

5.我注意到一些人和一群人说应该将应用程序分成层或某种级别。

这比其他问题更难回答，因为没有看到您的应用程序的其余部分。

Answer 3

Null Object Pattern 是一种设计模式，您总是返回一个对象以避免 NPE:s 和代码中的任何 null 检查。在您的情况下，这意味着不是返回null，而是返回一个空的Hashtable<Long, Dogs>。

原因是，由于它是一个集合，而您的其他代码将访问它，因此如果您返回一个空集合，它不会崩溃；它不会被迭代，它不会包含任何令人惊讶的东西，它不会导致 NPE:s 被抛出等等。

确切地说，空对象是类/接口的特殊实现，它完全不做任何事情，因此没有任何副作用。因为它不是null 的性质，所以使用它会让你的代码更干净，因为当你知道你总是会从你的方法调用中得到一个对象时无论方法内部发生什么你都不会'甚至不必检查空值，也不让代码对它们做出反应！因为 Null Object 不做任何事情，您甚至可以将它们作为 singletons 随意放置，从而节省内存。

Answer 4

不要像你正在做的那样通过连接字符串来构建 SQL 查询：

sql = new StringBuffer();
sql.append("SELECT * FROM ").append("dogs_table");
sql.append(" WHERE ").append(colName).append("='");
sql.append(colValue).append("'");

这会使您的代码容易受到众所周知的安全攻击SQL injection。而不是这样做，使用PreparedStatement 并通过调用适当的set...() 方法来设置参数。请注意，您只能使用它来设置列 values，不能像您正在做的那样使用它来动态构造列 name。示例：

PreparedStatement ps = connection.prepareStatement("SELECT * FROM dogs_table WHERE MYCOL=?");
ps.setString(1, colValue);

rs = ps.executeQuery();

如果您使用PreparedStatement，JDBC 驱动程序将自动处理转义可能出现在colValue 中的某些字符，因此 SQL 注入攻击不再起作用。

Answer 5

如果发生错误，则抛出异常。如果没有数据，则返回一个空集合，而不是 null。 （此外，通常您应该返回更通用的“地图”，而不是具体的实现），

Answer 6

您可以通过使用Spring-JDBC 而不是普通的旧 JDBC 来显着减少样板 JDBC 代码的数量。这是使用 Spring-JDBC 重写的相同方法

public static Hashtable<Long, Dogs> getSomeDogs(String colName, String colValue) {

    StringBuffer sql = new StringBuffer();
    sql.append("SELECT * FROM ").append("dogs_table");
    sql.append(" WHERE ").append(colName).append("='");
    sql.append(colValue).append("'");

    Hashtable<Long, Dogs> result = new Hashtable<Long, Dogs>();

    RowMapper mapper = new RowMapper() {

        public Object mapRow(ResultSet rs, int rowNum) throws SQLException {
            Dogs dog = new Dogs();
            //...initialize the dog from the current resultSet row
            result.put(new Long(dog.getId()), dog);
        }
    };
    (Hashtable<Long, Dogs>) jdbcTemplate.queryForObject(sql, mapper);
}

春天照顾：

1. 遍历 ResultSet
2. 关闭结果集
3. 一致地处理异常

正如其他人所提到的，您确实应该使用 PreparedStatement 而不是 String（或 StringBuffer）来构造 SQL。如果由于某种原因你不能这样做，你可以通过构造这样的 SQL 来提高查询的可读性：

    String sql = 
        "SELECT * FROM dogs_table " +
        "WHERE " + "colName" + " = '" + colValue + "'";