class goods {
vector< pair< map<string,string>, vector<int> > > good;
public:
virtual ~goods() = 0;
void setGood(const string& manufacturer,
const string& g, int day, int month, int year) {
map <string, string> tmp;
tmp.insert(make_pair(manufacturer, g));
good.push_back(make_pair(tmp, vector<int>{day, month, year}));
}
};
Перечислю ваши ошибки:
virtual ~goods() = 0 {};
это чистый виртуальный деструктор, у которого не может быть тела (даже пустого). Так что уберите скобки.
tmp.insert(manufacturer, good);
std::map<string, string>
содержит пары строк, а не две строки
Что касается архитектуры:
Если вы хотите в вектор ввести пару таблицы и даты, то вам нужен вектор пар таблиц и вектор с целыми типами, а не вектор пар.
Вообше то, мне думается что вам нужно хранить в классе
vector< pair< pair<string,string>, vector<int> > >
поскольку вы храните там одну пару с датой.
P.S. для читабельности и для того, чтобы избегать случайных ошибок, лучше избегать повторение имен. Или в аргументах или в приватном поле изменить имя гораздо проще, чем потом следить что есть что. Вообше то приватный член может иметь любое самое короткое имя(клиету класса она всеравно не нужна)
virtual ~goods() = 0 {}
- уже белиберда. Нельзя в одну строчку объявить функцию как= 0
и тут же сделать определение. Определение должно делаться отдельно за пределами класса. Во-вторых,make_pair("key", "data")
не является приемлемым инициализатором дляstd::map
. Можно:this->good.push_back({ { { "key", "data" } } , { { 1, 2 } } });
. В-третьих,tmp.insert(manufacturer, good);
- бессмыслица. Уstd::map
нет такого методаinsert
.tmp.insert({ manufacturer, good });
– AnT 16 июл '18 в 6:11virtual ~goods() = 0 {}
- вижуал студия так позволяет. – KoVadim 16 июл '18 в 11:59