Koodikatselmointi #2
Description
Projekti ladattu 20.2.2015 klo 12.15.22
Tasohyppelypeli on mielenkiintoinen vaikkakin hieman nopea minulle:) Pelissä toimi ensimmäinen taso,
Yhdyn ensimmäisen katselmoijan näkemyksiin: ohjelman rakenne on selkeä, luokilla selvät tehtävät, metodit eivät ole liian isoja ja ne ovat hyvin kuvaavasti nimetty. Turhaa toistoa ei ole. Keskityn tässä katselmoinnissa projektin viimeistelyyn, koska merkittäviä parannuksia itse ohjelman runkoon ei ehtine enää tehdä.
En aluksi saanut Pit-testejää toimimaan omalla NetBeansillä. Testiraportti näyttivät, että rivikattavuus on nolla. Ongelma johtunee NetBeansini asetuksista, joita en osannut muuttaa. Ongelma oli myös minulla, jonka sain lopulta korjattua sillä, että KAIKILLA testeillä on vastaava luokka source-koodin puolella. Huomasin tänään 23.2. kurssisivuilla ohjeen jossa vaihdettiin Pitin version pom.xml filessä. Pit-raportti alkoi toimimaan myös tässä projektissa.
Huomioita:
Peli on kehittynyt edellisestä katselmuksesta. Ennen kaikkea painovoiman lisääminen on tehnyt siitä varmasti hauskemman. Peli on itseasiassa aika koukuttava. Sitä ei minun mielestä kannata enää lainkaan laajentaa, jotta projektin aikataulu pitää.
Peli on rakennettu siten, että myöhemmin sitä pystyy laajentamaan suhteellisen helposti Tässä vaiheessa mielestäni edes kakkosleveliä tarvitse välttämättä saattaa loppuun, koska aiheen laajuus on näkemykseni mukaan riittävän laaja. Levelien laajentamissuunnitelmat kannattaa kuitenkin ehdottomasti kirjata dokumentointiin. Toki peliä voi laajentaa jo ennen palautusta, jos aikaa vain löytyy.
Testit:
Pit-testit ovat kattavia. PelimottoriTest luokkaan voisi keksiä jonkinlaisen todellisen testin. Tärkeämpää tässä vaiheessa on kuitenkin saada toimiva ohjelmisto aikataulun mukaisesti markkinoille, kuin hirttäytyä yhden ilmeisen luokan testaamiseen.
Kehitysehdotus:
Edelleen korostan projektin saattamista loppuun aikataulussa, mutta maaliin tulon kunniaksi voisi olla joku hieno ikkuna. Esimerkiksi mustalla tekstillä harmaalla pohjalla: "Onnea, pääsit maaliin!"
Hieno projekti, joka lataushetkellä tarvitsee vain loppusilauksen. Loppusilaukseen kannattaa kuitenkin varata aika monta tuntia. Kannattanee vielä varmistaa, että ohjelma ja ennenkaikkea testit toimivat laitoksella. Loppukiri alkaa, mutta kohta vihreä maali on jo kohdalla!