pheis / pelishakkilaudalla Goto Github PK

Koodikatselmointi suoritettu 30.1.2017 commitin 6b23e68 aikaisilla tiedostoilla.
Yleistä

• Kannattaa lisätä .gitignore ja sinne *.class yms., jottei käännettyjä .class tiedostoja päädy vahingossa mukaan lähdekoodin sekaan
• Lähdekoodissa useammassa paikassa sekaisin täbeja ja välilyöntejä, ehdottaisin, että vain toista käytettäisiin
• Checkstyle-virheitä löytyy 157 kappaletta, niitä kannattaa vilkaista mieluummin aikaisemmin kuin myöhemmin etteivät ne kasaudu

oha.shakkiproggis.Board.java

• Koodissa hyödynnetään runsaasti Java 8:n uusia ominaisuuksia, hienoa Streamin ja Optionalin käyttöä!
• Suurin osa ohjelmasta on keskittyneenä tähän tiedostoon - voisiko toiminnallisuutta hajauttaa muihin luokkiin? Esimerkiksi pelkästään Square-enumia koskevat metodit voisi siirtää Square:n sisälle (muistathan että enumit ovat Javassa myös luokkia, katso esimerkiksi Planet-esimerkki tästä linkistä).
• Shakkilogiikassa käytetään aika paljon tiiviitä, nokkelia temppuja. Ehkä logiikan toimintaa voisi vähän avata muuttujien, metodien ja kommenttien avulla?
• Koodissa muuttujat esitellään melkein aina ennen niiden arvon sijoittamista, esim:

  boolean t;
  t = ((a.ordinal()%8) - (b.ordinal()%8) < 0);
  

  Mielestäni yllä oleva voidaan laittaa ihan hyvin samalle riville:

  boolean t = ((a.ordinal()%8) - (b.ordinal()%8) < 0);
  

Testit

• Todella hyvä testikattavuus! Testit kattavat 95% koodista jos Mainia ei oteta huomioon.
• Testiluokat kannattaa laittaa samanlaiseen kansiorakenteeseen kuin lähdeluokatkin (esim. KnightAttackTest ja KnightAttack ovat paketissa oha.shakkiproggis, mutta KnightAttackTest.java ei ole kansiossa test/java/oha/shakkiproggis)
• Tuleeko jokaiselle nappulatyypille monta testiä? Jos ei, yksittäisten nappuloiden testit voi laittaa esimerkiksi Boardin testeihin