falzed/shakki-probable-carnival

javalabra

shakki-probable-carnival's Issues

Ohjelma ladattu 09.10.2016 Klo 18.15

Aika makee. Siinä ensi vaikutelma. Ja aika laaja ja haastava projekti, hienosti toteutettu.

Silmään pistäneet hyvät asiat:

• Variant rajapinnan default metodit
• nappulat, lauta alusta UI tehty näyttävästi
• Luokat ja pakkaukset selvästi nimetty (englantia toisena äidinkielenä puhuvan näkökulmasta)
• javadoc selkeä
• Peli toimii mallikkaasti ja ohjeet olivat tarpeeksi selkeät.

Mitä voisi parantaa:

• Nappula luokan getMerkki() swich-case lauseeksi if-sijaan.
• koodiin kommentteja (jos haluaa)
• checkstyle huomautti javadocin korjauksista

Netbeans ilmoitti myös tämmöisestä ongelmasta:

Project's main artifact is processed through maven-shade-plugin, resolvable by: org.netbeans.modules.maven.problems.ProblemReporterImpl$MavenProblemResolver@5fc9cf62 unresolved

When the final artifact jar contains classes not originating in current project, NetBeans internal compiler cannot use the sources of the project for compilation. Then changes done in project's source code only appears in depending projects when project is recompiled. Also applies to features like Refactoring which will not be able to find usages in depending projects.

Joka tapauksessa kokonaisuus oli hieno, ja tulipa siinä vietettyä pari tuntia pelatessakin.

Koodikatselmointi

Ohjelma ladattu 23.09.16 klo 19.00

Työlään aiheen valitsit, mutta uskomattoman hienon ohjelman olet jo tähän mennessä saanut aikaiseksi. Peli-logiikka ja siirrot toimivat todella hyvin ja olet huomioinut hienosti kohdat, joissa voisi mennä vikaan. Itse olen sen verran kokematon ohjelmoija, etten kovin yksityiskohtaisesti pysty näin monimutkaiseen koodiin perehtymään, ja löytämäni puutteet saattavat jo paikkautua koodilla, jota en ymmärrä. Parser-luokka esimerkiksi on minulle vieras enkä oikein ymmärrä sen toimintaa. Tässä kuitenkin joitain havaintoja ohjelman toiminnasta:

-Pelaan valkoisella ja siirrän sotilaan komennolla (a2-b3) vaikka b3:ssa ei ole mustan nappulaa. Ohjelma toteaa, ettei ruudussa ole syötävää nappulaa (tähän asti siis kaikki ok), mutta antaa vuoron mustalle.
-Antaessani siirron, jossa molemmat tai jompi kumpi koordinaateista on ulkona laudalta, ohjelma vastaa useilla eri tavoilla (laiton siirto, ruutu ei laudalla, ruudussa ei ole nappulaasi) tai ei mitenkään, riippuen koordinaateista. Ohjelman jatkumiseen tämä ei kuitenkaan vaikuta vaan vuoro pysyy valkoisella. Voisiko ongelmatilanteet jotenkin yleistää, mikäli koordinaatteja on muuten vaikea hallita? Esim aina vikatilanteen yllättäessä "Laiton siirto"?
-Onnistuneesta syönnistä ei tule mitään ilmoitusta. Mukava olisi myös nähdä syödyt nappulat jotenkin.
-Shakkia ei tule, jos kuningasta uhataan. (Tämä varmastikin työn alla :) )
-Tällä hetkellä sotilaan merkki on p tai P riippuen väristä. NetBeansissa pieni ja iso p ovat kuitenkin huomattavan samannäköisiä, joten ne on vaikea erottaa toisistaan pelilaudalla ellei ne ole vierekkäin. Toki ongelma poistuu, kun peliin luodaan graafinen käyttöliittymä, mutta tällä hetkellä se haittaa pelimukavuutta. :)
-Pelin alustus tapahtuu tällä hetkellä Main:ssa. Shakissa kuitenkin peli alkaa aina samasta tilanteesta, joten voisiko alustuksen tehdä omassa luokassaan?
-Testejä oli vielä melko vähän, mutta kaikki meni läpi minunkin koneella.

Kiva ohjelma ja kiva oli katselmoida. Tsemppiä jatkoon! :)