Question

所以，似乎很多C ++新手都有向量问题，因为他们对向量有很多疑问。我对C ++并不陌生，但我从未遇到过这个问题。无论如何，所以我的问题是矢量指针。我正在制作一个带有矢量的视频游戏。它包含对象的引用，因此它可以在碰撞函数内进行碰撞响应。要加载我以前做过的向量：

Object a = Object(12, 135, 123, 124);
collision_rects.push_back(&a);

它完全像这样，但是我不想输入每个单独的对象，所以我创建了一个从文本文件中读取它的函数，就像这样。

void get_from_file()
{ 
//Blah blah read numbers from file.
//then I would get the numbers and make a object from them.
Object a = Object(numbers from file);
collision_rects.push_back(&a);
}

但这样做不起作用！我摔倒在地！如果我采用相同的数据并以第一种方式加载它可以正常工作，但是当我从文本文件中的函数执行它时它不会。然后我意识到可能是因为对象超出了范围。所以我尝试将一个对象设为全局，然后在函数中执行相同的操作。这与第一个选项基本相同，但在函数中。所以我在相同的范围内制作了两个向量，这样原始对象就不会超出范围了。

std::vector<Object> objects;
void get_from_file()
{
//blah blah
objects[i] = Object(nums from file); //reading from numbers in a for loop
collision_rects.push_back(&objects[i]); //This throws weird

这也不起作用。（顺便说一下，这甚至不接近实际代码，只是代码基本上做的一个例子。我究竟做错了什么？谢谢。的修改所以我不知道我是否完美地解释了它，但我不认为它指向一个已删除对象的问题。这当然会使它失效，但这也是我相信的另一个原因。

player->cHandler.objects.push_back(&uno); player->cHandler.objects.push_back(&dos); player->cHandler.objects.push_back(&tres); player->cHandler.objects.push_back(&quatro); player->cHandler.objects.push_back(&cinco); player->cHandler.objects.push_back(&seis); player->cHandler.objects.push_back(&a8); player->cHandler.objects.push_back(&a9); player->cHandler.objects.push_back(&a10); player->cHandler.objects.push_back(&a11); player->cHandler.objects.push_back(&a12); player->cHandler.objects.push_back(&a13); player->cHandler.objects.push_back(&a14); player->cHandler.objects.push_back(&a15); player->cHandler.objects.push_back(&a16); player->cHandler.objects.push_back(&a17); player->cHandler.objects.push_back(&a18); player->cHandler.objects.push_back(&a19); player->cHandler.objects.push_back(&a20); 现在这是我的实际代码。但是，当我这样做时，它不起作用。

objs.push_back(uno); objs.push_back(dos); objs.push_back(tres); objs.push_back(quatro); objs.push_back(cinco); objs.push_back(seis); objs.push_back(a8); objs.push_back(a9); objs.push_back(a10); objs.push_back(a11); objs.push_back(a12); objs.push_back(a13); objs.push_back(a14); objs.push_back(a15); objs.push_back(a16); objs.push_back(a17); objs.push_back(a18); objs.push_back(a19); objs.push_back(a20); for (int i = 0; i < objs.size(); i++) { //player->cHandler.objects.push_back(&objs[i]); } 这不起作用，我不知道为什么。

Answer 1

您的问题与std::vector无关。您只使用vector发现了潜在的问题，但是如果您返回指向您创建的对象的指针，或者如果您有一个普通数组并且存储了指向数组中对象的指针，则会遇到同样的问题

您正在存储指向本地变量的指针：

void get_from_file()
{ 
    Object a = Object(numbers from file);
    collision_rects.push_back(&a);
}

当get_from_file()函数返回时，a将被销毁，因为它是本地对象。因此，您的collision_rects向量将具有指向不再存在的变量的指针。

此代码会出现同样的问题：

Object* get_from_file()
{ 
    Object a = Object(numbers from file);
    return &a;  // returning pointer to local variable.  Bad.
}

因此，解决方案是确保对象的生命周期存在于get_from_file函数之外。

您可以在向量中存储对象（std::vector<Object>），或者如果它必须是指针，则它应该是指向具有超出get_from_file()函数的生命周期的对象的指针。这可以通过使用new从堆分配，或者更好，使用智能指针类型来完成。

Answer 2

你正在存储一个指向对象的指针，但是这些对象正在被摧毁，所以你留下了一个指向任何东西的指针。

一个好的选择是将对象本身存储在向量而不是指针中。

或者你可以在堆上分配一个对象（使用例如new Object(1, 2, 3)）并存储该指针（这样对象就不会被破坏，直到你希望它为止）。如果您这样做，我建议您存储shared_ptr或unique_ptr - 他们会为您完成大量工作，并且可以更轻松地编写正确的代码。

Answer 3

如果您std::vector中的对象在std::vector失效时被销毁，可以将其存储在std::vector<Object>中，并使用emplace_back直接在std::vector<Object> collision_rects; collision_rects.emplace_back(nums from file);中构建它们。向量，不需要临时局部变量：

{{1}}

C ++矢量指针问题/指针矢量向对象中的向量中

3 个答案: