从文件读入指针向量的C++

Question

我的问题是，我有一个基类和3个子类，我想做一个向量，在那里我可以放置代表所有3个子类的所有元素。下面是处理从文件中读入的代码部分

vector<Robot*> robots;
vector<Mac> mac;
vector<Eco> eco;
vector<Pro> pro;
vector<int> charge;
vector<int> deliver;

try {
    string s;
    ifstream f;

    do {
        cout << "Add meg a filenevet" << endl;
        cin >> s;
        f.open(s.c_str());
    } while (!f.good());

    cout << "adatok beolvasasa..." << endl;

    int napok;
    if (!(f >> napok)) throw 1;
    charge.resize(napok);
    deliver.resize(napok);

    for (int i = 0; i<napok; i++) {
        if (!(f>>charge[i])) throw 1;
        if (!(f>>deliver[i])) throw 1;
    }

    string type, name;
    int battery;
    int m = 0; int e = 0; int p = 0;
    std::string line;
    while (std::getline(f, line)) {
      stringstream ss(line);
      if (ss >> type && ss >> name && ss >> battery) {

            if (type=="Mac") {
                cout << "mac" << endl;
                Mac r = Mac(name,battery);
                mac.push_back(r);
                robots.push_back(&mac[m]);
                m++;
            };
            if (type=="Eco") {
                cout << "eco" << endl;
                Eco r = Eco(name,battery);
                eco.push_back(r);
                robots.push_back(&eco[e]);
                e++;
            }
            if (type=="Pro") {
                cout << "pro" << endl;
                Pro r = Pro(name,battery);
                pro.push_back(r);
                robots.push_back(&pro[p]);
                p++;
            };
        }
    }

到目前为止，这是有效的，它编译并运行良好，但当我试图访问一个函数ex robots[i].getBattery();时，程序冻结。似乎这些指针只是指向无处，但我不知道为什么：

Answer 1

问题是你有两种独立的方法来访问你的对象，而这两种方法之间没有任何同步。

如果您将一项推入向量中，然后获取指向它的指针，则仅当向量的存储不被移动或无效时，该指针才有效。在您的示例中，重复的push_back()调用很可能触发向量重新分配，并且不能保证这会在适当的位置发生，因此元素可能会移动到新的内存位置，但显然指针不会更新。因此，您最终会得到指向任何地方的指针。

有一组容器来保存数据，而另一组容器只保存指向所述数据的指针，这在我看来是非常糟糕的设计。在我看来，您有两个选择来改进代码并修复您遇到的问题。在这两种情况下，您只需要一个容器来保存指向多态元素的指针，这只是您声明容器的方式：

1)使用boost::ptr_vector存储指向多态对象的指针。这个容器被设计为保存指针( std::vector没有，后者具有值语义)，并负责其中包含的对象的生命周期管理。

2)如果不能使用boost，可以使用std::vector<std::shared_ptr<Robot> >来保存指向多态对象的指针。

在这两种情况下，您都删除了按值保存项的容器，并且在if语句中，您在堆上分配对象，然后将指针放入容器中。