关于返回 null 的重构代码的有趣思考问题

Question

我有兴趣听到您的反馈。我最近看到一些按以下方式实现的 Java 代码：

Object1 SomeMethod(String key) {
    Object1 object1 = null;
    List<Object1> objectList = getAllRecordsWithKeyFromDatabase(key);
    if (!objectList.isEmpty()) {
        object1 = objectList.get(0);
    }
    return object1;
}

void AnotherMethod() {
    ...
    Object1 object1 = SomeMethod(key);
    if (object1 == null) {
        // throw exception
    }
    // continue working
}

每当在没有上下文的情况下返回 null 时，我总是很担心。我希望 SomeMethod 的响应更加明确，并正在考虑重构它。从 SomeMethod 抛出异常可以提供一种方法来做到这一点。它会在上下文中发生并发生在故障点。

我想知道是否有另一种方法可以让 SomeMethod 通知 AnotherMethod“在数据库中找不到任何内容”，而不是假设 null 始终等于“未找到”。我认为可以使用 NullObject，但我不清楚如果找不到数据， AnotherMethod 应如何避免“继续工作”。

您将如何重构代码？

Answer 1

我不认为它有任何问题，只是我会清楚地记录 SomeMethod 在某些情况下可以返回 null。

但是，如果发现 null 显然是一种例外情况，那么抛出异常是正确的方法。如果您更改 AnotherMethod 以便它声明受检查的异常，那么您的意图对于该方法的用户来说会更加清晰。像这样的事情：

void AnotherMethod() throws SomethingBadHappenedException {
  //snip
}

Answer 2

空对象并不是全部。它适用于某些情况（Collections.emptyXXX() 就是一个很好的例子），但有时您必须区分某些东西和什么都没有。如果什么都不返回是有效状态，那么它应该返回 null。

例外是指特殊情况，即正常情况下不应该发生的事情。捕获和处理异常比检查 null 困难得多。

Answer 3

我曾经参与过一个项目，其中查询结果非常自然地是集合。但在特殊情况下，集合应该为空；在其他情况下，集合应该只包含一个元素。

这些查询是在有时需要数据库故障的对象上执行的。

我们最终达成一致的方法是用我们自己的类型（而不是 ArrayList 或其他类型）返回结果。为了说明这一点：

public interface Results<T> implements Iterable<T> {
  Collection<T> all();
  Iterator<T> iterator();
  /**
   * @return one value from the result, or null if the result is empty.
   */
  T ifAny();

  /**
   * @return one value from the result.
   */
  T one() throws EmptyResultException;

  /**
   * @return if the result is empty, returns null, 
   *         if the result has one value, returns the value,
   *         if the result has more than one value, throws.
   */
  T unique() throws MultiValueResultException;

  /**
   * @return the value if the result has exactly one; throws otherwise
   */
  T exact() throws NoExactResultException;
}

如果语义对您很重要，您可以在您的案例中使用类似的方法：

public final class Result<T> {
  private static final Object NON = new Object();

  private final Object _value;

  public Result(T value) {
    if (value == null) {
      throw ...
    }
    _value = value;
  }

  public T get() {
    return (_value == NON) ? null : (T) _value;
  }

  public T use() {
    if (_value == NON) {
      throw ...
    }
    return (T) _value;
  }
}

请注意，Scala 习惯用法使用 Option 来达到类似的效果。以下是该主题的众多链接之一： http://www.codecommit.com /blog/scala/the-option-pattern