赛格故障的unique_ptr与工厂设计尝试

我得到一个segmentation fault试图用unique_ptr创建派生类的实例时。之前，我曾编码的七个子类的实例每次，一前一后的代码工作正常。赛格故障的unique_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

你看不到任何原因赛格故障？

来源

2013-01-06 Luis

function createInstance必须返回nullptr，则需要检查指针是有效的机会：

if (domain.get()) 
{ 
    domain->readDatabaseFields (database_config); 
}

来源

2013-01-06 11:21:56 billz

事实上，它指向NULL。忘记包括该支票。在createInstance中，在'main'中修改''dom =='voice''应该有'voice'大写。谢谢。 – Luis

此行是错误：

ComumPtr ptr;  // initialized to nullptr.

虽然我明白，无效是那么容易，它也是搬起石头砸自己的脚（无论是在C++或Java）的最佳方式，因为现在任何一个使用这个函数的结果需要进行检查。

相反，你可以：

使用空对象，方法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-06 13:52:52

赛格故障的unique_ptr与工厂设计尝试

回答

相关问题