sari-bee / veriryhmaapuri

Ohjelmistotekniikan kurssin projekti; Veriryhmäapuri. Project for software engineering course; Blood group interpreter.
0 stars 0 forks source link

Koodikatselmointi #1

Open theJSZ opened 2 years ago

theJSZ commented 2 years ago

Latasin koodin torstaina 9.12. klo 19:20

En meinannut keksiä tästä ohjelmasta mitään parannusehdotuksia. Matti Luukkainen tarjoutui katsomaan myös mutta ei hänelläkään ollut juuri mitään huomautettavaa. Ensimmäinen asia mitä Matti ehdotti oli että voisi miettiä kannattaisiko verinäytteen oliomuuttujia laittaa dicteihin anti_x ja x_cell mutta myönsi sitten kysyttäessä että ei siitä paljon hyötyä olisi koska niitä muuttujia on kuitenkin niin vähän. Hän oli myös sitä mieltä että jos ohjelman haluaisi kääntää eri kielelle, se olisi tällä rakenteella vaikeaa.

Siitä olin Matin kanssa samaa mieltä että koodissa voisi lukemisen helpottamiseksi olla tyhjiä rivejä sopivissa paikoissa, esim. for-luuppien ympärillä.

Koska en ymmärrä miten noi suositukset kuuluisi antaa, oli vaikea tarkistaa antaako ohjelma oikeita tulosteita. Logiikka näyttää kuitenkin vedenpitävältä ja uskon että se toimii oikein, selvästi tiedät aiheesta. Näytehistoriasta näytetunnisteella hakiessa tuli mieleen että olisiko hyvä jos voisi syöttää vain osan tunnisteesta mutta en tiedä olisiko se ohjelman käyttötarkoituksessa järkevää, onko esim noi tunnisteet jotenkin standardoitu tms, käyttäjä ei varmasti halua että vahingossa lukee väärää näytettä.

Yleisesti ottaen ohjelma näyttää ammattimaiselta ja ei esim. kaadu väärillä syötteillä, yritin saada sitäkin tapahtumaan mutta en onnistunut. Koodissa kaikki on musta tosi hyvää, tyhjillä riveillä voi vielä selkeyttää ulkonäköä.

sari-bee commented 2 years ago

Moikka! Kiitos kommenteista! Tuo koodin selkeyttäminen mm. tyhjiä rivejä lisäämällä on ihan totta, otan sen työn alle. Tuon näytetunnisteen osalta asia on juuri kuten arvelitkin, että olen päätynyt siihen, että haussa vaaditaan täsmällinen näytetunniste. Mietin kyllä toistakin vaihtoehtoa. Mutta tyypillisesti tilanne olisi se, että haetaan täsmällisellä näytetunnisteella tai se luetaan esim. näytetarran viivakoodista. Kommentistasi minulle tuli kuitenkin mieleen, että asia olisi hyvä mainita käyttöohjeessa, joten kiitos!