I was wondering if it would be easier for the council class to have API to add and remove players, so that the internal data structure isn't exposed. Or maybe something a bit more abstract, such as:
AddCouncilor(string player);
PromoteCouncilor(string player); // can throw errors if that player isn't a councilor
RemoveMember(string player); // removes player from either senator or councilor list
I was wondering if it would be easier for the council class to have API to add and remove players, so that the internal data structure isn't exposed. Or maybe something a bit more abstract, such as:
_Originally posted by @BryanFriestad in https://github.com/TankGameOrg/engine/pull/39#discussion_r1590420304_