Question

一切！我正在创建库存类。以下是我到目前为止实现文件的内容：

//aItem .cpp implementation file

#include "stdafx.h"
#include "aitem.h"
#include <iostream>
#include <string>
#include <vector>
using namespace std;

//setting this up default
aItem::aItem()
{
    vector<aItem> inventory;
    mitemName = "Default Name";
    mitemType = "Default Type";
    mdamage  = 9001;
}

void aItem::ItemList()
{
    for( vector<aItem>::size_type index = 0; index < inventory.size(); index++ ) 
        {
            //Makes a numerical list.
            cout << "Item " << index + 1 << ": " <<  inventory[index].mitemName << endl;
            index+= 1;
        }
}

void aItem::ItemAdd(string itemName, string itemType, int damage)
{
    mitemName = itemName;
    mitemType = itemType;
    mdamage = damage;

    inventory.push_back

ItemAdd（）中间的突然切入是我的问题。如果我没有从我知道名称的列表中删除它，如何将新创建的项目添加到我的库存矢量中？

Answer 1

首先，您应该使用m_name而不是mitemName，它更容易阅读，并且使用更广泛。

现在您的实际问题是，您有一个名为aItem的库存，其中包含一个项目的向量...这是您的问题，为项目创建一个类，为库存创建一个类（如果您需要）。他们不能在同一个班级。

所以在这里我创建了我的项目类（我将其命名为Item，如果您愿意，可以将其称为aItem），它仅控制项目的名称，类型和损坏。

#ifndef ITEM_H
#define ITEM_H

#include <string>

class Item {
    public:
        Item();
        virtual ~Item();
        virtual void setName(std::string name);
        virtual std::string getName();
        virtual void setType(std::string type); // consider enum for types here
        virtual std::string getType();
        virtual void setDamage(int damage);
        virtual int getDamage();

    private:
        std::string m_name;
        std::string m_type;
        int m_damage;
};

#endif // ITEM_H

我想您可以自己编写.cpp侧的代码，这是头文件。

这是我的Inventory.h，您可以重命名以符合您的喜好。

#ifndef INVENTORY_H
#define INVENTORY_H

#include "item.h"
#include <vector>

class Inventory {
    public:
        Inventory();
        virtual ~Inventory();
        virtual void addItem(Item* item);
        virtual void removeItem(int index); // You could implement removeItem(Item* item); too

    private:
        std::vector<Item*> m_items;
};

#endif // INVENTORY_H

现在你可以自由地为这些类中的任何一个实现任何方法，如果你还需要更多的东西。

Answer 2

如果您的库存不需要像其他人建议的那样是自己的类，则可以在引用需要时将其传递给item类。这可能不是最好的方法，但你不必写另一个类。

EG。

int main(){
    aItem sword("wooden","sword", 5);
    aItem axe("stone", "axe", 15);
    std::vector<aItem> inventory;
    sword.addTo(inventory);
    axe.addTo(inventory);
    ...

并在aItem.cpp中：

void aItem::add(std::vector<aItem> &inventory){
    inventory.push_back(*this);
}

或者，如果您想处理内存问题而不是范围问题，则可以使用指针：

int main(){
    aItem* sword = new aItem("wooden","sword", 5);
    aItem* axe = new aItem("stone", "axe", 15);
    std::vector<aItem*> inventory;
    sword->addTo(inventory);
    axe->addTo(inventory);
    ...

并在aItem.cpp中：

void aItem::add(std::vector<aItem*> &inventory){
    inventory.push_back(this);
}

但同样，在几乎所有情况下，正确处理范围或内存管理的库存类都会比这更好。

Answer 3

所以你有一个名为aItem的类，它应该是一个“项目”并存储库存？听起来你需要另一个类来维护包含项目向量的库存。

其次，您现在有一个堆栈变量作为您的库存。一旦你的构造函数超出范围就会消失。

最后，您需要创建一个临时“项目”以添加到广告资源。这就是被推回的东西;）

使用.push_back将新创建的项添加到库存中

3 个答案: