这是main/Initializer函数的简化版本。当我调用它时,它必须向玩家的发明者添加任何物品,我得到一个Debug Assertation Failed错误。
我觉得我好像把范围弄混了?
我是否在函数的作用域内声明了一些新的东西,然后无法在main中再次访问它?
我在函数内尝试了一些东西,比如使用getter/setter而不是完全赋值,比如p_player = p
,但我认为这实际上根本没有解决问题,我有点困惑。
int main()
{
Array<Item> items(3);
string itemsfilename = "itemsfile.txt";
Initializer::InitializeItems(items, itemsfilename);
Login login;
Player p1;
string filename = login.LoginToGame();
Initializer::InitializePlayer(p1, rooms, items, 3, filename);
}
void Initializer::InitializePlayer(Player& p_player, HashTable<string, Room>& p_rooms, Array<Item>& p_items, int p_numItems, std::string& p_filename)
{
ifstream playerfile(p_filename);
int inventorycount = 0;
//all the stuff needed to make a player
std::string name;
int health;
int confidence;
int humor;
int speed;
std::string room;
Room* currentRoom;
Inventory inventory(100);
//reading in values from file
for(int i = 0; i < inventorycount; i++)
{
playerfile.getline(value, 256);
std::string item(value);
for(int j = 0; j < p_numItems; j++)
{
if(p_items[j].GetName() == item)
{
inventory.AddItem(&(p_items[j])); //This line taken out, removes the error.
}
}
}
Player p(name, health, confidence, humor, speed, currentRoom, inventory);
p_player = p;
}
AddItem()接受一个指向一个项目的指针,然后将它附加到它的DLinkedList中。
编辑:
我得到的错误是
调试断言失败!
计划:打鼾声文件f: dd/vctools/crt_bld/self_x86/crt/src/dbgdel.cpp线:52
表达式:_Block_TYPE_IS_VALID(pHead->nBlockUse)
AddItem() Code:
bool AddItem(Item* p_item)
{
if(p_item->GetWeight() + m_weight <= m_maxWeight)
{
m_inventory.Append(p_item);
m_weight += p_item->GetWeight();
}
else
{
return false;
}
return true;
}
好的,所以我们仍然没有真正导致问题的代码,但我很确定我知道发生了什么,并且为了避免陷入"添加更多代码的20个问题"-有两种可能的情况:
- Items是一个对象数组,您可以在
m_inventory
容器中存储指向它们的指针。当销毁这个容器时,通过在项目上调用delete
来销毁对象-这不起作用,因为内容不是从堆中分配的。 - 当您复制
inventory
时,m_inventory
容器没有适当地复制,并且由于指向存储的指针失败,内容崩溃。
如果这没有帮助,那么请尝试减少你的代码到一些只显示这个问题的东西,不使用我们不知道内容的文件,可以作为一个完整的程序在问题中发布所有必要的代码[删除任何其他不需要的代码],所以我们可以看到一切。目前,我们只看到了一小部分代码,问题几乎可以肯定是直接在您展示给我们的代码中。