Skip to content

Code Review #1

@jklongi

Description

@jklongi

Ohjelma näyttäisi olevan hyvässä vaiheessa vaikuttaa
erittäin selkeälukuiselta. En kuitenkaan itse ohjelmaa
saanut pyöritettyä, koska se vaati jotain erikoisia
importteja. Myös kummastusta aiheutti tuo JavaApplication9 ym?
Decoderin koosta päätellen ohjelma ei vielä myöskään taida
purkaa :)

Kuitenkin mitä itse koodiin tulee, metodit on hyvin nimetty
ja koodi on hyvin selkeälukuista. Kommenteissa selitetään eri
"taikaluvut" mistä ne tulevat, sekä metodin toiminta hyvin. Koodia
on helppo ulkopuolisenkin lukea, koska kaikki on kommentoitu
hyvin laajalti metodeista muuttujiin. Copy pasteja yms en huomannut,
mutta jotkin metodit ovat melko pitkiä esim encoderissa encodeData
on todella pitkä ja luokkakin aika iso, mutta toisaaalta hyvästä syystä
sillä se tekee suhteellisen paljon. Pilkkominen osiin helpottaisi toki
testausta sillä testejä näyttäisi muutenkin olevan aika niukasti?

En ymmärtänyt toisaalta kaikkea toimintaperiaattesta tällä
tutustumisella, mutta toisaalta aihe on monimutkainen ja
pitäisi ymmärtää algoritmin toiminta ennen koodiin tutustumista
ehkä paremmin. Paremmalla perehtymisellä pystyisi mahdollisesti
jatkamaankin koodia.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions