在向量中对指针使用删除

Question

我是一个业余编码员，正在尝试通过在控制台上制作文本RPG游戏来学习。 我只是想考虑一下现在如何设计我的代码，而我陷入了项目和库存类如何协同工作的困境。 我有这样定义的库存类：

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创建物品，然后将其转移到玩家的物品栏中。 我的问题是，我应该只是学习智能指针并使用它们，还是我所说的删除方法还可以还是完全愚蠢？ 还是我应该以不同的方式考虑这种代码结构？ 如果您愿意，感谢您的回答！ 我真的很感激。

Answer 1

正确的方法是使用迭代器。 不要在其中传递“ Item”的指针

void AddItem（Item * item）方法。

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

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

PS使用此构造函数-> Inventory（）{items.resize（10，nullptr）;}