search

josepma / fofo

Programming project of the subject Models de procés
0 stars 2 forks source link

Equals Match #34

Open jordiniubo929 opened 11 years ago

jordiniubo929 commented 11 years ago

Al contenir nous atributs, el equals de match s'hauria de modificar.

Jo crec que un match que no té àrbitre assignat es correcte. Actualment si un match no te un àrbitre i crida a m.referee.equals(this.referee); dona error. Aquest es el motiu del error dels test de WeekMatch, ja que els match no tenen assignats àrbitres

De la mateixa forma, crec que s'haura de comprobar els atributs de les dades

anat0lius commented 11 years ago

Pel que fa als equals en general, proposo que si s'han de comparar una serie d'atributs, que aquests estiguin inicialitzats pel constructor. Sobretot els Strings. Ho he fet a Club i Referee. Es cert que la majoria d'aquests atributs s'harien de "omplir" obligatoriament, pero facilita bastant la feina a l'hora de testejar i no tindre que anar inventant-se i donant valors a absolutament tots els atrbuts. L'altra manera seria que si no es vulgues inicialitzar un Srtring amb "", doncs al equals no hariem de fer servir el string.equals() sino el comparador "==", d'aquesta manera pot fer comparacions de nulls.