Projekti ladattu 7.6.2019 kello 12:10

Tervehdys!

Sain tehtäväkseni arvioida kompressiotyösi. Hyvä niin, sillä teen itsekin harjoitustyön samasta aiheesta.

Ensihuomiot

Luettuani läpi toteutus- ja määrittelydokumentin, sain hyvän kuvan siitä, mitä ohjelmallasi tavoittelet. Ikävää, että olet ollut kipeänä, sillä edistyksesi muutaman ekan viikon aikana on selvästi ollut nopeaa!

Koodi

En ole kovin kummoinen koodari, mutta ohjelmakoodisi näyttää siltä, miltä tiedän hyvän koodin näyttävän. Minun on helppo seurata LZ77 toimintaa, vaikken ole sen teoriaan tutustunutkaan. Luokka-hierarkia on erittäin järkevä ja hilitty, etkä näytä tekevän mitään muistillisesti liian raskaita operaatioita.

HuffmanCompress

Rekursio huffman-koodien tuottamiseksi on todella napakka ja suoraviivainen.

LZ77Compress/Decompress

En aivan ymmärrä LZ77 toimintaa, mutta tässäkin luokassa on pelkästään hyvää ja selkeä koodia.

HuffmanTree

Sama juttu tässäkin, oma puu-totetukseni on huomattavasti sotkuisempi ja useamman kerran pitempi kuin sinun implementaatiosi, hyvää työtä!

Ohjelman toiminta

En osannut ajaa gradle-kokemattomuuttani ohjelmaasi. Kuitenkin siinä onnistuttuani, ihastelin sen tehokkuutta. Ajoin ohjelmasi Moby Dick novellilla ja sain samanasteisen kompressiotehokkuuden kuin mitä Seitsemästä Veljeksestä. Hyvää työtä!

Testaus

Testikattavuusraportti kertoo selkeää kieltä ohjelman tilasta. Testit ovat riittävän kattavia ja järkeviä. Olet ymmärrettävästi jättänyt Huffmanin testaamisen pois, kun et sitä vielä ohjelmassasi käytäkään.

Yleiset kommentit

Olet edistynyt todella hyvin, ja epäilemättä, kunhan saat keon toteutettua Huffmanin puu-tarpeisiin, niin projektisi alkaa hipomaan valmista työtä.

Ainut rakentava palautteeni työhösi ei liity niinkään koodiin, vaan ohjelman käytettävyyteen. Vaikka käyttöohjetta ei olekaan tarvinnut vielä tehdä, ohjelmasi ajoa hieman hidasti se, että minun täytyi itse selvittää ja lisätä build.gradle tiedostoon rivit: apply plugin: 'application' mainClassName = 'jpack.Main'
Hyvää työtä kertakaikkiaan! Monet ratkaisuistasi saavat oman kompressio-ohjelmani näyttämään vitsiltä. Onnea ja menestystä loppukurssille!
Yt. Akseli

Most of the changes since the last code review seem to be focused around UI improvements, so I shall be focusing on that as well. For the most part, the program was easy to use, since all I had to do was to run gradlew run and follow the usage instructions. The only problem with the usage instructions was the fact that it expected me to be running program via a jar file, but I couldn't get gradlew jar to produce a .jar with the correct main class, so instead I had to google how to pass arguments through gradlew run. It would be nice if the README included some information on how you'd like the user to run the program.

Otherwise the usability of the program was just fine, and I managed to compress and decompress the test file using both algorithms without any issues. The jar issue may very well just be me using gradle incorrectly, so as long as it works on your system, it's nothing to worry about.