对向量中的指针使用删除

Question

我是一名业余程序员，正在通过尝试在控制台上制作文本角色扮演游戏来学习。我现在只是想考虑如何设计我的代码，而我却被困在物品和库存类如何协同工作上。我有这样定义的库存类：

class Inventory {
private:
    vector<Item*> items;

public:
    Inventory() {
        items.resize(10);
        for (int i = 0; i < 10; ++i) {
            items[i] = nullptr;//so I can still display an empty inventory
        }
    };
    ~Inventory() {
        for (int i = 0; i < 10; ++i) {
            delete items[i];
        }
    }

    void AddItem(Item* item) {
        for (int i = 0; i < items.size(); ++i) {
            if (items[i] == nullptr) {//if the slot is empty
                items[i] = item;
                break;
            }
        }
    }

    void Show() const {
        for (int i = 0; i < items.size(); ++i) {
            cout << "Slot " << i + 1 << ": ";
            if (items[i] == nullptr) {
                cout << "<Empty Slot>" << endl;
            } else {
                cout << "<" << items[i]->GetName() << ">" << endl;
            }
        }
    }
};

最终我打算让敌人在死后掉落战利品，所以我需要使用new 创建物品，然后将它们转移到玩家的库存中。我的问题是，我应该只了解智能指针并使用它们，还是我在这里调用 delete 的方式可以还是完全愚蠢？还是我应该以不同的方式考虑这个代码结构？谢谢你的回答，如果你这样做了！我真的很感激。

Answer 1

正确的方法是使用迭代器。不要在

中传递'Item'的指针

void AddItem(Item* item) 方法。

您应该创建一个“Item”类的迭代器并将该迭代器传递给“AddItem”方法。类似地，创建另一个方法“RemoveItem”，该方法返回一个“Item”的迭代器，该迭代器可以发送到 Player 的“CollectMoney”方法。

如果您不想使用迭代器，您也可以使用“Item * item”，但您必须调用方法“RemoveItem”来获取“Item”的引用，以便将其传递给 Player。由于项目不是在类中创建的，因此不应在类中删除它们，因此请删除析构函数。

PS 使用这个构造函数 -> Inventory() {items.resize(10,nullptr);}