Latasin koodisi 21.9. klo 12. Ja tässä muutama huomio:
Mielestäni luokkia oli käytetty hyvin ja ne oli nimetty intuitiivisesti. Pienenä poikkeuksena Randomiziser, jolla on jostain syystä englanninkielinen nimi.
Myös metodit oli nimetty selkeästi, vaikka jäinkin hieman kaipailemaan kommentteina tarkempia kuvauksia (tosin niitä ei kai vaadittukaan vielä tässä vaiheessa).
Testeistä PelinAlkutilanneLuodaanOikeinTest ei toiminut, mutta muutoin testit näyttivät toimivan. Varmaan seuraavana hommana kannattaa laajentaa logiikka-paketin testejä esimerkiksi Randomizerin satunnainenMerkki metodista puutui testi.
Main ja Sanapeli luokat puuttuivat kaaviosta. Jäin muutenkin miettimään, että onku tuo erillinen Sanapeli-luokka tarpeellinen vai voisiko sen sisältö olla Mainin metodeina.
Itselläni oli ainakin ongelmia merkistön, kanssa jolloin ääkköset eivät tulostuneet oikein. Voisikohan tuonne lisätä koodia joka muuttaa kaikki käytetyt kirjaimet vaikka UTF-8 -formaattiin?
Kaikkineen vaikutti projektisi huomattavasti valmiimalta kuin omani.
BikkuMyy
commented
8 years ago
Latasin koodisi 21.9. klo 12. Ja tässä muutama huomio:
Mielestäni luokkia oli käytetty hyvin ja ne oli nimetty intuitiivisesti. Pienenä poikkeuksena Randomiziser, jolla on jostain syystä englanninkielinen nimi. Myös metodit oli nimetty selkeästi, vaikka jäinkin hieman kaipailemaan kommentteina tarkempia kuvauksia (tosin niitä ei kai vaadittukaan vielä tässä vaiheessa).
Testeistä PelinAlkutilanneLuodaanOikeinTest ei toiminut, mutta muutoin testit näyttivät toimivan. Varmaan seuraavana hommana kannattaa laajentaa logiikka-paketin testejä esimerkiksi Randomizerin satunnainenMerkki metodista puutui testi.
Main ja Sanapeli luokat puuttuivat kaaviosta. Jäin muutenkin miettimään, että onku tuo erillinen Sanapeli-luokka tarpeellinen vai voisiko sen sisältö olla Mainin metodeina.
Itselläni oli ainakin ongelmia merkistön, kanssa jolloin ääkköset eivät tulostuneet oikein. Voisikohan tuonne lisätä koodia joka muuttaa kaikki käytetyt kirjaimet vaikka UTF-8 -formaattiin?
Kaikkineen vaikutti projektisi huomattavasti valmiimalta kuin omani.
Terveisin Petri