jarsba / deepchess Goto Github PK

Projekti ladattu ja arvioitu 27.4.2021 klo 17.00

Satunnaisia huomioita

• Joissain luokissa on melko pitkiä metodeja, lähinnä kun etsitään ruudun ympäriltä eri linjoja (esim. kun etsitään tietyltä diagonaalilta jotakin). Niille saisi varmaan omat metodinsa jos haluaisi metodeista lyhyempiä/siistimpiä.

• En ihan tajunnut miksi kuningattaren getPossibleMoves metodeista toinen käyttää ja toinen ei käytä hyödyksi tornin ja lähetin metodeja. (Luulisi olevan mahdollista hyödyntää niitä kummassakin)

• Javan omat util:it käytössä (ArrayList jne). Jos ehtii, niin niistä pitäisi varmaan tehdä omia toteutuksia. (Itsellä sama homma)

---

Ideoita

• Kun olet saanut alpha-beta pruningin valmiiksi, niin yksi melko vaivaton tapa nopeuttaa algoritmiä on 'possibleMoves':in järjestäminen sellaiseen järjestykseen, että capturet ja 'check':it ovat ensin. Tämä ei tosin vaikuta niinkään pelin alkuun, jossa tuntuu menevän paljon aikaa ainakin omassa toteutuksessani.

---

Hyvät

• Nähty paljon vaivaa eri luokkien tekemiseen 'from scratch'
• Kokonaisuudet eroteltu eri luokkiin
• Selkeät luokkien/metodien nimeämiset

---

Enpä keksi mitään syvällisempää kritiikkiä. Onnea projektin viimeistelemiseen!

Project downloaded & reviewed 23.4.2021 around 7am

Dokumentit

• Määrittelydokumentti on selkeä ja sitä on helppo lukea
• Huomaan, että olet kohdannut samoja ongelmia kuin itse rakentaessa logiikkaa --> mikä on laillinen siirto

Koodi yleisesti

• Koodi on selkeää ja sitä on helppo lukea
• Testit jeesaavat kun kehittää uusia ominaisuuksia --> vanha ei hajoa.
• Hyvä ratkaisu rakentaa nappulat ja erittää liikkumislogiikka niihin.

Ehdoituksia

• Suosittelisin lisäämään kommentointeja koodii. Voisit käyttää tähän javadoc. Joskin itsekin tilanteessa, että olen kommentoinut aika vähän koodiin, mitä tapahtuu.
• Mietin että pitäisikö MoveUtils jakaa osiin. Alustavasti kävi mielessä, että sen voisi jakaa, mutta en ihan varma miten.

Yleisesti

• Koodi näyttää selkeältä ja hyvältä
• Fiksusti jaettu tuo logiikka noille nappuloille
• Ei muuta kuin tsemppiä loppu projektiin