Koodikatselmointi 2 #3
Description
Kloonattu to 13.6.2019 klo 16.22
Projekti on laaja ja sen rakenne on hyvä ja siisti. Hakemistoja on käytetty monipuolisesti ja testit ovat omassa hakemistossaan siististi ja myös sisäinen jaottelu on toteutettu siististi ja alihakemistoja on käytetty runsaasti. Pakkauksen abcb nimentä ei ole (ainakaan itselle) kovin kuvaava ja sitä ehkä voisi miettiä.
Metodit ja muuttujat on nimetty selkästi ja kuvaavasti. Javadocit ovat selkeitä ja niistä saa hyvän käsityksen toiminnoista. Vaikka esimerkiksi Minimax.java luokassa ei ole (vielä?) javadoceja, kuvaavista muuttujien/metodien nimistä saa jo hyvin selvää tarkoituksesta. Silti ainakin pisimpiin metodeihin olisi hyvä lisätä javadocit lisäselvennykseksi. Koodi on kaikkiaan hyvää ja se on jaoteltu hyvin metodeihin ja sitä on selkeää lukea. Metodit ovat lähes kaikki sopivan pituisia. Alphabeta metodi on melko pitkä, mutta voi olla, että sitä on luonteensa vuoksi vaikea jaotellakaan lyhyemmiksi.
Yksikkötestit ajoin komentoriviltä ja ne menivät läpi. Testausdokumenttiin tulee varmastikin vielä lisää asiaa. Sovelluksen graafinen ilme on hieno ja sinänsä käyttöliittymä on niin selkeä, etten tarvinnut ihmeempää käyttöohjetta. Nappi, josta pääsisi palaamaan valikkoon olisi mahdollisesti pelaajan kannata hyvinkin kätevä.
Kun latasin projektin, jouduin muuttamaan openings2.txt tiedoston nimen openings.txt nimiseksi, jotta peli aukesi. Tekoäly toimi hienosti ja onnistui voittamaan minut (joka ei tosin ole varmaan kovin vaikea), mutta siirrot olivat hyviä. Kun pelissä tuli shakkimatti, siitä ei tosin tullut ilmoitusta näytölle, mutten voinut myöskään liikuttaa enää mitään nappia. Katsojaroolissa peli ilmoitti lopussa shakkimatista pelinäytölle. Tekoäly vaikuttaa tehokkaalta ja on myös nopea. Suorituskykytestejä ei ollut tästä vielä nähtävissä, mutta ne ovat mitä luultavimmin tekeillä vielä. Pelin toiminnot ja sallitut siirrot vaikuttavat toimivan oikein.
Kaiken kaikkiaan huolellisesti ja hyvin rakennettu työ. Hyvin laaja, hieno projekti!