c++ - vector 的核心段错误

Question

以下是家庭作业。 Restraunt 宠物项目类型的东西，任务是更新它以使用 vector 。我遇到的问题是: 这最终会导致核心段错误，但在我使用 valgrind 时能够适本地检索所有信息。

void Table::partyCheckout(void)
{
        if(status == SERVED)
        {
                cout << " ---------------- " << endl;
                cout <<"Table: " << tableId << "\nParty Size: " << numPeople << "\nWaiter: " << waiter->getName() << "\nSummary: " <<endl;
                order->requestSummary();
                cout << "Total: " << endl;
                order->requestTotal();
                cout << " ---------------- " << endl;
                status = IDLE;
        }
        else
        {
                cout << "Error: " << tableId << " ";
                if(numPeople == 0)
                {
                        cout << "No one is at this table." << endl;
                }
                else
                {
                        cout << "This party hasn't been served." << endl;
                }
        }
}

设置:我将服务员和订单存储在 vector 中。
在运行时:当它执行 waiter->getName() 时，它会提示这是一次无效读取，并且内存位置已空闲通过解除分配器由 vector 处理。
我在这件事上的逻辑:它向前看，发现 vector 本身没有被再次访问，所以释放它。因为在这一点之后我不再写了，所以内存位置保持不变。当它试图读取它看到它已被释放的位置时，因此读取无效但它仍然获得适当的数据。
那么我的问题，我想有两个方面:
这个逻辑听起来正确吗？
我应该怎么做才能修复它？

#ifndef HW3_H
#define HW3_H
#include <vector>
#include "Table.h"
#include "Waiter.h"

class hw3
{
private:

        vector<Table> tables;
        vector<Waiter> waiters;
        vector<Order> orders;
public:
        void begin();

};
#endif

.cpp文件，大部分配置:

 ifstream configFile("config.txt"); //This guy is for initializing things
        string line;

        Menu theMenu;
        getline(configFile, line);
        stringstream intMaker;
        int t1;
        int t2;
        string temp;
        string temp2;
        string temp3;
        while (true)
        {
                getline(configFile, line);
                Tokenizer str(line, " \n");
                if(line =="")
                {


                        break;
                }
                else
                {
                        temp = str.next();
                        temp2 = str.next();
                        intMaker << temp;
                        intMaker >> t1;
                        intMaker.str("");
                        intMaker.clear();
                        intMaker << temp2;
                        intMaker >> t2;
                        intMaker.str("");
                        intMaker.clear();
                        tables.push_back(*(new Table(t1,t2)));


                }


        }

        getline(configFile, line);
        while (true)
        {
                getline(configFile, line);
                Tokenizer name(line, " ");
                string tabl = "";
                //Siphon off the name and the tables.
                temp = name.next();
                tabl = name.next();
                Tokenizer strink(tabl, ",\n");
                int numTables = (int) tables.size();
                Table * tabs[numTables];
                t1 = 0;
                int keepinTabs = 0;
                while(true)
                {
                        string temp2 = strink.next();

                        if (temp2 == "")
                        {

                                 break;
                        }
                        else
                        {
                                 intMaker << temp2;
                                 intMaker >> t1;

                                 intMaker.str("");
                                 intMaker.clear();
                                 for(int i = 0; i < numTables; i++)
                                 {
                                        if(tables.at(i).getTableId() == t1)
                                        {
                                                tabs[keepinTabs] = &tables.at(i);
                                        }
                                 }
                                 keepinTabs++;
                        }

                }

                waiters.push_back(*(new Waiter(temp, tabl, *tabs))); //Waiter(name, list of tables, and an array of table numbers.
                for(int j = 0; j < keepinTabs; j++)
                {
                        for(int i = 0; i < tables.size(); i++)
                        {
                                if(tabs[j]->getTableId() == tables[i].getTableId())
                                {
                                        tables.at(i).assignWaiter(&(waiters.back()));
                                }
                        }
                }
                if(line == "")
                {
                        break;
                }
        }

Answer 1

我可以看到多个问题:

tables.push_back(*(new Table(t1,t2)));

这段代码动态分配一个类型为Table的对象，然后将这个对象的拷贝压入tables，然后忘记地址动态分配的对象 - 你正在泄漏内存。

waiters.push_back(*(new Waiter(temp, tabl, *tabs)));

和上面一样，这次用Waiter。

tabs[keepinTabs] = &tables.at(i);

这会获取 vector 中对象的地址。虽然合法，但它非常脆弱。 std::vector 可以在调整大小时(例如，当您将其插入时)在内存中四处移动其内容。

这(或其他地方的类似代码)可能是您的段错误的原因。鉴于您正在动态分配对象，也许您应该声明您的 vector 只包含指针:

vector<Table*> tables;
vector<Waiter*> waiters;
vector<Order*> orders;

然后你会做例如tables.push_back(new Table(t1, t2));。当然，当您从 vector 中移除动态分配的对象时，您必须确保 delete 动态分配的对象。另一种方法是使用智能指针，例如:

vector<std::shared_ptr<Table> > tables;
vector<std::shared_ptr<Waiter> > waiters;
vector<std::shared_ptr<Order> > orders;