Koodikatselmointi #1

[Sxvz]

Projekti ladattu 25.9 klo 15:00

Kokonaisuudessan projekti näyttää olevan hyvässä vaiheessa. Koodi itsessään oli todella selkeää ja helppoa ymmärtää. Kiitos tästä hyville nimille. Valittu aihe oli myös lystin kuuloinen.

Muutamia mieleen tulleita huomioita:

• Tarvitsetko oikeasi YAML:ää tai vastaavaa jos meinaat tallentaa tiedostoihin vain yksinkertaisia muuttujia (ellen ole ihan sokea). Eikö raakateksti olisi ihan riittävä tapa? Jotakin seuraavan kaltaista per rivi tallennettavaMuuttuja1 10. Varsinkin jos ohjelma itsessään ei tallenna tiedostoon mitään niin tuollainen on helppo lukea ennaltamäärätyssä järjestyksessä tiedostosta.
• givenStringArrayIsCorrectLength -metodi Game.java:ssa ei vaikuta kauhean tarpeelliselta. Eikö samalla/vähemmällä vaivalla pääse kirjoittamalla if-lauseen aina tarvitaessa?
• Sallittujen merkkien määrittäminen on kiva idea, mutta tuntuu vähän turhalta valikoissa sillä if-lauseet hoitavat merkkien tarkistuksen jo. Annat toki kiellettyjen merkkien ja tyhjien inputtien kohdalla palautetta pelaajalle tästä, mutta jos pelaaja syöttää jonon sallittuja merkkejä ei ohjelma tee mitään eikä anna palautetta tästä. Jos haluat säilyttää palautteen niin kannattanee tarkistaa myös syötteen pituus ja antaa liian pitkistä syötteistä palautetta pelaajalle.
• Jos aiot käyttää teksti ui:n osia vielä myöhemmin eriyttäisin ainakin itse inputteja käsittelevät metodit omaan luokkaansa, josta niitä on helppo käyttää vaikka jossain myöhemmässä projektissa
• Switch case -konstruktiot on kivoja ja helppolukuisempia kuin if-else-if -viritelmät (IMO) mutta jälkimmäisellä pääse kyllä vähemmällä työllä silloin, kun switch case:n joutuisi breikkaamaan joka kohdassa. Samalla saattaa päästä vähemmällä vaivalla tulevaisuudessa, kun metsästää mystisiä bugeja, jotka johtuvat breikkien puutteesta.

Ihan pirun hyvää työtä siis tähän mennessä. Itselläni ei ole noin paljon mielikuvitusta.

-Sami Bäckroos (Sxvz)

[iah1016]

Kiitos paljon hyödyllisistä kommenteista!

• Juu, täytyy nyt vielä miettiä tuota YAML:ia. Tarkoitus oli, aikataulun salliessa, lisätä peliin tallennus-/latausmahdollisuus. Kyllähän siinäkin tilanteessa toki perustekstitiedosto toimii aika pitkälle.
• Sorruin ehkä turhaan kikkailuun, mutta pääasiallinen idea tässä oli saada nullchecker pois metodista. Nyt kun miettii, niin tässä ei varmaan tarvita nullcheckeriä ollenkaan, jos kutsuttaessa ei anneta ikinä nullia.
• Tekstikäyttöliittymä jäi tosiaan vähän vaiheeseen. Hyvä, kun mainitsit sen pituuden rajoitteen. Täytynee lisätä sitten, kun alan tehdä gui:ta.
• Juu, teksti-ui jäi melko kammottavannäköiseksi. :D Oikeassa olet inputia käsittelevien metodien kanssa. Kaikki teksti olisi myös tarkoitus siirtää tekstitiedostoon.
• Jep, täytyy miettiä tätäkin vielä.