GameManager looks too oversized, how about splitting it

... and maybe it even smokes 😉

these look suspicious

Line 59 in 8e2a963

void fillWithData ( const GameManager & data ) ;

Line 434 in 8e2a963

void PlayerItem::fillWithData( const GameManager & data )

GameManager as an argument, to copy the character-related data, and it's inside PlayerItem, hmmm ..... 🤔

it is also possible that this manager sometimes plays solitaire instead of managing

doesn't this

Line 199 in 8e2a963

unsigned short howManyDoughnuts ;

Line 146 in 8e2a963

unsigned short getDoughnuts () const { return this->howManyDoughnuts ; }

duplicate that

Line 423 in 8e2a963

unsigned short getDonuts ( const std::string& player ) const ;

Line 376 in 8e2a963

unsigned short donuts ;

and so on ?

well, now it looks okay

Line 127 in 2f70075

    
            * Manages the game, and nothing more. Doesn't play solitaire. Doesn't smoke. Anything at all. Really.