teeaateeuu / miinaharava Goto Github PK

Koodi kääntyy hyvin. Peli pyörii ja on KOUKUTTAVA. Alla koodikatselmointi Clean code- periaatteiden mukaisesti.

1. Luokat, metodit ja jne. ovat nimetty hyvin. Osin pitkiä nimiä, mutta erittäin kuvaavia.
2. Metodien pituudet hallittu erittäin hyvin. Vain Pelilauta-luokan kaksi metodia, tarkistaYmparoidytRuudut ja avaaYmparoidutRuudut ovat hieman liian pitkiä ja yksinkertaistettuja, mutta keskustelimme tästä Tatun kanssa ja tulimme molemmat siihen tulokseen, että parempi että yksinkertainen ja toimii kuin vaikea ja toimimaton. Single Responsibility periaatetta noudatetaan hyvin.
3. Ei lähes ollenkaan toistuvaa koodia, hyvä Tatu!
4. Luokilla selkeät toiminnot, ei pahaa sanottavaa.
5. Pakkauksiin jako suoritettu selkeästi, Logiikka, GUI ja testit omissaan.
6. Testit varsin kattavat: yhteensä 37 kolmesta eri luokasta. Kaikki testit menevät läpi.

Javadoc- ei vielä kokonaisuudessaan valmis. Ruutu-luokan kommentointi hyvä ja selkeä alku.

Yksi bugi löytyi. Ensimmäisen arvauksen vaaka- ja pystyriveillä ei muodostu miinoja. Ohessa ruutukaappaus liioitellusta tilanteesta.

Yleisesti ottaen koodi erittäin luettavaa, logiikka toimii selkeästi ja toiminnot rakennettu loogisesti. Tatu selkeästi allekirjoittanutta edistyneempi Javassa ja kyseinen työ jo erittäin hyvin edennyt kommentointia ajatellen.