himotimo / bitz Goto Github PK

Kurkkailin ohjelmaa ja yritän antaa jonkinlaista palautetta :D Renderöinnissä käytetyt kirjastot vaikuttaa huvältä valinnanlta, pitäis varmaan omassakin ohjelmassa harkita. Metodit on nimetty hyvin ja kuvaavasti. En tiedä oisko ollu hyvä jakaa homma useenpaan pakkaukseen, mutta ei tossa mitään ongelmaa ole. Javadoc antaa hyvin lisäselvyyttä.

Joissakin metodeissa ehkä turhan paljon tavaraa. Esimerkiksi Logic -luokan enemyLogicUpdate() -metodissa loopin sisällä selkeesti kaks eri hommaa (niinku kommenteistaki selvii) jotka ehk vois laittaa omiks metodeikseen. Ehkä ois helpompi hahmottaa, mutta toki aika selkee nytkin sinänsä. Kyse enemmän jostain muotoseikoista. Samalla lailla projectileLogicUpdate() koostuu kahdesta osasta. Toki niitäkään tuskii käyttämään uudestaan, mutta ehkä selkeempää jos rivipituus lähempänä viittä ku viittätoista. Omassa ohjelmassa viel pitkii ja sekavii metodei et oon vähän jäävi höpisee näist. Mutta ei tässä nyt parin metodin lisäks mikään pahemmin pistä selmään, näyttää selkeeltä nää koodit.

Annetaan ny vielä pari pidempää metodia: Shooter -luokan moveAllProjectiles() vähän pitkä ja vois ehkä jakaa? Samoin walls -luokan setupArray(). Varsin selkeit molemmat mut kyse ehk siitä et laiskana lukijana katon et ompas pitkä ja ois ehkä helpompi jos jakautuis. Mutta sitten toisaalta esim Enemy -luokassa olevien parin if-lauseita sisältävän metodin kuten move to player tapauksessa en tiedä onko järkee lähtee jakaa, nyt tavallaan aika selkee.

Ei mulla muuta, vaikuttaa hyvältä! -Jarmo

katselmoitu siinä tilassa kuin 9.10 2016 klo 20:42