artokaik / OhHa

Ohjelmoinnin harkkatyö, joulu 2012
0 stars 0 forks source link

Koodikatselmointi 2 #2

Closed Kari-R closed 9 years ago

Kari-R commented 11 years ago

Työ vaikuttaa edenneen reippaasti edellisestä katselmointikerrasta nyt kun siihen on tehty graafinenkin käyttöliittymä. Pelin testikattavuuskin alkaa olla hyvällä mallilla, ja metodien toimintaa on testattu monelta kantilta.

Kuten tekstipohjainenkin, myös tämä graafinen käyttöliittymä on saatu melko hyvin erotettua sovelluslogiikasta. Ainoa itseäni arveluttava kohta on se, kun ohjausolio sisältää kaksi tekstikäyttöliittymäkomponenttia. Oletko harkinnut sellaista vaihtoehtoa, että ohjausolio ei itse sisältäisi lainkaan käyttöliittymää, vaan käyttöliittymä vain antaisi sille käskyjä? Kun nythän ohjausoliota on muokattava jos käyttöliittymää tahdotaan vaihtaa. Voi tietenkin olla, että ratkaisusi on ihan hyvä, enkä vain itse tajua sitä omalla heikolla oliosuunnitteluosaamisellani. Joka tapauksessa yksi suosittu Mafia-pelin sovellutus on sen käyttö IRC-boteissa, ja tämä vaikuttaa sellaiselta toteutukselta, joka olisi vaivaton ja nopea liittää osaksi jotain Java-bottia, joten kyllä projektin oliosuunnittelu tuntuu ihan toimivalta.

Kooditiedostojen jako entistä tarkemmin omiin paketteihinsa on lisännyt merkittävästi projektin selkeyttä, mikä onkin tärkeää nyt, kun koodia on kertynyt enemmän. Projektin kooditiedostoissa osaisi nyt navigoida sellainenkin, joka ei vielä ole perehtynyt tarkemmin itse ohjelman toiminnallisuuteen. Koodiin on ilmestynyt myös paljon javadoc-kommentteja, tosin nythän niistä ei ole vielä kauheasti apua, kun suurin osa niistä on vielä tyhjillään. Mafia-pelin säännöt eivät välttämättä ole kaikille tutut, joten kattava kommentointi voisi olla paikallaan lopulliseen versioon. Toisaalta metodeilla ja muuttujilla on monessa paikassa niin kuvaavat nimet, että ne oikeastaan kommentoivat jo itse itsensä.

Edit. Latasin koodin 9.1 klo 11:30.