0

我试图验证数组中的指针，所以我没有导致任何内存错误，

这个方法：

for(int i=0;i<array_size;i++) {
     if (array[i]!=NULL)
          array[i]->stuff();
     }
}

过去曾工作过。

现在，我必须做同样的事情，除了根据对象变量按顺序执行所有操作。

我的新方法是：

Direct2Entity* nextset[MAX_ENTS]; // ents[MAX_ENTS] is also a Direct2Entity* array
for(int i=0;i<MAX_ENTS;i++) {
    nextset[i]=NULL; // note that ents[] is also flushed before this
}
int nextsetid=0;
int maxn;
bool stillnull;
while(true) { // infinite sorting loop
    maxn=-1;
    stillnull=true;
    for(int i=0;i<next_put;i++) {
        if (ents[i]!=NULL) {
            stillnull=false;
            if (ents[i]->depth<0) { // make sure no infinite loops occur with negative depth
                ents[i]->depth=0;
            }
            if (ents[i]->depth>maxn) {
                nextset[nextsetid++]=ents[i];
                ents[i]=NULL; // make NULL to further loop
            }
        }
    }
    if (stillnull) break;
}
for(int i=0;i<next_put;i++) {
    if (nextset[i]!=NULL) {
        ents[i]=nextset[i]; // copy nextset[] to ents[]
    }
}
for(int i=0;i<next_put;i++) {
    if (ents[i]!=NULL) {
        if (ents[i]->getroom()==current_room) {
            ents[i]->draw(this); // ents[i] is still NULL... ?
        }
    }
}

在最后一个 for 循环中，ents[i] 被显式检查以确保它不会取消引用 NULL 指针。然而 C++ 却越过了它并调用了该函数。在各种随机位置都有各种运行时错误，但我几乎可以肯定这是来自这里的未定义行为。

4

1 回答 1

0

我没有看到确定 next_put 值的逻辑。它是否可能只是超过了 ents[] 的长度？如果是这样，即使您已正确初始化 ents[]，当您的循环离开数组的末尾时，该内存也不会被初始化（至少正如您所期望的那样）并且您的 if (ents[i]!=NULL ) 将被通过（然后你的程序应该崩溃）。

于 2013-02-25T04:54:55.813 回答