search

udg-propro-spring-2020 / projecte-2020-a3

projecte-2020-a3 created by GitHub Classroom
0 stars 1 forks source link

Control de torns i moviments #11

Closed DavidNagisa333 closed 4 years ago

DavidNagisa333 commented 4 years ago

Bones!

Com hem anat parlant nois, ara mateix es pot jugar de forma correcte. S'ha implementat juntament amb en Miquel D. un control dels torns i de les peces utilitzant un enum que va proposar i construir. He hagut d'actualitzar lleugerament la classe ConsoleGame (mètode readMovements) per validar la jugada escollida, afegir algun metode i atribut auxiliar a Piece i he modificat la classe Chess, afegint nous mètodes i actualitzant d'altres (per exemple s'han construit arrays auxiliars per decidir quins moviments de la Piece s'han de mirar a l'hora de fer un moviment, s'ha creat un HashMap per guardar <Position,Piece> per construir el tauler i es te un control del color de la Piece).

Devant de dubtes anem parlant.

miquelbofill commented 4 years ago

Abans de divendres em miraré el codi i si hi veig coses a millorar us ho diré.

miquelbofill commented 4 years ago

@DavidNagisa333 i @miqueldedo, sembla que heu continuat implementant junts. Ja vam comentar que no podíeu fer cap classe entre els dos. Espero que subsaneu aquest aspecte.

Comentaris menors:

  1. A la classe Chess el mètode createInitialPositions té codi (gairebé) repetit; aquesta repetició és evitable parametritzant el codi.
  2. Veig constants "màgiques" com 50. Potser es tracta de codi provisional, però en tot cas recordeu que no és recomanable fer servir constants sense nom.