tiralabra-mnkgame's People
tiralabra-mnkgame's Issues
Koodikatselmointi
TiRa-labran koodikatselmointi 1
Denopia / tiralabra-mnkgame
katselmoijana xvixvi
ZIP ladattu ajassa 16:13 perjantaina 25.08.
Määrittely
Määrittelyssä silmiin pistää ensimmäisenä ohjelmoitavan tapauksen monimutkaisuus: tekijä kertoo suunnittelevansa 15x15 ruudukon toteuttamista ensitapauksena. Itse ehkä lähestyisin ensin mahdollisimman yksinkertaisesta tapauksesta, laajentaisin sitten. Toisaalta: mitä väliä?
Muuten määrittely on hyvää mutuilua, tosin en itse pystynyt edes näin konkreettiseen pohdintaan. Tuli mieleen pitäisikö määrittelydokumenttia päivittää nyt projektin edettyä pidemmälle, vai ajavatko toteutus- ja testausdokut asian?
Ohjelman ajaminen
Yksinkertainen Main luokan pyöräyttäminen IDE.llä käyntiin. Hieman jäi häiritsemään graafisen käyttöliittymän puuttuminen, mutta toisaalta algoritmejähän tässä ollaan tekemässä. Tekstikäyttöliittymä on selkeä ja yksinkertainen. Siirtojen syöttäminen on siedettävää tekstikäyttöliittymästä huolimatta, ja idea korostaa viimeisimmät siirrot toimii hyvin: tekee pelistä seurattavampaa.
Koodin laatu ja selkeys
Tekoälyluokan rakenne on selkeä, min-maxiin ei ole lisättävää. koodi on mukavasti strukturoitua. Luokka on ehkä vähän pitkä (+250 riviä) ottaen huomioon, ettei kommentteja ole juurikaan. Joitain metodeja olisi voinut kommentoida paremmin, esim pisteytysfunktion eri osat tai että tarkastaako seuraava metodi laudan reunat tyhjinä vai ei:
/**
* Tarkastaa etta ruudun ymparilla ei ole vain tyhjia ruutuja.
*
* @param gb Peliruudukko
* @param row Tarkasteltavan ruudun rivi
* @param col Tarkasteltavan ruudun sarake
* @return true jos on yksinainen ruutu, false jos ei ole
*/
private boolean notLonely(int[][] gb, int row, int col) {
if (row > 0 && gb[row - 1][col] != 0) {
return true;
}
if (col > 0 && gb[row][col - 1] != 0) {
return true;
}
if (row > 0 && col > 0 && gb[row - 1][col - 1] != 0) {
return true;
}
if (row < gb.length - 1 && gb[row + 1][col] != 0) {
return true;
}
if (col < gb[0].length - 1 && gb[row][col + 1] != 0) {
return true;
}
if (row < gb.length - 1 && col < gb[0].length - 1 && gb[row + 1][col + 1] != 0) {
return true;
}
if (row < gb.length - 1 && col > 0 && gb[row + 1][col - 1] != 0) {
return true;
}
if (row > 0 && col < gb[0].length - 1 && gb[row - 1][col + 1] != 0) {
return true;
}
return false;
}
Monet metodeista ovat myös privaatteja, eli niitä ei pysty erikseen testaamaan. Tarkoituksenmukaista?
Yhteenveto
Selkeä, kompakti projekti. Hieman olisi voinut kommentoida auki esim pisteytysfunktion osia, kyseessä kuitenkin älyn mielenkiintoisimpia ja merkittävimpiä metodeja (min-maxin toimimisen lisäksi).
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.