samadio / SDM_exam

1 stars 0 forks source link

newPlayer() throws NameAlreadyUsedException #34

Closed federico-camerota closed 4 years ago

federico-camerota commented 4 years ago

https://github.com/samadio/SDM_exam/blob/e829ee338a5b2dc93d657c29d1ca0bcfa05e77b8/src/main/java/gamesuite/game/GameSetter.java#L77

Il fatto che il metodo che non prende come argomento un nome da utilizzare per il player lanci un' eccezione mi sembra un pò assurdo e porta a codice piuttosto astruso e poco sensato come quello nel riferimento. Direi di risolvere il problema non accettando numeri come nomi validi in modo tale che il metodo newPlayer() possa creare un nuovo player con nome un numero senza doversi porre alcun tipo di problema.