Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Toinen koodikatselmointi #2

Open
Jonharju opened this issue Oct 8, 2015 · 0 comments
Open

Toinen koodikatselmointi #2

Jonharju opened this issue Oct 8, 2015 · 0 comments

Comments

@Jonharju
Copy link

Jonharju commented Oct 8, 2015

Ladattu 08.10.2015 kl 11.16

Olet jaotellut koodin selkeästi mainiin ja logiikka pakettiin, jossa kaikki kolme luokkaa hoitavat selkeästi omat hommansa. Koodi on siistiä ja selkeätä, sitä on helppo ja mukava lukea. Myöskin kommentteja löytyy kivasti välistä selventämässä. Myöskin algoritmit ja tietorakenteet vaikuttavat hoitavan tehtävänsä hyvin kyseiseen tarkoitukseen. Testejäkin löytyy kaikelle mitä tarvitsee testata.

Luokassa Pelimekaniikat metodi kumpivoitti, alustat aluksi voittajan arvolla 9 ja talletat siihen uuden arvon, jonka palautat samalla. Tämän voisi hoitaa niin, että alustat int voittaja; , eli ilman arvoa, ja sitten if tapauksissa asetetaan sen arvo, niinkuin teet nyt, mutta suorittaa returnin vasta metodin lopussa, yhteisenä kaikille tapauksille.

Luokassa Tekoaly, metodi paras valinta vosii ehkä jakaa kahteen metodin, että joko parhaan indeksin etsiminen tai tai parhaan voittavan valinta olisi oma metodi, jota vain kutsutaan sitten siellä. Myöskin jokin haamukoodi on jäänyt kommentiksi luokkaan, mutta sillä ei ole mitään käytännön merkitystä.

Tällä hetkellä tekoäly siis tekee valintansa sen perusteella mitä on pelattu eniten edellisen jälkeen. Mietin voisiko tekoäly esimerkiksi tutkia miten pelaaja reagoi tekoälyn valitsemaan merkkiin. Voisit mahdollisesti tehdä toisella lailla toimivan tekoälyn, esimerkiksi pelaajan viime liikettä matkivan tai aina satunnaiseti valitsevan, ja vertailla niiilä miten hyvin erilaiset tekoälyt toimivat.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant