是“断言错误”；一个好的做法？

Question

我目前正在编写一个函数，根据给定的标准从列表中返回一个对象。它看起来像这样：

for (Object object : list) {
    if (condition(object)) {
        return object;
    }
}

这个函数应该总是从列表中返回一些东西，如果没有找到匹配的对象，这是一个错误的调用，一个严重的错误，程序应该停止。

因此，当我启用断言时，我在循环之后执行了以下操作：

assert false; // Will always trigger in debug mode.
return null; // No matter anyway, an AssertionException has already been thrown.

但我想知道我做得好不好？
如果没有，我应该怎么做？自己抛出异常？

无论如何，这种情况有什么规范吗？

Answer 1

我宁愿在调用时检查函数的返回值。

if (yourFunctionWithList(parameter) == null)
   //error handling, maybe throw new NPException or whatever. 
else
   //some object was returned

您也可以编写自己的异常类并以您想要的任何方式处理它。

我个人认为assert false 不是好习惯。

编辑

如果是关于将被抛出的AssertionException，那么你也可以像这样使用它

throw new AssertionError ("your error msg here");

所以你可以用同样的方式处理它

Answer 2

断言或抛出异常的问题在于您需要使用异常处理来处理并非真正异常的情况。

此外，您无法确定是什么引发了您捕获的异常/断言。它可能在您期望的地方被抛出，但它也可能被抛出在过滤代码中，例如 - 因此检查异常以检测“未找到”情况可能会混淆其他问题用那种情况

另一种方法是使用Optional（Java 8 之前的 Guava 中存在一个类似的类；或者您可以简单地使用 Set<Object>，但这并不表示您期望正好是 0 或 1 值找到）：

Optional<Object> method(List<?> list) {
  for (Object object : list) {
      if (condition(object)) {
          return Optional.of(object);
      }
  }
  return Optional.empty();
}

现在，在您的调用代码中，您明确知道您可能在列表中找不到项目：

Optional<Object> opt = method(list);
if (opt.isPresent()) {
  Object obj = opt.get();
  // Handle the fact it was found.
} else {
  // Handle the fact it wasn't found.
}

而不是您可能忘记添加的异常处理。

Answer 3

我个人不信任他们；它们通常在默认情况下被禁用，即使在开发环境中也是如此。这会使他们产生一种非常错误的安全感——静态分析工具和其他程序员都可能错误地认为它们按预期工作。

Answer 4

你应该抛出一个异常，这是java中的正确方法：

for (Object object : list) {
    if (condition(object)) {
        return object;
    }
}
throw new Exception('Failed: no matching item found!');

如果有一天你改变主意并且不希望程序停止，你将能够捕获异常。