0

我有一种情况，我有一堆类对象，我需要随着时间的推移更新其成员变量。我需要拥有的对象的数量可以增加和减少，并且在我的整个程序中都会迅速增加和减少。因为我需要一个可调整大小的类对象数组，所以我选择使用 std::vector。问题是，我当前的代码在执行大约一分钟左右后崩溃（我假设内存泄漏或其他原因，但我不确定）。这是我编写的一个示例程序，用于演示我在做什么：

#include <Windows.h>
#include <iostream>
#include <vector> 


char* names[] = {"Larry", "Bob", "xXx_Quicksc0p3zl33t_xXx", "InsertUnoriginalNameHere", "Idunno"};

class CEnt
{
public:
    const char* name;
    int         health;
};

std::vector<CEnt>   entities;

int main()
{
    while (1)
    {
        int iEntCount = rand() % 1000 + 1;  //Generate random value from 1000 to 2000.  This simulates the changing ingame entity count that I grab

        if (entities.size() != iEntCount)
        {
            entities.resize(iEntCount);
        }

        //Print.ToConsole(TYPE_NOTIFY, "%i", iEntCount);

        for (int iIndex = 0; iIndex < iEntCount; iIndex++)
        {
            CEnt& Ent = entities[iIndex];
            Ent.health =    rand() % 100 + 1;   //Generate random value to fill the objects.  This simulates when I grab values from ingame entities and put them in each object
            Ent.name =      names[rand() % 5 + 1];

            printf("Index: %i    Name: %s  Health: %i\n", iIndex, entities[iIndex].name, entities[iIndex].health);
        }
    }
}

它看起来很草率，但它证明了我在做什么。有没有更好的方法来实现这一目标？我需要在代码中的随机点访问一个容器，该容器包含向量中每个对象的最后更新变量。

4

1 回答 1

2

看起来可疑的一件事是

        Ent.name =      names[rand() % 5 + 1];

这将在 1..5 范围内选择一个值。但最高有效名称是names[4]，它将从数组的末尾读取。

我希望这会使它立即崩溃或根本不崩溃，但有可能那里有一些其他变量会发生变化并最终成为无效指针。

一种稍微好一点的写法是

const int n_names = sizeof(names)/sizeof(*names);

....

    Ent.name =      names[rand() % n_names];

虽然更好的风格可能是将名称本身放在向量中，等等。例如，参见这个问题及其许多骗局

于 2013-06-12T04:35:37.287 回答