Reformed chess

[DavidNagisa333]

Alo,

s'ha suprimit (comentat de moment) de Chess alguna funció que ja no ens es util ja que el JSONparser ens retorna la informació filtrada. S'ha adaptat la classe al nou codi i atributs de les altres classes. La funció de comparar Chess està creada i s'està treballant en la que retorna els moviments possibles de una peça amb els seus respectius valors. També es farà el control de les peces que moren de cada jugador.

He agefit descripcions amb pre i post a les funcions per clarificar l'avanç i facilitar la comprenció.

[miquelbofill]

Comentaris sobre la classe Chess:

1. Els print de l'operació checkMovement s'haurien de reemplaçar per excepcions.
2. Hi ha operacions massa llargues (p.ex. destinyWithValues). S'haurien de descomposar.
3. Què és el número "màgic" 50?

Ànims, la cosa va tirant bé.

[DavidNagisa333]

Bones,

no entenc el perque pero no m'arriben les respostes al mail... sento tardar en respondre.

1. Si, els prints haurien de ser excepcions pero de moment vaig deixan-los aixi perque vaig fent bastants canvis adaptant la classe segons anem afegint coses.
2. Entés, faré altres funcions que facilitin aquestes funcions llargues.
3. Aquest 50 fa referencia a una peça que es pot moure un nombre ilimitat de caselles, com les torres o alfils (mentre el tamany del tauler ho permeti). Actualment diferenciem aquestes peces amb un valor que només puguin adoptar elles, com 50, aixi tenim un int com la resta de posicions. Se que a primera vista es algo sobtat perque no es sap d'on surt, pero no es correcte?

[miquelbofill]

no entenc el perque pero no m'arriben les respostes al mail

És estrany perquè vas obrir tu l'issue i apareixes com a participant. Potser has de canviar les notificacions a l'estil:

Sobre els números màgics com 50, el que es recomana és definir una constant (atribut final static), és a dir donar-los un nom i una descripció en forma de comentari. Això facilita tant la comprensió com la modificabilitat del codi.