laimikko1/jmtetra

Jmtetra

Dokumentaatio

Aiheen kuvaus

Käyttöohjeet

Tuntikirjanpito

JavaDoc

Pit

CheckStyle

jmtetra's Issues

Koodikatselmointi..

..versiolle 201702151722

Tähänhän jää koukkuun! Varoitan jo etukäteen, että käytän käsitteitä tetromino ja pala sujuvasti synonyymeinä, vaikka oletkin käsitteellistänyt palan tetrominon osaksi (ja tämä on ihan oikein).

Ohjelmoinnin perusteet, Ohjelmoinnin jatko- sekä Ohjelmistotekniikan menetelmät -kursseilla opitut hyvät käytänteet sekä Clean code -periaatteet

Yleensäkin koodi on hyvin jäsenneltyä (pakkaukset) sekä luokat, muuttujat ja metodit hyvin nimetty lukemista helpottamaan. Lisäksi this- ja super-avainsanojen käyttö auttavat tässä. Mielestäni perintää ja toteuttamista on käytetty tarkoituksenmukaisesti. Erityisen mielekästä on lukea javadocitettuja käyttöliittymäluokkia.

Tetrominojen createRotations-metodeissa if-ketjun voisi korvata if..elseif-ketjulla. Jokaisella Tetronome-ilmentymällä kun on vain yksi rotation-arvo kullakin hetkellä. Vähän vastaavasti voisi menetellä InputListener-luokan keyPressed-metodissa.

Koodissa ei ole havaittavissa merkittäviä koodihajuja. Mainittakoon kuitenkin Tetronome-luokan perillisissä toistuvat updateXCords-metodit, jotka vaikuttavat kömpelöiltä. En kyllä keksi tähän parannusehdotusta...

Raportit paljastavat testien kattavuuden. Testit on nimetty kuvaavasti ja testaavat pääasiassa yhtä asiaa/kokonaisuutta kerrallaa. testBoard-taulukot ja assertArrayEquals havainnollistavat testattavissa olevaa ominaisuutta.

Pikkujuttna: Luokkien JavaDocien @author-tunnisteessa on välillä Mikko, välillä mikko ja joskus laimikko. Yhdellä luokalla ei ole @author-tunnistetta. Sama henkilö kun lienee kyseessä, olisi johdonmukaista käyttää samaa nimimerkkiä kaikkialla.

Ohjelman dokumentointi, ulkonäkö ja toimivuus (eli ylimääräistä)

Pelin päättyessä peli loppuu kuin seinään. Tähän tullee jokin JoptionPane.showMessageDialog lyhyellä ”Game over”-viestillä ja ”New game”-painikkeella.

Koin painikevalinnat hieman epäintuitiivisiksi, mikä johtunee pitkästä tetris-tauosta; viimeisintä pelaamaani toteutusta pelattiin pelkästään nuolinäppäimillä – ja seuraavaksi arvottu pala näytettiin etukäteen. Kuitenkin loppujen lopuksi putoamisen kiihdyttäminen alas-painikkeella ja pudottaminen välilyönnillä on hyvä idea. Pudottaminen ei lukitse palan paikkaa sivusuunnassa, vaan palaa pystyy vielä siirtämään sivusuunnassa välilyönnin painamisen jälkeen! Tämä lienee korjattu viimeisimmästä koodikatselmoinnista?

Hieman häiritsee, että esimerkiksi I-tetrominoa ei pysty kääntämään ennen kuin se on pudonnut tarpeeksi, tai siis ennen kuin palalla on tarpeeksi tilaa kääntyä. Tämä tosin taitaa olla ”by design”. I-tetrominoon liittyen, jos palan pudotessa pala käännetään pitkittäin, siirretään pelialueen laitaan ja painetaan kiertopainike pohjaan, pala alkaa välkkyä, mikä luo illuusion palan liikkumisesta ylöspäin. Tämä tosin lienee vain kosmeettinen seikka eikä sillä ole itse pelin kannalta mitään merkitystä.

---

Odotan innolla pisteytysjärjestelmän implementointia. Pisteytysmenetelmiä kun on niin monia mistä valita…

Koodikatselmointi

Jmtetra-master ladattu 5.2.2017 17:17

Pelin perusmekaniikka toimii kuten pitääkin siltä osin mitä koodattu. Lisää ominaisuuksia tullee ajan myötä, mutta tähän astisessa työssä ei ole moitteita.

• Koodi on selkeää ja helposti ymmärrettävissä ilman kommenttejakin.
• Metodit pidetty lyhyinä ja nimeäminen luontevaa
• Raportointi täyttää kaikki kriteerit (niin oma kuin generoidut raportit)
• Testiluokat kattavia
• Pelaillessa harmitti vain palikan osuessa lattiaan palikkaa ei voinut enää liikuttaa yhtäkään pykälää, vaikka tilaa olisi ollut