0

有一个映射，它将一个映射int到一个Test*。

所有Test*指针都在之前分配，然后分配给映射。然后，我正在输入 mapdelete的值并将它们设置为null.

之后，它检查 的有效性one，它应该是null。但是，one不是null。

#include <QString>
#include <QMap>
#include <QDebug>

class Test {
    QString name;
public:
    Test(const QString &name) : name(name) {}
    QString getName() const { return name; }
};

int main() {
    QMap<int, Test*> map;

    Test *one = new Test("one");
    Test *two = new Test("two");
    Test *three = new Test("three");

    map.insert(1, one);
    map.insert(2, two);
    map.insert(3, three);

    for (auto itr = map.begin(); itr != map.end(); itr++) {
        Test *x = *itr;
        if (x) {
            delete x;
            x = 0; // ** Sets null to the pointer ** //
        }
    }

    if (one) // ** Here one is not 0 ?! ** //
        qDebug() << one->getName() << endl; // ** And then here crashes ** //
}

我想，当我在循环中使用它们时，我错过了一些东西delete。

怎么能修好？

第二个问题是，分配delete的指针是否正确？

4

2 回答 2

4

在循环中，变量x是仅在循环内部的局部指针。当您将其设置为时，NULL您实际上并没有将任何其他指针设置为NULL.

您应该将通过取消引用迭代器返回的引用设置为NULL：

*itr = nullptr;

这将使映射中的指针变为NULL，但其他指针仍将指向现在已释放的内存区域。

当你有两个指针时，它看起来像这样：

+-----+
| 一个 | ---\
+-----+ | +---------------+
             >--> | 测试实例 |
+-----+ | +---------------+
| x | ---/
+-----+

如果您设置其中一个指针，它看起来像这样：

+-----+
| 一个 | ---\
+-----+ | +---------------+
             >--> | 测试实例 |
+-----+ +--------------+
| x |
+-----+

变量x是NULL，但变量one仍然指向对象。如果对象已被删除，那么取消引用该指针将导致未定义的行为。

于 2013-03-31T09:07:12.310 回答
2

删除所有内容的最简单方法是：

qDeleteAll(map);
于 2013-03-31T14:40:56.637 回答