Koodi on ilmavaa ja enimmäkseen suhteellisen helppolukuista. Luettavuutta helpottaisi vielä esim. @Override -merkinnät muutamaan kohtaan.
Osa pidemmistä metodeista on jaettu apumetodeihin. Voisiko jäljellä olevia pitkiä metodeita jakaa vielä pienempiin osiin (Strategiapelin laskeLiike -metodi)?
Luokkien, metodien ja attribuuttien nimentä on selkeää.
Tarvitseeko Värinarpoja peliä attribuutiksi?
Strategiapeli-luokan ei kannattaisi toteuttaa Runnablea, vaan siitä pitäisi tehdä oma luokka, jolloin Strategiapeli olisi sovelluslogiikkaa ja Runnablen toteuttava luokka kuuluisi käyttöliittymään. Nyt Strategiapeli ei toteuta Single responsibility -periaatetta.
Pelille, TasonLataajalle ja Pisteenlaskijalle voisi ehkä tehdä vielä oman pakkauksen ohjharjoitus.peli. Värinarpojan voisi siirtää käyttäliittymä-pakkaukseen.
Pelatessa joskus kaikki pelin osat eivät aivan mahtuneet ikkunaan ja jotkut värit (ainakin vaalean vihreä) ei erotu taustasta.
20.4.2013 klo 21.35