异常消息为空

Question

我有一个异常类，如下所示：

class FileNotFoundException : public std::exception
{
public:
    FileNotFoundException(const char* message) : errorMessage(message){ }
    const char* what() const throw() override
    {
        return this->errorMessage;
    }
private:
    const char* errorMessage;
};

我像这样throw这个例外：

std::string message = "Message";
throw ::FileNotFoundException(message.c_str());

但是当我尝试使用以下方法处理它时：

try
{
    // the code that throws 
}
catch(::FileNotFoundException& ex)
{
    std::string message = ex.what(); 
}

string为空。如果有人能帮忙，我将不胜感激。

Answer 1

不能只存储指向消息的指针。尝试将其存储在 std::string 中，或者更好的是将其传递给父构造函数。在这种情况下，也许从std::runtime_error继承会更好。

下面是一个完整的示例：

#include <iostream>
#include <string>
#include <stdexcept>
class FileNotFoundException : public std::runtime_error
{
public:
  FileNotFoundException(const char* message) : std::runtime_error(message)
  {
  }
};
int main()
{
  try {
    throw ::FileNotFoundException("oops, something happened");
  }
  catch(const ::FileNotFoundException& ex) {
    std::cout << "Exception: '" << ex.what() << "'" << std::endl;
  }
}

编译和运行：

$ g++ -W -Wall --std=gnu++11 a.cpp -oa
$ ./a
Exception: 'oops, something happened'

简而言之(没有细节(：类std::exception没有任何构造函数。它只是所有其他异常使用的父类。另一方面，std::runtime_error有一个构造函数，可以为您正确存储消息。完整的解释可以在差异中找到： std：：runtime_error vs std：：exception((

我认为这种方法比定义what()并使用std::string自己存储消息要好。也就是说，如果您对异常类没有特殊需求。

还应查看C++异常层次结构。

Answer 2

你的问题在这里：

throw ::FileNotFoundException(message.c_str());

您正在存储指向异常中message拥有的内存的指针。当message超出范围时(在抛出期间发生(，数据将不再有效。这意味着this->errorMessage返回一些未定义的内存。要修复它，您可以将一些非常常量的字符串传递给您的异常，或者您需要异常来拥有该字符串，例如通过errorMessage成为std::string。