nimirum/miinaharavanratkaisjia

Tietorakenteiden harjoitustyö 2015

miinaharavanratkaisjia's Issues

Koodikatselmointi 1

Koodikatselmointi 1

En ole ikinä oppinut pelaamaan miinaharavaa joten aiheesi oli minulle erityisen mielenkiintoinen. Huomasin, että Algoritmit ja tietorakenteet -harjoitustyösi oli osa ohjelmoinnin harjoitustyötä. Vaikka tämä ei varsinaisesti kuulunut koodikatselmointiin niin katselin myös sinun harkkatyötäsi innolla läpi.

Kansiorakenteesi oli todella hyvä ja selkeä. Määrittelydokumentti kertoi selkeästi projektista, sekä lyhyesti sen miten ohjelma toimii, sekä lyhyesti myös algoritmien aikavaativuudesta.Mainitset tämän lisäksi lähteitäsi, joka kertoo siitä, että olet perehtynyt asiaan. Vilkaisin myös niitä läpi ihan mielenkiinnosta.

Viikkoraportit ovat selkeitä ja lukaisemalla ne läpi selviää nopeasti mitä teit jokaisella viikolla ilman että varsinaista koodia tarvitsee avata ja lukea. Viikkoraporteissasi oli myös mukana kuvia jotka selvittivät suunnitelmiasi ja pohdintojasi syvemmin.

Huomasin, että muutamista metodeista puuttui kommentointi, esimerkiksi doubleCapacity() metodi ArrayDeque luokassa sekä muutama metodi luokassa SuorituskykyTestaaja. Tämän lisäksi kielesi vaihtelee, välillä koodi on suomeksi, välillä taas englanniksi. Nämä kuitenkin ovat niin pieniä asioita, että saat korjattua ne todella helposti, jos tarvitsevat edes korjausta.

Mielenkiintoista oli myös huomata, että toteutit ArrayListin itse sillä tämä taisi olla kiellettyä käyttää valmiina, koska esimerkiksi itse jätin Arraylistin kokonaan käyttämättä. En ole aiemmin nähnyt vastaavaa ja opin tästä myös itse uutta.

Tsemppiä projektin loppuvaiheeseen!