Koodikatselmointi 2

[Torppo]

21.04.2013 klo 9.41

Terve!

Aiheenmäärittely oli loistava. Se oli selkeä ja antoi selkeän kuvan siitä mitä ollaan tekemässä ja mitä toiminnallisuuksia uutistenlukijaan olisi tarkoitus tehdä.

Itse rakennetta oli todella hankala kuitenkin nähdä luokkakaaviosta sekä itse koodista. Luokkakaaviota tehdessä olisit voinut kiinnittää enemmän huomiota sen siisteyteen ja käsialasi selkeyteen. Kaavio voisi olla myös täysin englanniksi kuten koodisi. Kaaviotasi on hankala lukea, koska luokat on nimetty englanniksi ja riippuvuudet ovat suomeksi. Sekvennsikaavioista olisi apua luokkiesi metodien ymmärtämisessä.

Koodisi sisältää paljon webiin liittyviä toiminnallisuuksia johon en ole itse tutustunut ajemmin. Tämä tekee itse toteutuksen minulle hankalaksi ymmärtää, mutta uskon, että sinä olet perillä asioista ja ne toimivat. Koodi oli ainakin hyvin siistiä.

Minulla on myös ongelma ohjelman ajamisen kanssa, koska omasta NetBeans java SE:stä puuttuu ohjelmasi kaipaamia tiedostoja, kuten selaintoiminnallisuuteen liittyviä tiedostoja. Olisi mukava, jos mainitsisit tälläisten standard editionista puuttuvien tiedostojen tarpeesta dokumentoinnissa ja madollisesti kertoisit mistä ne voisi saada. Se helpottaisi varmasti hyvin paljon ohjelmasi ylläpitäjien elämää.

Ymmärsin, että ohjelmasi hakee Reddit-sivustolta uutisia, jotka PostProcessor ja PostHandler -luokat käsittelevät niin, että ne voidaan näyttää lukijalle graafisen käyttöliittymän kautta. Jos osuin oikeaan niin hyvä. Mielelläni ymmärtäisin vähän enemmänkin. Jos ei mennyt lähellekkään, niin lisää selkeää dokumentointia, kiitos. Suurin osa luokka- ja metoidkuvauksista puuttuivat muutenkin.

Toivottavasti tästä oli apua! Mukavaa jatkoa!