Koodikatselmointi

[skajanti]

Koodi ladattu 26.4 21:15

1. Käyttöliittymä.
   • Käyttöliittymän luokat ovat erillään ja itse näkymät erikseen xml:issä, on järkevästi ymmärrettävissä.
   • Käytännössä käyttöliittymä on intuitiivinen eikä siinä ole mitään ongelmia.
2. Käytettävyys.
   • Sudokulauta jäätyy pieneksi hetkeksi sen generoimisen tai lataamisen jälkeen. Se kuitenkin toimii hyvin, itse lisäisin napin jolla voisi poistaa lisätyn numeron. Nollan painaminen ei tuntunut kovin sujuvalta.
   • Jatkokehityksessä tallennuksessa voisi lisätä toiminnallisuuden tallennuksen nimeämiseen. Tallennuksien automaattinen poistaminen saattaisi myös tuottaa ongelmia tosikäytössä.
3. Koodi
   • Luokat ovat loogisesti jaettuja ja metodit lyhyitä (tosin kolme sisäistä for-looppia säikähdytti).
   • Testi compareToReturnsPositive() epäonnistuu, muut menevät läpi. Testit ovat kattavia jä näyttävät järkeviltä.
   • Checkstyle ok.