Koodikatselmointi

[HeikkiHei]

Ohjelma ladattu 5.2.2016 klo 19.28

• Koodia on paljon, mutta koodi on osittain vaikealukuista, johtuen lähinnä lyhyistä muuttujien nimistä. Metodit ovat muuten kokonaisuutena selkeitä ja ne ovat itsessään nimetty erittäin hyvin.
• Toiminnallisuutta olisi ollut kiva testata, mutta ymmärrän, että fysiikkamallinnus vie paljon aikaa. Kaikki testit menevät kirkkaasti läpi (paitsi pari, niistä kohta), joten oletan fysiikan toimivan.
• Testereitä on todella runsaasti, mikä on hyvä. Pari setteriä antaa vielä punaista, koska niitä ei ole testattu lainkaan ( NO_COVERAGE ), mutta niitähän ei tarvinnutkaan testata. Kaiken kaikkiaan kattavaa testausta.
• Kuten aiemmin todettua, metodit on nimetty hyvin, mutta samanlaista tarkkuutta kaipaisi myös luokkien nimiin. Entity ja BoundingBox jäivät hieman epäselviksi, mutta oletan niiden tarkoittavan jotain aineellista esinettä ja pelialueen rajoja. Vaatii aika paljon koodin tutkimista tämän selvittäminen.
• Luokkien sisältö on hyvä, jokaisella näyttäisi olevan juuri oikeat vastuualueet (Karttu on karttu, Thrower heittää, jne), tosin Point voisi ehkä korvata muutamaan muuhunkin metodiin x-, y- ja z-akseleita, toisi lisää selkeyttä.
• Tekijällä on selvästi vahva visio siitä, miten ohjelma tulee toimimaan ja mitä toiminallisuuksia tulee olemaan. Tämä ei välttämättä tässä vaiheessa vielä näy ulkopuoliselle, mutta odotan innolla, mitä tästä tulee.
• Luokkakaavio ilmentää ilmeisesti lopullista ratkaisua, joka auttoi myös hahmottamaan koodin kokonaisuutta.

EDIT: Samalla kun kirjoitin tätä, tekijä päivitti työtänsä. Luokkakaavio on laajempi kuin mitä ladatessani sen, mikä on hyvä, parantaa vielä enemmän selkeyttä.

[laitalaj]

Jees, kiitos näistä.

Yritän pitää mielessä pidempien muuttujien nimien käytön tulevaisuudessa, ainakin tapauksissa joissa muuttuja ei ole jatkuvassa käytössä.

Entity on fysiikan lakeja noudattava objekti, BoundingBox taas käytännössä hitboxi. Voisin vaikka refraktoida nuo kaksi esimerkkiä vähän kuvaavammiksi saman tien.

Pitääpi myös vilkaista noita Pointin käyttömahdollisuuksia.