删除关键字 c++

Question

我有一个有 2 个布尔值和一个指针数组的类，我在堆上分配。问题是当它调用析构函数时它给了我一个错误，可能是因为它删除了太多，我看到它试图访问 0xdddddd并向我展示这个“抛出异常：读取访问冲突。 这是 0xDEEEDEEF。”

1.如何更好地使用“删除”，是不是因为操作符重载？

2.还说我没有初始化“QuadTree::childs”，为什么？

class QuadTree {
public:
QuadTree* childs[4];
    bool info;
    bool parent;

QuadTree() {
    for (int i = 0; i < 4; ++i) {
         childs[i]=NULL;
    }
    info = false;
    parent = false;
}


~QuadTree() {
    for (int i = 0; i < 4; ++i) {
            delete childs[i];
    }
}
    QuadTree& operator=(const QuadTree& tree) {

    for (int i = 0; i < 4; ++i) {
        childs[i] = new QuadTree;
        if (tree.childs[i]->parent == 1) {
            childs[i] = tree.childs[i];
        }
        childs[i]->info = tree.childs[i]->info;
        childs[i]->parent = tree.childs[i]->parent;
    }
    return *this;
 }
}

所以这是添加两棵树的代码。我创建了重载运算符的下一个原因，如果一棵树的节点是白色的，另一个是父节点，我只想复制父节点。

void addTrees(const QuadTree& tree1, const QuadTree& tree2, QuadTree& end) {

if (tree1.info == 1 || tree2.info == 1) {
    end.info = 1;
    return;
}
else if (tree1.parent == 1 && tree2.parent == 1) {
    end.parent = 1;
    for (int i = 0; i < 4; ++i) {
        end.childs[i] = new QuadTree;
        addTrees(*tree1.childs[i], *tree2.childs[i], *end.childs[i]);
    }


}
else if (tree1.parent == 1) {
    end.parent = 1;
    end = tree1;

}
else if (tree2.parent == 1) {
    end.parent = 1;
    end = tree2;

}
else {
    end.info = 0;
}


}

Answer 1

childs[i] = tree.childs[i]; 行没有做你认为它正在做的事情。

您现在正在引用他们分配的内存，而不再引用您分配的内存。谁试图在第二次删除这段记忆，谁就会有一个糟糕的时间。

如果您想将他们的孩子复制到您最近分配的孩子中，您将需要取消引用指针以对对象本身进行操作。 *childs[i] = *tree.childs[i]

Answer 2

问题出在你的赋值运算符上。

    QuadTree& operator=(const QuadTree& tree) {

        for (int i = 0; i < 4; ++i) {
            childs[i] = new QuadTree;
            if (tree.childs[i]->parent == 1) {
                childs[i] = tree.childs[i];
            }
            childs[i]->info = tree.childs[i]->info;
            childs[i]->parent = tree.childs[i]->parent;
        }
        return *this;
    }
};

在这些行中：

if (tree.childs[i]->parent == 1) {
    childs[i]->info = tree.childs[i]->info;
            childs[i]->parent = tree.childs[i]->parent;

childs[i] 可能是 nullptr，它可以用于分配，但不能用于取消引用。

而使用->parent，您确实可以取消引用nullptr

请在解除引用前检查nullptr。