samuel0c/dna-converter

dna-converter's Issues

Koodikatselmointi

Tosi hyvän näköinen projekti!

Ainut mitä muuttaisin hieman olisi
private String codonToAminoAcid(ArrayList codon) {
metodi

Case voisi vähän selventää metodin toteutusta, mutta muuten aivan loistavaa työtä! :)

Koodikatselmointi

Ensi huomiot

En ikävä kyllä tiedä aiheestasi juuri mitään, mikä saattaa hiukan vaikeuttaa ohjelman ymmärrettävyyttä minulle.

Koodi

Keskityn tässä logiikan koodiin, koska ymmärtääkseni UI luokille ei ole asetettu clean code vaatimuksia.

Luokka: Protein
Rivi: 42:
Metodi Clean code periaatteita pidempi. Sen voisi jakaa osiin, esim while loopin käydä läpi toisessa metodissa.

Rivi: 76:
Myös tämä metodi liian pitkä, mutta ymmärrän, että aivan joka tapauksessa ei voida pysyä tuossa 10 rivin säännössä ja tämä saattaa hyvinkin olla tuollainen tapaus.
Eikö tämän metodin olisi voinut tehdä switchiä käyttäen if:n sijasta? Se olisi voinut olla siistimpi ratkaisu.

Katselmoinnin jälkipuinti

Projektissasi ei ole kovin paljoa koodia (mikä ei ole huono asia), ja se on lähes kokonaan siististi tehty clean code periaatteiden mukaan. Tämä teki koodin arvioinnista erittäin helppoa. Huomioimani pari metodia eivät välttämättä ole huonoja ja sinä itse tiedät paremmin, voiko niitä ylipäänsä parantaa neuvojeni mukaan. En myöskään lupaa, että neuvoni ovat päteviä.

Siisti ja näpäkkä projekti. Kumpa olisin itsekin pitäytynyt hiukan lyhyemmässä aiheessa!

Koodikatselmointi

Edellisellä katselmoijalla on näköjään mennyt koodit sekaisin,, mutta anyways tässä nyt vielä toinen review DNA-converter koodista.

19.02.2017 | 18:00

Logiikka paketissa olevat luokat on nimetty selvästi, ja niistä huomaa heti mitä asioita luokat pitävät sisällään, myös muuttujat on nimetty asianmukaisesti. Protein.java luokassa Stringin palauttava metodi codonToAminoAcid(); on suhteellisen pitkä ja sen aikavaatimus voi olla turhan iso. Metodin lyhentämiseksi tekstimuuttujat voisi mahdollisesti käydä läpi myös Hashmap listana, kun syötteeksi antaa tietyt merkkijono yhdistelmät sen avaimena.
Muut luokat ovat Clean code:n mukaisia ja alle 100 riviä, mikä tekee koodin lukemisesta helppoa, vaikka en itse tunne projektin aihetta kovinkaan hyvin. Missään luokassa ei ole turhaa koodia ja vaikka sen saisi niputettua vieläkin sievemmäksi, on kokonaisuus tälläisenäänkin täysin toimiva.