Question

我有一种情况，我有一堆类对象，其成员变量我需要随时更新。我需要拥有的对象数量可以增加和减少，并且在我的程序中快速完成。因为我需要一个可调整大小的类对象数组，所以我选择使用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);
        }
    }
}

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

Answer 1

看起来可疑的一件事是

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

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

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

写一个更好的方法是

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

....

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

虽然更好的风格可能是将名称本身放在矢量等中。例如参见this question及其许多欺骗

矢量的改变类对象的数量

1 个答案: