0

好的，所以我有一个基本上运行市场模拟的程序。我的很多程序都运行良好，我只有一个问题。在我的程序中，我有一个检查器结构，如下所示：

 14 struct Checker
 15 {
 16  int m_money_in_register;
 17  int m_start_work;
 18  int m_time_checkout;
 19  Cust *m_cust;
 20 };

我创建了一个指向检查器对象数组的指针，如下所示：

118 Checker *checkers = new Checker[num_checkers]; // Initializing all checkers
119  for(int i = 0; i < num_checkers; i++)
120  {
121   checkers[i].m_money_in_register = 500;
122   checkers[i].m_start_work = 0;
123   checkers[i].m_cust = NULL;
124   checkers[i].m_time_checkout = 0;
125  }

这工作得很好，当 checkers[i].m_cust = NULL 时，这意味着检查器可用，我可以将我的 checkout_queue 中的 Cust 对象分配给检查器。

我的程序为第一个签出的 Cust 执行我希望它执行的操作，但是当该 Cust 完成签出时，我想设置 checkers[i].m_cust = NULL 以便它再次可供客户使用，但是，这给了我一个分段错误，如下所示：

195 if(checkers[i].m_time_checkout == clock && checkers[i].m_cust->get_status() == "shopper" && checkers[i].m_cust != NULL)
196      {
197       int amount_paid = checkers[i].m_cust->get_num_items()*5;
198       checkers[i].m_money_in_register += amount_paid;
199       checkers[i].m_cust->print_done_checkout(cout, clock, amount_paid, i);
200       Cust *tmp = checkers[i].m_cust;
201       delete tmp;
202       num_custs--;
203       checkers[i].m_cust = NULL; // SEG FAULT CAUSED BY THIS
204      }
205
206      if(checkers[i].m_time_checkout == clock && checkers[i].m_cust->get_status() == "robber" && checkers[i].m_cust != NULL)
207      {
208       int checker_cash = checkers[i].m_money_in_register;
209       checkers[i].m_cust->print_done_stole(cout, clock, checker_cash, i);
210       checkers[i].m_money_in_register = 0;
211       Cust *tmp = checkers[i].m_cust;
212       delete tmp;
213       num_custs--;
214       checkers[i].m_cust = NULL; // SEG FAULT CAUSED BY THIS
215      }

如何正确将对象设置为 NULL？我得到的输出示例如下：

1: Joe entered store
Should be done at: 7
2: James entered store
Should be done at: 8
3: Bob entered store
Should be done at: 9
4: Abby entered store
Should be done at: 10
5: Leo entered store
Should be done at: 11
7: Joe done shopping
7: Joe started checkout with checker 0
time checkout: 13
8: James done shopping
8: James started checkout with checker 1
time checkout: 14
9: Bob done shopping
10: Abby done shopping
11: Leo done shopping
13: Joe paid $10 for 2 items to checker 0
Segmentation fault

就像我说的，第 13 行是我想要发生的，但是将指向 Cust 对象的指针设置为等于 null 会给我 seg 错误。如果需要更多说明或者您想要 GDB 消息，我可以提供。任何输入表示赞赏，谢谢！

整个循环如下：

175    for(int i = 0; checker_queue.empty() != true && i < num_checkers; i++)
176    {
177      int checker_number = i;
178      if(checkers[i].m_cust == NULL && checkers[i].m_start_work == 0)
179      {
180       checkers[i].m_cust = checker_queue.dequeue();
181       checkers[i].m_cust->print_start_checkout(cout, clock, checker_number);
182
183      if(checkers[i].m_cust->get_status() == "shopper")
184       {
185        checkers[i].m_time_checkout = clock + (checkers[i].m_cust->get_num_items()*3);
186       }
187
188       else if(checkers[i].m_cust->get_status() == "robber")
189       {
190        checkers[i].m_time_checkout = clock + 7;
191       }
192       cout << "time checkout: " << checkers[i].m_time_checkout << endl; //check for checkout time
193      }
194
195      if(checkers[i].m_time_checkout == clock && checkers[i].m_cust->get_status() ==    "shopper" && checkers[i].m_cust != NULL)
196      {
197       int amount_paid = checkers[i].m_cust->get_num_items()*5;
198       checkers[i].m_money_in_register += amount_paid;
199       checkers[i].m_cust->print_done_checkout(cout, clock, amount_paid, i);
200       Cust *tmp = checkers[i].m_cust;
201       delete tmp;
202       num_custs--;
203       checkers[i].m_cust = NULL;
204      }
205
206      if(checkers[i].m_time_checkout == clock && checkers[i].m_cust->get_status() == "robber" && checkers[i].m_cust != NULL)
207      {
208       int checker_cash = checkers[i].m_money_in_register;
209       checkers[i].m_cust->print_done_stole(cout, clock, checker_cash, i);
210       checkers[i].m_money_in_register = 0;
211       Cust *tmp = checkers[i].m_cust;
212       delete tmp;
213       num_custs--;
214       checkers[i].m_cust = NULL;
215      }
216     }

如果我注释掉设置 checkers[i].m_cust = NULL，我会得到更多的输出，如下所示：

1: Joe entered store
Should be done at: 7
2: James entered store
Should be done at: 8
3: Bob entered store
Should be done at: 9
4: Abby entered store
Should be done at: 10
5: Leo entered store
Should be done at: 11
7: Joe done shopping
7: Joe started checkout with checker 0
time checkout: 13
8: James done shopping
8: James started checkout with checker 1
time checkout: 14
9: Bob done shopping
10: Abby done shopping
11: Leo done shopping
13: Joe paid $10 for 2 items to checker 0
14: James paid $10 for 2 items to checker 1
Segmentation fault
4

1 回答 1

4

我怀疑问题在于您正在测试优化的代码，因此崩溃发生位置的表示不正确。问题可能是您的 if 语句中的这段代码：

checkers[i].m_cust->get_status() == "robber" && checkers[i].m_cust != NULL

你有这个错误的方式; 在尝试调用 get_status() 之前，您需要检查 m_cust 是否为空

checkers[i].m_cust != NULL && checkers[i].m_cust->get_status() == "robber"

IE

if(checkers[i].m_time_checkout == clock && checkers[i].m_cust != NULL && checkers[i].m_cust->get_status() == "robber")

当给定两个逻辑 AND 条件时：

如果 ( a && b )

b只需要检查是否a为真。一旦 C 和 C++ 达到使其余测试变得不必要的条件（称为“短路评估”），它们就会停止评估条件。

如果 ( a || (b && c) )

如果a为真，则不会评估 b 和 c。只有当为假且为真c时才会被评估。ab

- 在旁边 - 

Cust *tmp = checkers[i].m_cust;
delete tmp;

我猜这是你“这里发生了什么”调查的一部分，但你应该没问题

delete checkers[i].m_cust;

只要对象 checkers[i].m_cust 是用“new”单独分配的。确保你没有做

Customers* customer = new Customer[someNumber];
...
checkers[i].m_cust = customer[x];
...
delete checkers[i].m_cust;  // << BAD

如果您的客户是这样分配数组的，请不要单独删除它们，使用

delete [] customers;

当你完成了他们。

于 2013-11-05T20:24:31.407 回答