tobtobtob
commented
11 years ago Kiitos palautteesta. Tosiaan, käyttöliittymä oli lataushetkellä vähän vaiheessa, mutta tuota poikkeuksen heittämistä en ollut vielä huomannut! Se oli arvokas löytö, olisi nimittäin voinut päätyä lopulliseen koodiin tuo bugi.
-topi
ladattu 8.1 klo 14:13
Lataushetkellä käyttöliitymä vaikutti olevan työstövaiheessa joten uskon suurimman osan huomaamistani asioista olevan palautushetkellä jo korjattu.
Peli näytti toimivan hyvin, palikat liikkuivat, tuhoutuivat ja pisteet kasvoivat. Tuloslista ei vielä nähtävästi ollut implementoitu käyttöliittymään, mutta sen koodi näytti olevan hyvä.
Ainoa poikkeus jonka sain aikaan tuli kun ruutu täyttyi ja peli pysähtyi ja koitin liikuttaa palikkaa, antoi tällöin null pointteria. Korjautuisi varmaan helposti poistamalla pysäytyshetkellä näppäimistönkuuntelija käytöstä. Liittyy varmasti keskeneräisyyteen ja korjaantuu tulosten implementoinnin yhteydessä.
Koodi oli selkeää ja helposti luettavaa sekä hyvin jäsennelty, jopa käyttöliittymän metodit pysyivät kivan lyhyinä. Koodi noudatti selkeästi clean code periaatteita. Metodit ja luokat vaikuttivat noudattavan hyvin single responsibility principleä ja metodit olivat sopivan mittaisia ja loogisia.
Testejä alkaa mielestäni olemaan jo reilusti sekä kaikki testit toimivat ja näyttivät testaavan järkeviä asioita.
Kaaviot olivat selkeitä ja antoivat hyvän kuvan pelin toiminnoista ja helpottivat pelitoiminnallisuuden ja rakenteen ymmärtämistä huomattavasti. Melkein kaikkia metodeja olikin jo keretty kommentoimaan mikä helpotti huomattavasti osaltaa ymmärtämistä.
Tetris vaikutti mielestäni hyvin tehdyltä enkä löytänyt mitään ihmeellisempää huomautettavaa, yllättävää oli löytää vielä 10 min jälkeen pelailemasta peliä. :) Hyvän näköistä koodia ja hyvännäköinen käyttöliittymä.