Minikaavio on juuri niin suuntaa antava, kuin pitääkin.
Luokkakaaviossa jo luokkien tärkeimmät ominaisuudet kirjattuna.
Sekvenssikaavio oikein toteutettu ja selkeä. Olisi voinut olla toinen kaavio vaikka vielä lisäksi toiminnallisuudesta, kun pelaaja klikkaa avaamatonta ruutua.
Testit: (koodi herjaa BUILD FAILED, eikä ollut aikaa lähteä etsimään syytä, joten katsoin nopeasti testit päällisin puolin)
Testit ovat selkeästi kasattuja ja loogisesti nimettyjä. On helppo ymmärtää mitä kukin testi testaa.
Moni testi testaa useilla eri arvoilla, jolloin saadaan testikattavuus isoksi, hyvä!
Koodi:
Paketit ja miinat ovat hyvin jaettu selkeisiin kokonaisuuksiin.
Tykkään, että koodi on englanniksi, mutta silti selkeää.
Javadocin määrää voisi lisätä pienempiinkin metodeihin MiinaEngine-luokassa, koska se nopeuttaa huomattavasti metodin toiminnan ymmärtämistä. (vaikka hyvin nimetyt metodit oikeastaan ajaakin käytännössä saman asian)
Lisäksi ConsoleUi:ssä puuttuu vielä kaikki docit
J.doc olisi hyvä myös laittaa luokille. Toistaiseksi vielä tyhjänä.
(ymmärrän toki, että javadocit ei ole se tärkein asia vielä tässä vaiheessa, mutta helpottaa huomattavasti tulevaa urakkaa, kun laittaa edes vähän jotain infoa metodista samalla, kun sen luo.)
Kokonaisuutena oikein selkeä paketti. Koodi on puhdasta ja hyvin sisennettyä, metodit hyvin nimettyjä ja dokumentointi lähes ajan tasalla! Good job!
tpatja
commented
11 years ago
Projekti ladattu 3.6.2013 kello 16.20
Dokumentaatio:
Testit: (koodi herjaa BUILD FAILED, eikä ollut aikaa lähteä etsimään syytä, joten katsoin nopeasti testit päällisin puolin)
Koodi: