Question

以下内容涉及家庭作业。 Restraunt宠物项目类型的东西，任务是更新它使用向量。我遇到的问题是：这最终导致核心分段错误，但是当我使用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;
                }
        }
}

设置：我将服务员和订单存储在向量中。
在运行时：当它执行服务员 - > getName（）时，它会抱怨它是无效的读取，并且内存位置已经存在通过deallocater向量释放。我在这个问题上的逻辑：它向前看并看到向量本身不再被访问，因此解除分配。由于在此之后我不再写作，因此内存位置保持不变。当它试图读取它看到的位置时它已被释放，因此无效读取但它仍然获得适当的数据。
那么我的问题是，我想是两折：这个逻辑听起来合适吗？
我该怎么做才能解决它？

#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);

这将获取向量内对象的地址。合法的，它非常脆弱。 std::vector可以在调整大小时将内容移动到内存中（例如，当你进入它时）。

此（或其他地方的类似代码）可能是您的段错误的原因。看到你动态分配对象，也许你应该声明你的向量只包含指针：

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

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

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

带矢量的核心分段故障

1 个答案: