Inconguenze nella classe Personaggio

[Fs00]

1. Perché abbiamo i setter e i getter se i campi sono protected? I campi dovrebbero essere private, a mio avviso
2. modificaHP() deve essere un override di subisciColpo(), altrimenti a cosa serve questo metodo nella classe base?
3. setPuntiStamina() avrebbe più senso come decrementaStamina(), un metodo che dovrebbe essere chiamato dalla classe di gestione a ogni turno

[Fs00]

Punti 1 e 2 sistemati con la commit 603472e, ditemi cosa ne pensate riguardo al punto 3.

[alegotta]

Ma sbaglio o la stamina può anche essere recuperata? In tal caso forse si potrebbero predisporre due metodi:

• decrementaStamina() chiamato ad ogni turno, il quale diminuisce di uno l'attributo puntiStamina
• ripristinaStamina() chiamato a fine scontro, il quale riporta la stamina al valore puntiStaminaTotali

[Fs00]

Sulla questione del recupero della stamina non abbiamo mai detto nulla. In caso dovremmo decidere se implementarlo o meno. Comunque, a cosa dovrebbe servire ripristinaStamina() se poi tanto la classe di gestione si riazzera alla fine di ogni partita?

[Fs00]

decrementaStamina() è stato aggiunto con la commit 826ba7a, dato che mi serviva per poter gestire la partita. Chiudo dato che per questa discussione non c'è altro da dire.