Koodi ladattu 28.04 klo 17:01

Koodin rakenne ja luettavuus ovat erinomaisia. Hahmotin nopeasti, miten eri luokat liittyivät toisiinsa selkeiden luokka- ja metodi nimien avulla.

Virheiden hallinta UserOperationResult enumien avulla on mielestäni hyvä idea. Olisin ehkä itse tehnyt omia poikkeuks (Exception) luokkia, jotka laajentavat (extends) luokan "Exception". Mutta tämä lienee mielipide asia.

User luokalla on oma UUID, mutta käytät silti käyttäjän nimeä löytämään käyttäjää UserDao oliosta SnakeService luokassa, renameUser ja setUserColor metodeissa. Näissä tapauksissa olisi varmasti turvallisempi yksilöidä käyttäjä UUID:n avulla

Tykkään, miten olet toteuttanut eri ikkunoiden vaihtumista switchin avulla. Mielestäni erinomainen tapa toteuttaa käyttöliittymän logiikka.

Eri käyttöliittymien ikkunoiden (view) koodi rakenne on selkeä ja olioiden nimet ovat selkeitä ja kuvaavia. Tämä tekee koodin luettavuudesta erinomaista.

Olet myös käyttänyt rajapintoja (interface) ja abstrakteja luokkia hyvin.

Testejä on kattavasti ja ne testaavat, että metodit tekevät mitä kuuluukin. Hienoa että testaat myös virhetilanteita!

Harmittaa ettet ole vielä kerennyt tekemään itse pelin logiikkaa, olisi ollut mukava katsella sitä myös.