c++ - 删除另一个类的方法中的多个指针

Question

我正在编辑一个开源游戏的一些代码，通常代码不会直接访问玩家或生物类；然而，它的参数 Cylinder 在所有方面都处于食物链的顶端。

我的问题是我应该删除所有这些指针还是在完成它们后将它们设置为 NULL？

这是我写的代码；它工作正常，但我不想让服务器因悬垂指针(对 C++ 来说仍然有点新)等问题而崩溃。

bool Game::removeMoney(Cylinder* cylinder, uint64_t money, uint32_t flags /*= 0*/)
{
    if (cylinder == nullptr) {
        return false;
    }

    if (money == 0) {
        return true;
    }

    if (Creature *creature = cylinder->getCreature()) {
        if (Player *player = creature->getPlayer()) {
            uint64_t cash = player->getBankBalance();
            if (cash < money) {
                return false;
            }
            player->setBankBalance(cash - money);
        }
    }
    return true;
}

void Game::addMoney(Cylinder* cylinder, uint64_t money, uint32_t flags /*= 0*/)
{
    if (Creature *creature = cylinder->getCreature()) {
        if (Player *player = creature->getPlayer()) {
                player->setBankBalance(player->getBankBalance() + money);
        }
    }
}

Answer 1

一般来说(除非文档另有说明)，不要 delete对象，如果你传递了一个指针。假设您没有被授予该对象的所有权。

现代 C++ 可帮助您避免需要知道您是否被授予所有权:您可能会获得 std::shared_ptr<Cylinder>或 std::unique_ptr<Cylinder> - 无论哪种方式，当智能指针超出范围时，都会为您处理删除。但通常情况下，您不得不与无法给您这种保证的图书馆合作。

无需清空在小范围内使用的任何指针(例如函数)。如果您将指针变量保留更长时间(也许在成员变量中)，那么这样做可能有助于防止事故发生。由于 C++ 不是垃圾收集语言，因此将即将超出范围的空指针置零没有任何好处。