严重错误(一)

这几天写代码的时候遇见一个严重的错误，导致我思考很久没有想到原因，一开始代码

class MyClass
{
private:
std::vector<int> s;
public:
void insert(int n);
MyClass() :s()
{

}
};
void MyClass::insert(int n)
{
s.push_back(n);
}
int main() {

复制
std::vector<MyClass*> test;
for (int i = 0; i < 3; i++)
{
    MyClass my;
    my.insert(i);
    test.push_back(&my);
}

return 0;
}

我本来想每次向test添加不同的元素，但是每次都没有成功，问题在于 my 对象是在每次迭代的循环体内创建的。这意味着在每次迭代结束时，my 对象会超出其作用域并被销毁。因此，当你在 test 向量中存储 my 对象的地址时，实际上是存储了一个已被销毁的对象的地址，这会导致未定义行为。为了解决这个问题，你可以使用动态内存分配来创建 MyClass 对象，并在使用完后手动释放该内存。以下是修改后的代码示例：

#include <vector>

class MyClass
{
private:
    std::vector<int> s;

public:
    void insert(int n);
};

void MyClass::insert(int n)
{
    s.push_back(n);
}

int main()
{
    std::vector<MyClass*> test;
    for (int i = 0; i < 3; i++)
    {
        MyClass* my = new MyClass();  // 使用动态内存分配创建 MyClass 对象
        my->insert(i);
        test.push_back(my);
    }

    // 使用完后释放动态分配的内存
    for (MyClass* my : test)
    {
        delete my;
    }

    return 0;
}