Yea makes sense, I use structs for a fair few aspects of my game, for instance, weapon data here:
https://github.com/belven/Mech_RPG/blob/master/Mech_RPG/Weapon.h
Mostly because I can then use inheritance in the stuct to add data for stuff like:
https://github.com/belven/Mech_RPG/blob/master/Mech_RPG/MagazineWeapon.h
An inventory could be a stuct but it would only be used for “Items”, stuff you would actually carry with you, although I do think of a quest as a to do list, I don’t think an inventory should manage it. Also because not all inventories will be used by a player, like chests etc.
I’m very concerned that some simple UI is causing the game to crash though, I don’t see how an object that basically stores a string and x objects that store, at most, a string a 2 other properties plus a few methods would do this. The game runs at less than 1 GB RAM and I’ve stress tested it with over 100 AI…