dukeyli / NapoleoninHauta

0 stars 0 forks source link

Koodikatselmointi 2 #2

Open kymcoscooters opened 7 years ago

kymcoscooters commented 7 years ago

Ladattu 8.1.2017 14:06

Projektin idea on hyvä ja olen itsekin pelannut kyseistä peliä joten siksi ymmärrän pelin idea vaikka aihemäärittely ei sinänsä kerro itse pelin kulusta sen enempää.

Pelin graafinen ulkonäkö ei ole kovin kummoinen mutta jos tilanne on sama kuin minulla eli ohpe ja ohja on ollut ensikosketus ohjelmointiin niin ymmärrän sen ihan hyvin. Pinojen nimet näyttää hieman hassuilta, olisi parempi jos teksti olisi keskellä ruutua.

Et ole ilmeisesti päässyt ohjelmoimaan mitään toimintoja peliin, ainakin lataushetkellä molemmat ActionListenerit oli suhteellisen tyhjiä. Olisi ollut kiva päästä kokeilee edes jollain tasolla.

Testejä on toteutettu tosi kattavasti pakka ja pelialusta-luokille, main-luokkaa ei ole testattu ollenkaan ja sitä voisi ehkä jättää testauksen ulkopuolellekin.

Nyt taisi itse asiassa näitä kirjoittaessa tulla joku isompi pläjäys päivityksiä, mutta kommentit koskevat siis ohjelmaa siinä tilassa missä se oli 14-aikaan.