exit(） 在应该返回引用的函数内调用

exit() call inside a function which should return a reference

本文关键字：引用 函数 调用 返回 exit      更新时间：2023-10-16

在一个库中，我有一个函数，它搜索数据库中的键并返回对对象的非常量引用。我想处理找不到键的情况，这通常是由调用函数时的错误引起的。这种情况非常糟糕，程序无法继续，因此我打印了一条消息以帮助发现错误并调用exit(1)。问题在于 return 语句，在这种情况下永远不会执行，但无论如何都必须在那里。如果它是一个指针，我可以return nullptr;但有参考？我应该做类似这个伪代码的事情吗？

 Type & get(const Key & k) {
     if (my_db.key_exists(k)) {
       return my_db.at(k);
     }
     std::cerr << k << " not foundn";
     exit(1); 
     return *(new Type(some_dummy_parameters));
 }

看起来太可怕了！也许我应该避免这样的功能。请让我知道您的意见！

这种情况非常糟糕，程序无法继续，所以我打印了一条消息来帮助发现错误并调用 exit（1）

不。 如果此代码是库的一部分，则库不应是决定应用程序是否应退出的库。

如果文件已打开并需要关闭，或者需要清理其他资源，或者数据库类的用户想要记录错误并继续执行其他操作，该怎么办？

答案绝不是你现在正在做的事情。 抛出异常，返回错误代码等。 但不要在库或类代码中关闭应用程序。

信

不信由你，有一个商业数据库库完全按照你正在做的事情（关闭应用程序）。 他们从图书馆的用户那里得到了很多愤怒的回应，为什么他们意外关闭了应用程序。 你知道吗 - 给客户的答案是"我们认为错误严重到足以停止应用程序，因为我们的库无法继续正常工作"。 这不仅是糟糕的推理，而且近乎傲慢，客户让他们知道这一点。

异常

这是许多程序中的常见情况。为了克服这个问题，使用了异常。

  • 为了处理意外情况，将创建新的异常并从代码中"抛出"。
  • 然后它们必须被调用函数的程序"捕获"。

您可以在此处阅读有关例外的更多信息。

希望这有帮助。

正如其他受访者所说，答案应该是：抛出一个异常......

Type & get(const Key & k) {
     if( !my_db.key_exists(k) ) {
          std::stringstream error;
          error << "key " << k << " not found";
          throw std::runtime_error(error);
     }
     return my_db.at(k);
}

库不应退出最托管的应用程序。

使用"返回 NULL"，进入"不一致状态"，在每次调用中都返回 NULL。
库用户必须处理它。

或例外...

相关文章：