Koodikatselmointi

[samitsal]

Moi,

Kloonasin reposi tänään torstaina 12.2.

Kuvanpakkaus ei aihepiirinä ole itselleni lainkaan tuttu, joten pystyn valitettavasti kommentoimaan työtäsi vain hieman karkeammalla tasolla.

Sitten asiaan: Koodisi on todella selkeää ja nätisti jäsenneltyä. Voisi ehkä olla mielekästä muuttaa muutamat suomea sisältävät muuttujanimetkin englanniksi, niin kokonaisuus olisi koherentimpi. Toiminnallisuuden jakaminen luokkiin hyvä, samaten JavaDoc-kattavuus.

Dokumentaation puolesta täytyy kehua etenkin määrittelydokumenttia, joka kiteytti mielestäni mukavasti alkuperäisen ongelman sekä sen, kuinka olet lähtenyt sitä ratkomaan. Propsit myös selkeistä käyttöohjeista. Tosin voisi olla hyvä mainita, että convert-komennon käyttö vaatii ilmeisesti ImageMagickin.

En ikävä kyllä pystynyt ajamaan ohjelmaasi sillä Decoder-luokan BinaryIn-tyyppistä muuttujaa b ei voitu alustaa ((cannot find symbol) ehkä puuttuva import?) kun yritin ajaa ohjelmaa NetBeansissa, ja valmiiksi paketoitua jar-tiedostoa tai vastaavaa en löytänyt.

• Sami