denopia / tiralabra-mnkgame Goto Github PK

TiRa-labran koodikatselmointi 1

Denopia / tiralabra-mnkgame

katselmoijana xvixvi

ZIP ladattu ajassa 16:13 perjantaina 25.08.

Määrittely

Määrittelyssä silmiin pistää ensimmäisenä ohjelmoitavan tapauksen monimutkaisuus: tekijä kertoo suunnittelevansa 15x15 ruudukon toteuttamista ensitapauksena. Itse ehkä lähestyisin ensin mahdollisimman yksinkertaisesta tapauksesta, laajentaisin sitten. Toisaalta: mitä väliä?

Muuten määrittely on hyvää mutuilua, tosin en itse pystynyt edes näin konkreettiseen pohdintaan. Tuli mieleen pitäisikö määrittelydokumenttia päivittää nyt projektin edettyä pidemmälle, vai ajavatko toteutus- ja testausdokut asian?

Ohjelman ajaminen

Yksinkertainen Main luokan pyöräyttäminen IDE.llä käyntiin. Hieman jäi häiritsemään graafisen käyttöliittymän puuttuminen, mutta toisaalta algoritmejähän tässä ollaan tekemässä. Tekstikäyttöliittymä on selkeä ja yksinkertainen. Siirtojen syöttäminen on siedettävää tekstikäyttöliittymästä huolimatta, ja idea korostaa viimeisimmät siirrot toimii hyvin: tekee pelistä seurattavampaa.

Koodin laatu ja selkeys

Tekoälyluokan rakenne on selkeä, min-maxiin ei ole lisättävää. koodi on mukavasti strukturoitua. Luokka on ehkä vähän pitkä (+250 riviä) ottaen huomioon, ettei kommentteja ole juurikaan. Joitain metodeja olisi voinut kommentoida paremmin, esim pisteytysfunktion eri osat tai että tarkastaako seuraava metodi laudan reunat tyhjinä vai ei:

    /**
     * Tarkastaa etta ruudun ymparilla ei ole vain tyhjia ruutuja.
     *
     * @param gb Peliruudukko
     * @param row Tarkasteltavan ruudun rivi
     * @param col Tarkasteltavan ruudun sarake
     * @return true jos on yksinainen ruutu, false jos ei ole
     */
    private boolean notLonely(int[][] gb, int row, int col) {
        if (row > 0 && gb[row - 1][col] != 0) {
            return true;
        }
        if (col > 0 && gb[row][col - 1] != 0) {
            return true;
        }
        if (row > 0 && col > 0 && gb[row - 1][col - 1] != 0) {
            return true;
        }
        if (row < gb.length - 1 && gb[row + 1][col] != 0) {
            return true;
        }
        if (col < gb[0].length - 1 && gb[row][col + 1] != 0) {
            return true;
        }
        if (row < gb.length - 1 && col < gb[0].length - 1 && gb[row + 1][col + 1] != 0) {
            return true;
        }
        if (row < gb.length - 1 && col > 0 && gb[row + 1][col - 1] != 0) {
            return true;
        }
        if (row > 0 && col < gb[0].length - 1 && gb[row - 1][col + 1] != 0) {
            return true;
        }
        return false;
    }

Monet metodeista ovat myös privaatteja, eli niitä ei pysty erikseen testaamaan. Tarkoituksenmukaista?

Yhteenveto

Selkeä, kompakti projekti. Hieman olisi voinut kommentoida auki esim pisteytysfunktion osia, kyseessä kuitenkin älyn mielenkiintoisimpia ja merkittävimpiä metodeja (min-maxin toimimisen lisäksi).