Projekti ladattu katselmointia varten 8.10 n. klo 20:00
Projektista oli suoraan sanottuna vaikeaa löytää huonoja puolia. Mielenkiintoinen aihe!
Jotkin 'lyhennykset' luokkien ja muuttujien nimissä tekevät niiden nopeasta ymmärtämisestä hankalaa, esim. generaltoolsin StrIntMapStringListInterconversion, StrStrMapStringListInterconversion
ja Player-luokan argSkills voisi vaikka nimetä suoraan argumentationSkills-nimiseksi. Nämä selviävät kyllä nopeasti kun katsoo luokkien sisään.
Testien lukumäärä ja testikattavuus sekä rivien ja mutaatioiden osalta on täydellistä joten niistä ei
mitään moitittavaa.
Pakkaukset on toteutettu mielekkäästi ja loogisesti vaikka niitä onkin paljon.
Abstraktin Conversion-luokan käyttö mielestäni hyvä idea pelin laajentamisen ja selkeyden puolesta.
Koodi on helppolukuista luokkien sisällä ja JavaDoc toteutettu hyvin selventävästi. Esimerkiksi switchin käyttö GameLogicissa mielestäni selventää koodin luettavuutta mukavasti.
Kaiken kaikkiaan projekti näyttää olevan todella hyvässä vaiheessa.
Projekti ladattu katselmointia varten 8.10 n. klo 20:00
Projektista oli suoraan sanottuna vaikeaa löytää huonoja puolia. Mielenkiintoinen aihe!
Kaiken kaikkiaan projekti näyttää olevan todella hyvässä vaiheessa.