tetris's Issues
Koodikatselmointi
Projektin zip ladattu 13.8.2016 noin klo 20.30.
Harmi kun ohjelma ei ole vielä ajettavassa kunnossa, olisi ollut kiva päästä kokeilemaan.
Erimuotoiset tetrominot voisivat olla Tetromino-luokan aliluokkia, jolloin ei tarvitse montaa eri metodia Tetromino-luokassa, vaan aliluokat voisivat syrjäyttää uuden tetrominon luovan metodin omalla toteutuksellaan.
Blockin värit voisivat olla esim. BlockColor-nimisessä enumissa, jolloin vaikkapa punainen olisi BlockColor.RED.
Metodit ovat sopivan lyhyitä ja niiden nimet ovat kuvaavia. Jokaisella metodilla on myös selkeästi oma vastuunsa.
Pelialustan piirtämisen voisi eriyttää omaksi luokakseen. Tosin kun graafinen käyttöliittymä tulee mukaan kuvioihin, niin eiköhän se päädy omaan luokkaansa.
Hyvää työtä! Jatka samaan malliin.
Koodikatselmointi
Koodikatselmointi: Projekti ladattu 22.08.2016 klo 18:18
Koodi on selkeää, metodit ja tekijät ovat nimetty todella hyvin, mikä helpottaa ymmärtämistä. Koodi on jaettu hyvin paketteihin ja selkeisiin luokkiin.
Luokkakaaviosta olen hiukan epävarma, että onko se oikein? Siinä muodostuu turha silmukka, eli luokka Game voisi saada tiedon tetronimosta Boardin kautta.
Jotta koodia olisi vielä helpompi ymmärtää, voisi sitä komemmentoida enemmän. Lisäksi Javadocin @param ja @return selitykset auttavat hahmottamaan tietyn metodin toimintaa.
Tällä hetkellä luokka Game vaikutti hiukan turhalta. Boardi näköjää tuntee tetronimot ja sitä kautta Blockit. Lisäksi Tetris luokan voisi siirtää gui pakettiin.
Kokonaisuutena projekti vaikuttaa hyvältä ja mielenkiintoiselta. Olisi kiva nähdä toimivuutta, kun projekti on valmis. Lisäosia en osaa ehdottaa, kun en vielä nää pelin graafista toimintaa.
-Jonna :)
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.