Nì ... alcune cose non le hai proprio seguite ...
Capisco che sei all'inizio ma innanzitutto:
- metti i campi di istanza private.
- metti i metodi "accessori" getter/setter public.
- metti i costruttori public
(se non ci sono altre esigenze ben più precise .... in Mazzo e Giocatore non hai fatto nulla di tutto questo!!)
Poi hai usato un HashSet. Di per sé HashSet non garantisce "casualità". In altre parole, HashSet non è certo fatto per fare da generatore di dati "pseudo" casuali.
Da quanto posso capire, devi ordinare secondo l'ordinamento di Briscola/Scopone (che sono dei comparator), le N carte estratte dal giocatore.
Bene:
- l'oggetto Giocatore ce l'hai
- le carte pescate dal Giocatore le hai (sono in Giocatore, nella lista che devi passare a sort)
- il comparatore (Briscola/Scopone) ce l'hai, che devi passarlo al sort
Quindi quale è il dubbio?