Полная ошибка:
Ошибка 2 ошибка C2678: двоичный файл «==»: не найден оператор, который принимает левый операнд типа «Элемент» (или нет приемлемого преобразования) c: \ program files (x86) \ microsoft visual studio 12.0 \ vc \ включить \ алгоритм 1734 1 GameStore
Класс инвентаризации (файл cpp)
#include <iostream>
#include <string>
#include <vector>
#include <algorithm>
#include <Windows.h>
#include "item.h"
class Inventory
{public:
void Inventory::viewInventory()
{
for (int i = 0; i < inventory.size(); i++)
{
Inventory::inventory[i].getItemName();
}
}
void Inventory::addItem(Item& item)
{
inventory.push_back(item);
}void Inventory::removeItem(Item& item)
{
if (std::find(inventory.begin(), inventory.end(), item) != inventory.end())
{
inventory.erase(std::remove(inventory.begin(), inventory.end(), item), inventory.end());
}
else
{
std::cout << "Item does not exist" << std::endl;
}
}
//Player Gold methods
int Inventory::getGold()
{
return playerGold;
}
void Inventory::setGold(int newGold)
{
playerGold = newGold;
}int Inventory::addGold(int newGold)
{
playerGold += newGold;
return playerGold;
}
int Inventory::removeGold(int newGold)
{
playerGold -= newGold;
return playerGold;
}
//Player Gold methodsprivate:
std::vector<Item> inventory = {};
int playerGold;
};
По сути, я пытаюсь создать систему инвентаризации, в которой хранятся объекты из класса «Предмет». Я потратил много времени на то, чтобы заставить мои циклы for и все мои методы работать, а затем, поскольку небо никогда не было более ясным, я получаю ошибку, из которой выход из моей лиги.
Класс изделия (CPP)
#include "item.h"#include <iostream>
#include <string>Item::Item(int id, std::string name, std::string description, std::string examime)
{
itemName = name;
itemID = id;
itemDescription = description;
itemExamine = examime;
}
void Item::setItemName(std::string newName)
{
itemName = newName;
}
void Item::setItemDescription(std::string newDescription)
{
itemDescription = newDescription;
}
void Item::setItemExamine(std::string newExamine)
{
itemExamine = newExamine;
}
void Item::setItemID(int newID)
{
itemID = newID;
}
std::string Item::getItemName()
{
return itemName;
}
std::string Item::getItemDescription()
{
return itemDescription;
}
std::string Item::getItemExamine()
{
return itemExamine;
}
int Item::getItemID()
{
return itemID;
}
Класс товара (заголовок)
#include <iostream>
#include <string>
class Item
{
public:
//constructor
Item::Item(int id, std::string name, std::string description, std::string examime);
//setters
void setItemName(std::string newName);
void Item::setItemDescription(std::string newDescription);
void Item::setItemExamine(std::string newExamine);
void Item::setItemID(int newID);
//getters
std::string Item::getItemName();
std::string Item::getItemDescription();
std::string Item::getItemExamine();
int Item::getItemID();
private:
std::string itemName;
int itemID;
std::string itemDescription;
std::string itemExamine;
};
Если у вас есть какой-либо совет, даже если это будет переделать всю мою систему, это было бы здорово. Занятия, очевидно, очень просты, и я собираюсь добавить гораздо больше. Это означает, что я не ищу ответ типа «Вам даже не нужен класс предметов»
Спасибо за любую помощь!
Кажется, что основной проблемой вашего кода является отсутствие operator==
реализация для вашего Item
класс, как говорит ошибка компилятора.
Как новичок в C ++ (не беспокойтесь: каждый из нас был новичком), вы можете спросить Зачем это op==
требуется, как вы не называете это эксплицитно в вашем коде. Ну, это правда, что вы не вызываете это явно, но код в стандартной библиотеке реализация Является ли. В частности, вы вызываете std::find
(реализовано в<algorithm>
заголовок, который цитируется в вашем сообщении об ошибке):
void Inventory::removeItem(...) { if (std::find(inventory.begin(), inventory.end(), item) != inventory.end()) { ...
std::find
нужно сравнить Item
случаи с op==
,
Итак, чтобы попытаться исправить ваш код, пожалуйста, определите operator==
для сравнения Item
с, как это:
// In the Item class header:
class Item
{
...
};
inline bool operator==(const Item& a, const Item& b)
{
// Implement your comparison logic for Items a and b.
// ...
// Return true if "a == b".
}
В качестве дополнительных примечаний, когда вы объявляете функции-члены внутри вашего класса, вам не нужно использовать Item::
префикс перед функцией-членом:
class Item { public: //constructor Item::Item(int id, std::string name, std::string description, std::string examime); //setters void setItemName(std::string newName); void Item::setItemDescription(std::string newDescription); void Item::setItemExamine(std::string newExamine); void Item::setItemID(int newID); ...
Просто делать:
class Item
{
public:
//constructor
Item(int id, std::string name, std::string description, std::string examime);
//setters
void setItemName(std::string newName);
void setItemDescription(std::string newDescription);
void setItemExamine(std::string newExamine);
void setItemID(int newID);
...
Кроме того, рассмотрите возможность создания ваших добытчиков const
, чтобы сделать ваш класс const-правильным, например:
class Item
{
public:
...
std::string getItemName() const;
std::string getItemDescription() const;
std::string getItemExamine() const;
int getItemID() const;
...
};
Других решений пока нет …