katri96 / bejeweledz Goto Github PK

Projekti ladattu 03.02.2017 klo 16:12

Bejeweledz-luokka

Luokan sisällä on myös toinen luokka (Jalokivi-luokka), muun koodin välissä. Sen pitäisi olla omana erillisenä luokkanaan, sillä jokaisella luokalla tulisi olla vain yksi tehtävä. Toinen luokka sisällä vaikeutti myös koodin tarkastelua.

Testeistä

Testejä oli melko vähän, mutta tässä vaiheessa peliä varmasti hankala niitä tuottaakaan. Lisäksi mielestäni eilisataNollaaJalokivea() ja eiLisaaJalokivia() -testimetodit eivät toimi oikein/testaa mitään. Kaikki tähän astiset testit kuitenkin menivät läpi.

Yleisesti muusta koodista

Luokissa rivien tulisi olla oikein sisennettynä (alt+shift+F voi auttaa), se helpottaa koodin lukua. Joissakin luokissa oli ylimääräisiä/tarpeettomia kommentteja, jotka voisi poistaa. Pelissä ei vielä ollut paljoa testattavaa toiminnallisuutta, mutta sen pystyi kuitenkin jo suorittamaan. Laudan värilliset pallot ilmestyivät joka käynnistyskerralla eri paikkaan, mikä olikin tarkoitus (hyvä!).
Omat metodit oli nimetty mielestäni hyvin eivätkä metodit olleet liian pitkiä.