Non ho ancora letto bene la parte del pdf relativa alla partita e nemmeno la tua classe Partita. Avrò tempo forse stasera di darci una occhiata meglio.Originariamente inviata da PuntaT
Posso dirti alcune cose in generale e su Tessera/Giocatore. Innanzitutto tieni sempre i campi generalmente come "private", se non ci sono altre indicazioni o necessità ben precise.
Riguardo Tessera, è ok. Per il toString è richiesta la forma [a|b] ma tu hai messo la virgola in mezzo. Nulla di grave comunque.
Riguardo Giocatore:
- estrai (non Estrai) si potrebbe ampiamente migliorare.
- finito è sbagliato (hai sempre un oggetto ArrayList, non è mai null).
- anche toString è migliorabile ma questo è assolutamente il meno.
Ricorda che il remove(int index) restituisce l'elemento che viene eliminato, questo è utilissimo!
Rispondi quotando