coderb0b/tetris

tetris's Issues

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 :)

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.