使用.push_back將新創建的物品添加到庫存中

Question

all！我正在創建庫存類。以下是目前爲止的實現文件：

//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的庫存，其中包含aItems的向量...這是您的問題，創建物品的類和庫存的類（如果需要的話）。他們不能在同一班上。

所以在這裏我創建了我的物品類（我會稱之爲物品，如果您願意，可以調用您的物品），它只控制物品的名稱，類型和傷害。

#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

所以你有一個叫做aItem的類，它應該是一個「物品」，也存儲庫存？聽起來就像你需要另一個類來維護包含項目向量的庫存。

其次，您現在有一個堆棧變量作爲您的清單。只要你的構造函數超出範圍，它就消失了。

最後，您將需要創建一個臨時「項目」以添加到清單。這是什麼被推回;）

Answer 3

如果您的庫存不需要像其他人所建議的那樣是自己的類，則可以在需要時通過引用將其傳遞給物品類。這可能不是最好的辦法，但你不必寫另一個班級。

例如，

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

和aItem.cpp：

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：

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

或者，你可以，如果你想處理內存問題，而不是範圍的問題用指針來代替.cpp：

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

但是，在幾乎所有情況下，正確處理範圍界定或內存管理的庫存類都比這更好。