c++ - 我希望程序打印数字 0 到 99，但是当我运行代码时，我看到 100 行 99

Question

我有一堂课foo：

class foo{
public:
        int i;
};

还有一个类bar：

class bar{
public:
        foo* foo_;
};

和类似的代码：

int i=0;
vector<bar*> fs;
while(i<100){
    foo f;
    f.i = i;
    bar* b = new bar;
    b->foo_ = &f;
    fs.push_back(b);
    i++;
}

我希望每个都bar->foo_指不同的foo，所以当我有：

 for(i=0;i<fs.size();i++){
     cout << (fs[i]->foo_->i) << "\n";
 }

我希望程序打印数字 0 到 99，但是当我运行代码时，我看到 100 行 99！

为什么所有指针都指向最后一个对象？

score 6 · Accepted Answer

从技术上讲，您foo f;都已经死了，当循环的当前迭代结束时，它们超出了范围。

你的指针都指向最后一个对象的原因是因为f在下一次迭代中分配的似乎总是覆盖前一个，但你也不能依赖这个。你在做什么会导致未定义的行为。

score 3 · Accepted Answer

您的代码正在获取并存储局部变量的地址f。

这是一个禁忌。

在每次迭代中，变量将被销毁，您的类将指向一个死对象。

该程序在执行代码时不会给出运行时错误，因为 C++ 的设计理念是程序员永远不会这样做，因此检查它是浪费时间。

当你犯这种错误时会发生什么是“未定义的行为”，在这种情况下可能发生的最糟糕的事情是程序似乎无论如何都可以正常工作（只是稍后再咬你）。

score 3 · Accepted Answer

在您的循环中，生命周期foo f太短，无法作为用户参考，您应该将其声明为指针

...
foo *f = new foo();
f->i = i;
bar* b = new bar;
b->foo_ = f;
...

3 回答 3