alehuo / 2dplatformgame Goto Github PK

Projekti ladattu 5.1.2017 klo 21.40

Täsmällistä ja tarkkaa työtä. Ohjelman toiminnasta sai heti kättelyssä hyvin selvää, kun otti käteen aiheen kuvauksen, luokkakaavion ja avasi projektin Netbeansiin. Projekti avautui nätisti, testit menivät läpi ja ohjelma käynnistyi ja latasi tason onnistuneesti, vaikka liikkuminen ei vielä onnistunutkaan.

Luokat oli jaettu loogisesti hyvin paketteihin, ja luokat itsessään olivat kompaktin kokoisia. Jäin miettimään, olisiko pakettien logiikan ja single responsibility -ajatuksen kannalta ollut hyvä siirtää Player-luokasta pistelaskentaan liittyviä tehtäviä esimerkiksi GameLevel-luokkaan? Tällöin Player-luokalla ei olisi sekä pelihahmon liikuttamiseen että pisteytykseen liittyviä tehtäviä.

Koodin laatuvaatimukset täyttyivät mielestäni pääsääntöisesti oikein hyvin, ja kommentit tarkensivat koodin toiminnallisuutta. Itse en ole niin tuttu muiden kuin ohjelmoinnin perusteiden ja jatkokurssin asioiden kanssa, ja olisin toivonut ehkä enemmän kommentteja etenkin niihin metodeihin, joissa käytetään itselleni vieraita luokkia. Esimerkiksi bstraktin Entity-luokan render- ja tick-metodeja käyttävissä aliluokissa (Cloud, Tile, Coin) olisi ollut kiva, jos olisi metodin toiminta olisi kussakin tapauksessa avattu parilla sanalla. Vaikuttaisi kyllä siltä, että tämä on work in progress -tyypinen ongelma, ja kommentit lisääntyvät sitä mukaa kuin koodista tulee "valmista".

Metodit olivat pääosin kompaktin mittaisia, joskin ehkä luokissa SpriteSheet ja GamePanel konstruktorit alkavat paisua aika mittaviksi. Vaikka ne sinänsä ovat ihan ymmärrettäviä, niin testauksen ja kokonaisuuden hallittavuuden kannalta purkaminen pienemmiksi metodeiksi voisi olla paikallaan.

Projekti ladattu 19.2.2017 klo 12.45

Erittäin hyvä aihemäärittely, lukemalla sen selvisi jo pelin idea ja toiminnallisuus selvästi. Luokkakaavio oli aika iso ja jostain syystä github antoi siitä vain huonolla resoluutiolla olevan version.

Peli näyttää todella hyvältä, oli hyvä ratkaisu tehdä piirtäminen kuvien avulla sen sijaan että olisi Swingillä piirtänyt. Peli myös pyörii todella sulavasti, vaikkakin hyppiminen näyttää hieman erikoiselta.

Luokkien jaottelu pakkauksiin toteutettu selvästi siten, että projektin eri osuudet erottuvat selvästi ja nälkee helposti mihin mikäkin luokka kuuluu.

Testit kattavia ja pit-testien rivikattavuus myös melko hyvä. Checkstyle myös kunnossa ja Javadoc tehty erittäin perusteellisesti.

Projekti hyvällä mallilla, koodi näyttää toimivalta ja luokat sekä metodit ovat sopivan pieniä. Toimivuus eroteltu hyvin palasiksi. En itse näe mitään korjattavaa tai paranneltavaa tähän mennessä tuotetussa koodissa.