samadio / SDM_exam

1 stars 0 forks source link

NameAlreadyUsedException superclass #33

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/players/NameAlreadyUsedException.java#L3

Estendere AttributeInUseException direi che è decisamente sbagliato, suddetta eccezione viene usata in un contesto ben specifico (da documentazione riguardano le interfacce Context e DirContext). Quindi direi di rimuovere tale classe come superclasse ed estendere semplicemente Exception.