koodikatselmointi

[jpaljakk]

Katselmoitu koodi ladattu 19.02.2016 21:10

Kokonaisuus jaettu kahteen pakettiin, joiden nimet ovat selkeitä ja
kuvaavat sisältöä. Ei GUI osion olisi voinut tosin jakaa kahteen eri osaan, jolloin
units olisi selkeämmin sisältänyt pelin unitteja.
Paketeissa on kuitenkin sopiva määrä luokkia, joiden nimet eivät itsessään ole
erityisen kuvaavia, mutta projektin tutkimisen jälkeen muuttuvat selkeiksi.

Metodien ja muuttujien nimet voisivat olla pidempiä ja kuvaavampia.
Se helpottaisi ja nopeuttaisi tutustumista koodiin.

Itse pelin idea ei auennut tai peli ei vielä ole toimiva. Palloja
pystyi ampumaan, mutta pisteitä tai game overia ei tullut vastaan.

Luokat sisältävät paljon käyttämättömiä importteja, ja joitain
käyttämättömiä muuttujia ja olioita.

Testejä löytyi lähes kaikille units paketin luokille ja testit
vaikuttivat selkeiltä ja ymmärrettäviltä sekä ilmeisen kattavilta.