2.6 klo 19.30
- Readme oli hyvä ja kohtuullisen selkeä selitys ohjelman toiminnasta. Jonkin verran kirjoitusvirheitä ja muuta, jotka hankaloittaa lukemista.
- Pääluokka hieman epäselvä lukea. Javadocin perustelemat ominaisuudet kuitenkin löytyivät sieltä. Tyhjät rivit voisivat helpottaa lukemisen sujuvuutta, kun samaan asiaan liittyvät komennot olisivat eroteltu toisistaan tyhjällä rivillä. Toki harmittaa myös hiukan, kun projekti vaikuttaa siistiltä, enkä pysty kokeilemaan!! :)
- Tarkennus edelliseen: ohjelma ei siis käänny ainakaan tällähetkellä. Edit. (3.6 klo 9) Ei käänny siksi, että jdk(?)ni on vanhempaa mallia.
- KenttaProfiili erittäin siisti. Muuttujat myös selkeästi nimetty, joten mietin, miksi dokumentoitu kahdesti? Tämä siisteys ja selkeys näköjään koski muitakin miinaharava.Entiteetit paketin luokkia. Kayttaja-luokassa huomioitu hienosti, jos valittu nimimerkki on jo käytössä ennestään.
- Kayttoliittyma-paketin luokat tietysti hankalampia lukea, mutta ihan hyvältä näyttää. Mietin, että onko AloitusNakyma-luokan lisaaAloitusNapit-metodissa tarkoituksella samanniminen JButton muuttuja kuin tuloksia hoitavalla LinkedListillä?
- Miinaharavan moottorin muutama aukaisumetodi on pitkästi dokumentoitu ja siitä nousi vähän ns. karvat pystyyn. Metodeilla on paljon vastuuta. Voisiko ne jakaa useampaan metodiin? (Single Responsibility)
- Ne testit, jotka sinne jo on kirjoitettu, vaikuttavat varsin hyviltä ja luettavilta. Kykenen seuraamaan, mitä niissä testataan. Tässä on kuitenkin se, että ymmärrän kohtuullisen vähän testaamisesta, joten niiden järkevyydestä en osaa valitettavasti kauheasti sanoa, saatika jos jotain voisi vielä testata, mitä ei vielä ole testattu.
- Luokkien suhde toisiinsa vaikuttaa järkevältä, erityisesti Entiteetit ja Pelikentta -pakettien osalta. Näihin mulla on hiukan tottumaton silmä, mutta mielestäni vaikuttaa hyvältä.
Yleisesti ottaen näyttää siltä, että hyvällä mallilla on jo. Toivottavasti saan nähdä tämän saman myös ensi viikolla ja pääsen testaamaan ohjelman toimintaa ! :)
2.6 klo 19.30
Yleisesti ottaen näyttää siltä, että hyvällä mallilla on jo. Toivottavasti saan nähdä tämän saman myös ensi viikolla ja pääsen testaamaan ohjelman toimintaa ! :)