jekiiski / Raksamanageri

0 stars 0 forks source link

Koodikatselmointi #2

Open swpesone opened 10 years ago

swpesone commented 10 years ago

Projekti ladattu klo 16.31 la 11.10.

Aihemäärittely selittää hyvin, mistä ohjelmassa on kysymys. Aloitin ajamalla ohjelman (kahdesti, sillä olin ensimmäisellä yrittämällä heti vararikossa - tästä en syytä ohjelmaa) ja siirryin sen jälkeen tutkimaan koodia.

Domain: Pätevyys-luokka hämmensi alkuun, kun ihmettelin, olinko missannut jonkin ominaisuuden testaillessani. Peli-luokkaan päästyäni tämä selvisi, ominaisuutta ei siis vielä ole. Tutkin png-muotoisesta luokkakaaviosta (en saanut muita auki), kuinka Pätevyys-luokka liittyy muihin luokkiin, mutta en löytänyt tähän vastausta. Tämä siis jäi hieman hämäräksi.

Kaikki fi.raksamanageri.domain-pakkauksen luokat ovat helppolukuisia ja hyvin kommentoituja, jolloin niistä saa ulkopuolinen hyvin selvää. Metodit ja muuttujat ovat selkeästi nimettyjä

Gui: Tämän pakkauksen sisältö saa näin hienoon ulkoasutoteutukseen tottumattomalle kylmän hien pintaan. Mahdoton arvioida, onko toteutuksessa viilattavaa, kun tuollaiseen törmää ensimmäisen kerran.

Tyokalut: SatunnaisGeneraattori nimensä mukaisesti sisältää useampia arpovia metodeja. Kommentointi on jälleen helppolukuista, metodit ovat mukavan lyhyitä. Tiedostonkäsittelijän ymmärtäminen on itselleni hieman haastavampaa. Joitakin koodinpätkiä on kommentoituina, kenties tämä osuus ei ole ihan valmis?

Testit: Kaikki testit menivät läpi.

Lopuksi: Koodi on helppolukuista, en löytänyt copy-pastea tai huonosti nimettyjä muuttujia/metodeja, joista voisi huomauttaa. Luokkien jako vaikuttaa loogiselta ja ohjelman yleisilme on siisti.