c++ - 在构造函数中调用列表上的迭代器会更改其最终值吗？

Question

我是 C++ 新手，但不是编程新手，我遇到了一个非常令人困惑的问题。我有一个类，它在其构造函数中创建了一个列表。

如您所见，我以两种不同的方式打印出列表的最终值，这两种方式通常彼此一致:一种使用 list::end，另一种使用 列表::返回。然后我在主函数中调用此类的构造函数，访问创建的列表，并尝试打印最终值。示例代码如下。

#include <iostream>
#include <fstream>
#include <sstream>
#include <string>
#include <typeinfo>
#include <list>
#include <algorithm>
#include <queue>
using namespace std;
class Process{
public:
    Process(int CB);  
    int CB;
};

Process::Process(int c){
    CB = c;  
}

class Event{
public:
    Event(Process *process);
    Process *process;   
};

Event::Event(Process *ps){
    process = ps;
}

typedef list<Event> EventList;

class DES{
public:
    DES(string originFile);
    EventList events;
};



DES::DES(string originFile){
    ifstream infile (originFile.c_str());
    string str;
    while (getline(infile, str)) {
        // output the line
        //cout << str << endl;
        istringstream iss(str);

        int AT,TC,CB,IO;
        if (!(iss >> AT >> TC>>CB>>IO)) { 
            cout<<"breaking out of while loop \n"; 
            break;
        } 
          Process p(CB);
          Event evt(&p);
        this->events.push_back(evt);

    }

  int cb =  this->events.back().process->CB;
EventList::iterator inserter2 = this->events.begin();
EventList::iterator inserter3 = this->events.end();

//inserter3--;

//cout<<"CB after while loop using List<>::end(): " <<inserter3->process->CB<<endl;
 //cout<<"CB after while loop using LIST<>::back "<<cb<<endl;

    infile.close();

}


int main (int argc, char* argv[]) {
string inputFileName = argv[1];


DES des(argv[1]);
EventList::iterator b = des.events.end();
b--;
cout<<"CB at back of list in main: "<<b->process->CB<<endl;

return 0;
}

这就是我感到困惑的地方。 main 中的打印语句应该与构造函数中打印语句的输出相匹配，因为它们都只是打印列表最后一个元素的字段 ->process->CB。但是，出于某种原因，这仅在我取消注释 //EventList::iterator inserter2 = this->events.begin(); 我的构造函数中的行时才有效。同样，如果我保留该行并注释掉 EventList::iterator inserter3 = this->events.end(); 行，它也不起作用。只有当我在列表的结尾和开头都构造一个迭代器时，正确的值才会在 main 中打印出来。

谁能解释一下这种奇怪的行为？由于我对 C++ 不熟悉，我知道这一定是一些简单的误解，但我不得不承认这种行为对我来说似乎有点不自然。

编辑:这是注释掉构造函数中的迭代器之一的输出:

CB after while loop using List<>::end(): 10
CB after while loop using LIST<>::back 10
CB at back of list in main: 306496

这是构造函数中两个迭代器的输出:

CB after while loop using List<>::end(): 10
CB after while loop using LIST<>::back 10
CB at back of list in main: 10

-保罗

Answer 1

您正在使用/存储本地地址:

      Process p(AT,TC,CB,IO);
      Event evt(AT,&p,CREATED,READY);

一旦声明 p 的代码块退出，对该指针的任何引用都会导致未定义的行为。

---

由于 Process 包含一个简单的 int，您可以只存储 Process 的拷贝，而不是使用指针。

class Process{
public:
    Process(int CB);  
    int CB;
};

class Event{
public:
    Event(const Process& process); 
    Process process;   
};

Event::Event(const Process& ps) : process(ps) {}

那么第一段代码应该是这样的:

      Process p(CB);
      Event evt(p);

或者简单地说:

      Event evt(Process(CB));

这至少应该让您摆脱不一致的结果。

---

如果您确实需要指针，请考虑使用智能指针，例如 std::unique_ptr，或者如果认为有必要，请使用 std::shared_ptr 而不是使用原始指针。