Skip to content
This repository has been archived by the owner on Feb 1, 2018. It is now read-only.

Fase di attacco: le modifiche introdotte con la commit 65b16df #10

Closed
Fs00 opened this issue Jan 17, 2018 · 1 comment
Closed

Fase di attacco: le modifiche introdotte con la commit 65b16df #10

Fs00 opened this issue Jan 17, 2018 · 1 comment
Labels
discussion enhancement New feature or request

Comments

@Fs00
Copy link
Owner

Fs00 commented Jan 17, 2018

Con la commit sopra citata, ho modificato ciò che avviene in fase di attacco (per ora solo nel Contadino, ma ho già predisposto anche Personaggio per questo cambiamento).
Adesso, attacca(Personaggio nemico) chiama il metodo riceviColpo(IAttaccante, int danno), che si preoccupa di effettuare il contrattacco se ce n'è la possibilità e, nel caso questo fallisca, di mitigare il danno con la statistica difesa e chiamare modificaPuntiVita con il valore del danno già sistemato.

Note:

  • in quest'implementazione, ho considerato il contrattacco come esclusivo di chi implementa IAttaccante (da confermare in Il contrattacco: come lo implementiamo e chi lo può usare? #7)
  • @AlibabaSakura il valore ritornato dal tuo metodo attacca() doveva essere gestito dalla classe di gestione della partita? Non trovi che nel tuo caso appesantiremmo troppo la classe di gestione con compiti che potrebbero benissimo "sbrigarsi" i personaggi fra di loro?
@Fs00 Fs00 added enhancement New feature or request discussion labels Jan 17, 2018
@alegotta alegotta mentioned this issue Jan 17, 2018
19 tasks
@Fs00
Copy link
Owner Author

Fs00 commented Jan 18, 2018

Dato che nessuno si è manifestato contrario riguardo a questa modifica, chiudo l'issue.

@Fs00 Fs00 closed this as completed Jan 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
discussion enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant