jaakkolipsanen / imgedit.rs Goto Github PK

Kokeilin ohjelmaa muutamalla kuvalla eri filtterein, ja täytyy sanoa että on kiinnostava idea. Rust on toisaalta hieman vieras kieli itselleni, erityisesti importtien toiminta on hieman kryptistä, mutta noin yleisesti vaikuttaa järkevältä kieleltä.

Itse koodiin liittyen:

• Koodi on erittäin luettavaa, jopa henkilölle joka ei ole koskaan ennen paneutunut Rust-projektiin. Muutama huomio kuitenkin:
  • Joissakin kohdissa olisi toki voitu käyttää vähän kuvaavampia muuttujanimiä, kuten esim. tiedostossa hsl.rs olevassa to_rgb-funktiossa. Tässä funktiossa ei ole oikein selvää mitä x ja m ovat, tai mitä ne tekevät.
  • Koodissa kommentointi on muuten hyvällä tasolla, mutta esim. tiedostoissa hsl.rs ja implementations.rs käytetään jonkin verran "magic numbers", jotka tekevät koodin toimintalogiikasta epäselvän. Esim. unsharp_mask on hieman kryptinen, ja voisi olla selvempi jos kommentissa annettaisiin perustelu numeroiden valinnalle kommentissa linkin lisäksi – linkit eivät välttämättä elä ikuisesti. Muutamaan muuhun funktioon pätee sama asia.
• Projektin rakenne on järkevä, ja toiminnallisuuden erittely projektitasolla selkeä.
• Osittain liittyen rakenteeseen, tuli mieleen pieni kysymys (ja mahdollisesti parannusehdotus): täytyykö testien olla samassa tiedostossa toteutuksen kanssa? Jos ei täydy, niin ehdottaisin siirtämään testit dedikoituun testikansioon samalle tasolle missä src löytyy.
• Pakko mainita että CliArgs on aika näppärä implementaatio structopt:in avulla.

Kaiken kaikkiaan siisti projekti – mielenkiinnolla odottelen miltä projekti tulee näyttämään valmiina!

Ladattu 25.4. klo 17

Vertaisarviointi

Koodi vaikuttaa hyvältä ja hyvin kirjoitetulta, vaikka Rust ei olekaan entuudestaan tuttu, joten vaikea kommentoida muuta kuin pintapuolisesti koodia.

Kokeilin ohjelmaa muutamalla kuvatiedostolla ja se vaikutti toimivan hienosti! Ohjelman kanssa leikkiessä hieman ihmetytti miksi tuki vain png-kuvatiedostoille? En ainakaan nopealla vilkaisulla keksinyt tähän syytä, mutta se voi hyvinkin löytyä syvältä koodin toteutuksesta.

Itse olisin saattanut eritellä filttereiden toteutukset omiin tiedostoihin niin yhdestä tiedostosta ei tule massiivista. Tosin osa filttereiden toteutuksista on hyvin lyhyitä, joten implementations.rs ei ole tällä hetkellä kovin suuri, vaikka testitkin näyttävät olevan samassa tiedostossa.

Koodi oli mielestäni hyvin dokumentoitua, erityisesti filttereiden osalta. JavaDoc-tyylistä kommentointia olisin ehkä kaivannut enemmän, mutta pääsääntöisesti metodit olivat melko simppeleitä ja niistä ymmärsi hyvin mitä ne tekevät.

Nopealla kokeilulla oletin ohjelman olevan nopeampi simppeleissä filttereissä (esim inverse), mutta varmasti esimerkiksi I/O-operaatiot vie oman aikansa joten varsinaista suorituskykyä vaikea kommentoida. Vertailu vaikka ImageMagick-kirjastoon voisi olla mielenkiintoinen! Kokonaisuudessaan projektin rakenne on hyvä ja ohjelman toiminnallisuuteen oli helppo päästä sisään, vaikka kieli olikin vieras. Mielenkiintoinen aihe!