techstalgist/sudoku

sudoku's Issues

Koodikatselmointi 2

Latasin projektin lauantaina 18.2.2017 klo 17.26

Ensimmäisenä huomiona, että kun käynnistin ohjelman ensimmäisen kerran ja otin helpon sudokun, niin ohjelma tuotti sudokun, jonka ratkaisu ei ollut oikein. Yhteen sarakkeeseen olisi väkisin tullut kaksi samaa numeroa, eikä ohjelma hyväksynyt mitään numeroa oikeaksi vastaukseksi (taisi olla kaksi tai kolme ruutua, joihin ei mikään luku kelvannut). En osaa sanoa, mistä ongelma johtuu.

Yleisesti koodausjälki on todella hyvää. Koodi on jaettu selkeästi metodeihin ja isot kokonaisuudet pilkottu useampiin luokkiin. Muuttujat, metodit ja luokat on myös nimetty selkeästi ja johdonmukaisesti. Pakko vielä nostaa erikseen, miten paljon luettavuuteen on panostettu, kun GeneraattoriUtil-luokassa on metodi tyhjaRuutu, joka tarkistaa, onko tietyn alkion arvo 0.

Vaikka dokumentaatiota ei tarvinnut arvostella, niin pakko siitäkin todeta, että ei voi muuta sanoa ku et hattuu nostaa. Luokkakaavio on todella huolellisesti tehty, samoin sekvenssikaaviot.

Sen verran koodin luettavuudesta voisi kommentoida, että käytössä on paljon staattisia metodeja, joita kutsutaan ilman luokkaviitettä, mikä vaikeuttaa kutsujen hierarkian hahmottamista. Esim. Sudokupelin metodi paivitaArvo kutsuu toista metodia annettuLukuOnSopivaLuku, mutta omaan mieleeni olisi selkeämpää, jos tätä metodia kutsuttaisiin muodossa SopivatLuvut.annettuLukuOnSopivaLuku, koska kyse on toisen luokan staattisesta metodista.

Koodauksessa kieleksi on valittu suomi, mikä on ihan makuasia, mutta ainakin sitä on käytetty johdonmukaisesti kaikessa koodissa (paitsi tietysti @override-metodeissa), sekä javadocissa.

Ei nyt valitettavasti pysty kauheasti auttamaan kehityksen suhteen, en pysty näin lyhyessä ajassa paikantamaan, mistä tuo virheellisen laudan tuottaminen johtui. Muuten koodi on niin hyvälaatuista ja projekti pitkällä, ettei tähän kauheasti mielestäni löydy lisättävää tai parannettavaa, ainakaan omasta arsenaalistani. Tsemppiä loppuhiomiseen, toivottavasti tuo virheellisen laudan luova bugi löytyy! Olen nyt useamman pelin pelannut uudestaan, enkä ole onnistunut toistamaan virheellistä lautaa. Tätä pelailee mielellään ihan huvikseen. Todella hyvä projekti ja laadukasta jälkeä joka tapauksessa! Kateeksi käy!

Koodikatselmointi

Latasin projektin 5.2 sunnuntaina. Ohjelma vaikuttaa olevan jo aika pitkällä ja valmiintuntuinen. Helppo ja keskivaikea sudoku tuli pelattua läpi ja toimi hyvin. Testeistä UtilTestissä tulostaaOikein() ei mennyt läpi.