Koodikatselmointi 1

[tfarm]

Ladattu 2.6 klo 19

• Hyvät ja selkeät kaaviot. Aihemäärittelyssä kuitenkin mainitsit AI-vastustajan mutta sellaista ei luokkakaavioissa tai sourceissa näy, aiotko yhä toteuttaa sen?
• Pelilaudassa oli mukavasti kommentteja, mutta muissa luokissa niitä ei näkynyt. Koodin pilkkominen rivivaihdoilla ja ahkerampi syventäminen saattaisi myös helpottaa luettavuutta. Etenkin pelikäyttöliittymä oli osittain vaikeaselkoista.
• Itse koodi näyttää hyvältä eikä turhaa toistoa ole havaittavissa.
• Testejä on ohjelman joka osa-alueella hyvä määrä. Pitäisi auttaa laajentamisessa.

5/5

[perdperd]

Hei, kiitos kommenteista.

Joo, AI-vastustajaa ei kyllä luokkakaavioissa näy. Saatan vain jättää sen toteuttamatta, täysin satunnaisesti pelaavaa bottia vastaan on aika tylsä pelata.

Kommentteja ei kieltämättä Pelilaudan ulkopuolelta paljon löydy, tämä tulee muuttumaan tulevaisuudessa.

Pelikäyttöliittymä on aika kauhea luokka tällä hetkellä, tuli koodattua sitä joku 4h putkeen ja laatu alkoi kärsiä jossain vaiheessa. Sitä tullaan varmasti refaktoroimaan kovasti.