DariodAbate / ingsw2022-AM57

Eryantis board game
MIT License
3 stars 2 forks source link

Use more constant. #10

Closed cerutigiacomo closed 2 years ago

cerutigiacomo commented 2 years ago

Also is not a good idea to duplicate classes just for some constraint. Use a single class and parameters (maybe loaded from resources) am57 am57-2

DariodAbate commented 2 years ago

Salve, questa implementazione è legata al factory pattern che abbiamo deciso di implementare a seguito della peer review e che ci è anche stata suggerita dal prof. Margara. Non ho ben capito perché eliminare le due classi che contengono le costanti del gioco: abbiamo sfruttato questo pattern proprio per poter scollegare le costanti dalle altre classi e farci ritornare un oggetto che le contenesse tutte in base al numero di giocatori, senza dover usare degli if- else

cerutigiacomo commented 2 years ago

L’issue riguarda l’uso di valori scritti direttamente in del codice, é buona pratica renderli parametri. Che possono essere fissi o caricati da risorse esterne.

Lerrylore commented 2 years ago

Abbiamo messo dei parametri fissi nelle classi