0

我segmentation fault在尝试使用 aunique_ptr创建派生类的实例时遇到了问题。之前，我一个接一个地编写了七个派生类的每个实例化代码，并且代码工作正常。

当前代码如下：

typedef std::unique_ptr<Comum> ComumPtr;

ComumPtr createInstance ( string dom, map<string, string> & config, map<string, string> & config_fields )
{
    ComumPtr ptr;       // initialized to nullptr.
    if ( dom == "voice" ) {
    ptr.reset ( new Voice (config, config_fields) );
    //    } else if ( dom == "account" ) {     // FOR OTHER DERIVED CLASSES
    //  ptr.reset ( new Account (config, config_fields) );
    }
    return ptr;
}

// At  main function:
 for (vector<string>::const_iterator cit = for_domain.begin(); cit != for_domain.end(); ++cit) { 
    const char * section (cit->c_str());
    string fsn = *cit + "_fields";
    const char * fields_section_name (fsn.c_str());
    const char * db_section ("Database");

    map <string, string> domain_config = cfg.getSectionConfig (config_file.c_str(), section);
    map <string, string> domain_config_fields  = cfg.getSectionConfig (config_file.c_str(), fields_section_name);
    map <string, string> database_config = cfg.getSectionConfig (config_file.c_str(), db_section);

    std::unique_ptr<Comum> domain = createInstance(*cit, domain_config, domain_config_fields);

    domain->readDatabaseFields (database_config);   // <- segmentation fault

您认为有什么理由会导致故障吗？

4

2 回答 2

1

function createInstance有机会返回nullptr，您需要检查指针是否有效：

if (domain.get())
{
  domain->readDatabaseFields (database_config);
}
于 2013-01-06T11:21:56.087 回答
1

这一行是错误：

ComumPtr ptr;       // initialized to nullptr.

虽然我知道 nullity 很容易，但它也是自责的最好方法（无论是在 C++ 还是 Java 中），因为现在需要检查对这个函数结果的任何一次使用。

相反，您可以：

  • 使用 Null 对象：该方法readDatabaseFields什么也不做
  • 选择抛出异常而不是返回空指针

上述替代方案没有一个天生就比另一个更好，这在很大程度上取决于情况；但是两者都比返回 null 好unique_ptr。

假设您选择异常方法：

ComumPtr createInstance ( string dom, map<string, string> & config, map<string, string> & config_fields )
{
    if ( dom == "voice" ) {
        return ComumPtr ( new Voice (config, config_fields) );
    } 
    if ( dom == "account" ) {
        return ComumPtr ( new Account (config, config_fields) );
    }

    throw std::runtime_error("Unknown config field");
}
于 2013-01-06T13:52:52.887 回答