Передача std :: list из одной функции в другую по значению

Question

У меня есть программа на C++, как показано ниже. Я пытаюсь передать std :: list из одной функции в другую по значению. Я ожидаю, что список будет доступен в функции вызывающего абонента с помощью итератора? Я ожидаю, что возврат вызовет конструктор копирования std::list, и он будет доступен в вызывающем. Я ошибаюсь в своем предположении? Если нет, то почему я получаю ошибку сегментации.

#include <list> 
#include <map> 
#include <string> 
#include <set> 
#include <iterator> 
#include <iostream> 

const char *sy_path = "/var/log"; 
struct Setting 
{ 
    typedef std::list<const Setting*> List; 

    const char*  path; 
    const char*  filename; 
    const char*  name; 

    int def; 
    int min; 
    int max; 

    struct Original 
    { 
     const char*  filename; 
     const char*  name; 

     Original(const char* filename_, const char* name_) 
        :filename(filename_), name(name_) 
     { 

     } 
    }original; 
    static const List settings(); 
}; 

const Setting::List Setting::settings() 
{ 
    const Setting c_settings[] = 
    {              //default min max  
     { sy_path, "cs.cfg",  "num_a",     1, 1, 29,   Original("sys.cfg", "num_a") } 
     ,{ sy_path, "cs.cfg",  "num_b",     1, 1, 6,    Original("sys.cfg", "num_b") } 
     ,{ sy_path, "cs.cfg",  "num_c",     1, 1, 29,   Original("sys.cfg", "num_c") } 
    }; 

    Setting::List lst; 

    int numelem = sizeof(c_settings)/sizeof(Setting); 
    for (int i = 0; i < numelem; i++) 
    { 
     const Setting & tmpSetting = c_settings[i]; 
     lst.push_back(&tmpSetting); 
    } 

    return lst; 
} 

static int get_settings(void) 
{ 
    Setting::List lst; 
    lst = Setting::settings(); 

    for (Setting::List::const_iterator it = lst.begin() ; it != lst.end(); ++it) 
    { 
     const Setting *cs = *it; 
     std::cout << "path: " <<cs->path << "filename: " <<cs->filename << "name: " << cs->name << std::endl; 
    } 
} 

int main() 
{ 
    get_settings(); 

    return 0; 
} 

Answer 1

Да, return lst; возвращает копию lst. Проблема в том, что вы помещаете указатели lst в данные, расположенные в стеке (переменная const Setting c_settings[]). Эти указатели становятся недействительными после того, как вы вернетесь из функции, следовательно, с ошибкой сегментации. Решение состоит в том, чтобы либо выделить память для ваших настроек в куче, либо использовать std::list<Setting>.

typedef std::list<Setting> List; 

lst.push_back(c_settings[i]); // make sure you have the right copy constructor 

или

lst.push_back(new Setting(c_settings[i])); // make sure you have the right copy constructor 

Кроме того, я хотел бы избежать использования const char * и использовать std::string вместо этого.