voxwave/javalabpac

Mun Javalabra. Aihe on pacman

JavaLabPac

Mun Javalabra. Aihe on pacman

arvioijalle!!!: projekti on eclipse projekti. Älä missään nimessä avaa netbeansissa. Eli lataat eclipsen sivuilta eclipse keplerin (tai uudemman) ja asennat m2e pluginin ja git pluginin. en suosittele projektin avamaamista laitoksen koneilla koska ad tunnukset. olen tehnyt toimivan jar tiedoston repositorion juureen.

javalabpac's Issues

Koodikatselmointi

Projektia testattu kääntää 16.02.2014 klo 14:00

Aivan aluksi haluan kiittää siistin koodin kirjoittamisesta. Projektin tutkiminen ja sörkkiminen on ollut ilo silmille sekä mielelle, vaikkakin valmiiden kirjastojen yhteydessä olevat metodit välistä saivat pään pyörälle.

Projektin käynnistyessä on pirteää nähdä että jotain oikeasti tapahtuu, vaikkakin selvästi luokkia tutkimalla projekti on vielä kovin keskeneräinen. Toiminnalisuus ainakin jonkinnäköiseksi pacmanpeliksi on kuitenkin rakentunut, joten ei muutakuin innolla rakentamaan yksittäisten olio-luokkien rakenteluun sekä kentän pariin.

Projektin pakkauksiinjako saa sen näyttämään myös järkevältä poislukien itseäni hämmentävää tilannetta, jossa projektin source packagesista löytyy yksi testauspaketti (test.java.com.unkownpotato.javalabpac.entities) sisältäen testiluokat samalla kun taas itse Test Packagesin alta löytyy samat.

Testejä voisi lisätä projektiin enemmän. Vaikkeivat ne ole tällähetkellä kaikkein kriittisimpiä, auttaa etenkin testien tekeminen huomattavasti projektin edistämisessä, ettei tule tilannetta vastaan jossa hämmennytään oudosta bugista johon törmää vasta usean tunnin ohjelmoimisen jälkeen. Tähän toivoisin itse parantamista ja laajentamista.

Dokumentoinnin puolella javadocin laihuutta lukuunottamatta asiat näyttävät olevan hyvällä mallilla. Etenkin tarkennettu luokkakaaviosi on selkeä ja helppo ymmärtää, sekvenssikaavioistakin käy ilmi idea ja toiminnallisuus hyvin, mitänyt hiukan toivoisin parempaa käsialaa näin henkilökohtaisesti. Aihemäärittely on selkeä ja yksiselitteinen, mahdollisen projektin laajentamisen osalta voisin olla utelias näkemään myös muita toiminnallisuuksia itse pelin lisäksi, kuten highscorelista.

Kokonaisuudessa projekti näyttää pätevältä, vaikkakin hiukan keskeneräiseltä vielä. Nokka siis pystyyn ja koodaamaan vaan niin pääsemme nauttimaan pelistäsi myös vapaa-ajallamme.