jakaarl/2015-periodi-3

TiRan harjoitustyö

2015-periodi-3's Issues

Koodikatselmointi

Projekti ladattu 3.3.2015 kello 17.15

En saa ohjelmaa jostakin syystä ajettua. Toisaalta näin on käynyt aikaisemminkin. Koodi sen sijaan on toimivan näköistä ja sitä on runsaasti. Työn ideakin on mielenkiintoinen ja olisin mielelläni nähnyt sen toiminnasssa. Testejäkin on kirjoitettu enemmän kuin jaksan lukea läpi ja kaikki menevät hienosti läpi. Muutamassa niistä oli kommenteissa hyvät selvennykset, jotka nopeuttivat ja helpottivat niiden tarkoituksen ymmärtämistä. Niitä olisin kaivannut, muutamaan kohtaan varsinaista koodiakin. Toisaalta bonuksiahan nekin olivat. Testien nimet kertoivat oleellisimman.
Selvennyksen vuoksi jotkin metodit voisi nimetä hieman selvemmin. Tällä tarkoitan niitä metodeja, joita on luokassa kaksi tai useampi samanlaista. Esim eri remove-metodit voisi nimetä vaikkapa: removeElement ja removeIndex. ArrayListin growthFactoria käytetään yhdessä kohtaan koodia ja sen voisikin tilatehokkuuden kannalta poistaa, ellen sitten aivan väärin ole ymmärtänyt sen toimintaa.
En löytänyt dokumentteja mistään. Ja vaikka readMe olikin hyvin selventävä, olisi käyttöohjeet olleet minulle hyödylliset... Hämmennyin myös SuppressWarningsin käytöstä ja nopean googlauksen jälkeen selvisi, että niihin kuuluisi laittaa selventävät kommentit mukaan. Tähän en ota kuitenkaan sen kummemmin kantaa kun ei oma mielipiteeni ole. Sen sijaan BoundCuben koordinaateissa Y:n oli kerrottu vähenevän kun lähestytään BottomRightCorneria. Koodi oli oikein, eli Y-koordinaatti nousi, mutta kommentin voisi korjata, etteivät muut hämäänny.
Kaiken kaikkiaan monipuolinen ja enimmäkseen selkeä projekti. Harmi etten saanut käynnistymään.