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

Ensimmäinen koodikatselmointi #1

Open
jan-aleksi opened this issue Sep 23, 2015 · 0 comments
Open

Ensimmäinen koodikatselmointi #1

jan-aleksi opened this issue Sep 23, 2015 · 0 comments

Comments

@jan-aleksi
Copy link

22.9.2015 14:00

Ensinnäkin, koodisi on hyvin selkeää ja helppolukuista. Ei ole ollenkaan vaikeaa ymmärtää, mitä olet tehnyt tai mihin pyrit. Erityisesti kommentit koodin välissä helpottavat koodin lukua huomattavasti. Tämä on yksi asia, josta voisin itse ottaa oppia.

App-luokkaa voisi jakaa palasiksi. Tekisin Main-luokasta mahdollisimman pienen. Esimerkiksi syötteen kelvollisuuden analysointi voitaisiin hoitaa erillisessä metodissa, samaan tapaan kuin alkuhöpinät hoidetaan. Myös esim. tulosten printtauksesta voisi hyvin tehdä oman metodin. Tekisin vaikka jonkin "Peli" -luokan, joka hoitaa koko pelin toiminnan, jonka Main-luokka vain käynnistää.

Ohjelman tekoäly on vielä melko alkeellinen. Yksi ehdotus sen parantamiseksi olisi analysoida, kuinka pelaaja reagoi pelin tulokseen. Jos pelaaja voittaa, vaihtaako hän valintansa? Toinen ehdotus olisi analysoida, miten koneen valinta vaikuttaa pelaajan valintaan. Jos kone valitsee paperin, valitseeko pelaaja seuraavassa pelissä sakset? Vai yrittääkö pelaaja kenties jotenkin ennakoida koneen valintaa? Tätä voi tietenkin analysoida pitkään.

Tällä hetkellä tietorakenteet ja algoritmit ovat tehokkaita, ja ohjelma näyttää kevyeltä ja tehokkaalta. En itse ainakaan huomaa mitään parannettavaa siltä osaa.

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