Koodikatselmointi
Koodikatselmointi on osa harjoitustyötä. Katselmointeja on kurssilla kaksi ja ne suoritetaan kurssiviikoilla 3 ja 5. Katselmoinneista voi saada maksimissaan 2 lisäpistettä, joilla voi korvata puutteita kurssin muilla osa-alueilla. Kurssin maksimipistemäärä on siis edelleen 60.
Katselmoinnin ajankohta
Koodikatselmointeja on tällä kurssilla kaksi:
- Koodikatselmoinnin DL on viikolla 3: ti 18.11. klo 23:00
- Koodikatselmoinnin DL on viikolla 5: ti 2.12. klo 23:00
Tämän katselmoinnin perimmäinen tarkoitus on oppia hahmottamaan toisen henkilön kirjoittamaa koodia ja kokonaista ohjelmointiprojektia. Toisten koodia luetaan kursseilla toistaiseksi valitettavan vähän, vaikka sujuva koodinlukutaito on välttämätön taito ohjelmoijan työssä. Hyvin suoritettu katselmointi on tehokas tapa havaita ongelmakohtia ja korjata virheitä mahdollisimman varhaisessa vaiheessa ohjelmointiprojektia.
Ohjeet
Näet sinulle osoitetun katselmointikohteen repositoriolinkin labtoolista koodikatselmoinnin käynnistyttyä http://tsoha-labtool.herokuapp.com/mypage. Sinun tehtävänäsi on lukea läpi toisen opiskelijan harjoitustyö ja antaa siitä rakentavaa palautetta. Ohjelmaa kannattaa kokeilla myös esittelysivulla annetuista linkeistä. Jos projekti on kuitenkin täysin tyhjä runko, niin ota yhteyttä ohjaajiin, niin valitsemme sinulle toisen katselmointikohteen.
- Lataa zip-pakattu projekti koneellesi annetun repositorion ZIP -painikkeesta
- Ota talteen kellonaika ja päivä, jolloin latasit projektin
- Pura projekti
- Lue ensimmäiseksi projektin aihemäärittely
- Avaa projektin esittelysivu ja testaa projektia ensin selaimessa
- Tutustu mahdollisimman kattavasti ohjelmakoodiin
- Yritä ymmärtää, mitä mikäkin luokka ja metodi tekee
- Ole sinnikäs: Kaikkea ei aina osaa, eikä tarvitsekaan ymmärtää!
- Haastavin osuus lienee eri luokkien ja sivujen keskinäiset suhteet. Käytä hyväksesi toisen piirtämiä luokka- ja käyttöliittymäkaavioita
Kun olet tutustunut riittävän tarkasti toisen projektiin, on aika antaa toiselle henkilölle palaute. Palautteessa ei tarvitse ottaa kantaa ohjelman dokumentointiin, ulkonäköön tai toimivuuteen. Tärkeintä on kiinnittää huomiota Ohjelmoinnin Perusteissa, Ohjelmoinnin Jatkokurssilla, Ohjelmistotekniikan menetelmissä, sekä Tietokantojen perusteet -kurssilla opittuihin hyviin käytänteisiin sekä tällä kurssilla noudatettaviin Clean code -periaatteisiin.
Palautteenanto
Palautteessasi kerro toiselle opiskelijalle, missä on parannettavaa. Mitä tarkempi palaute, sen arvokkaampaa se on palautteen saajalle. Osatessasi voit antaa vinkkejä siitä, miten asioita voisi tehdä toisin tai onko ohjelman tai tietokannan rakenteessa parannettavaa. Jos havaitsit selkeitä bugeja tai virheellistä koodia, kerro siitä. Kannattaa kertoa myös, jos jotkin ohjelman osat eivät toimineet. Tarkoitus ei ole tehdä labtoolin-tyylisiä hieman pintapuolisia mainintoja, vaan tutkia koodia tarkasti ja antaa täsmällisiä parannusehdotuksia. Muista myös positiivinen palaute!
Palaute annetaan Githubin Issuena:
- Mene selaimellasi toisen opiskelijan repositorioon
- Valitse välilehti Issues
- Valitse oikeasta reunasta New Issue
- Anna otsikko “Koodikatselmointi”
- Kirjoita palautteesi kommenttilaatikkoon, Preview -välilehdestä näet palautteesi ulkomuodon
- Kommenttilaatikon yläpuolella on linkki tekstin muotoiluohjeisiin
- Palautelaatikko tukee myös kuvia
- Varmuuden vuoksi kopioi kirjoittamasi palaute koneellesi tekstitiedostoon
- Lisää palautteen alkuun päivä ja kellonaika, jolloin latasit annetun projektin
- Lähetä palaute valitsemalla Submit new issue
Issue eli tässä tapauksessa palaute on koko repositorion tapaan julkinen, joten sen voi lukea kuka tahansa. Ohjaajat lukevat ja pisteyttävät annetun palautteen mahdollisimman pian deadlinen jälkeen.
Katselmoinnista jaetaan 0-2 pistettä. 5-10 laadukasta rakentavaa palautekommenttia riittää 2 pisteeseen. Palaute kirjoitetaan vapaamuotoisena, mutta yritä kirjoittaa selkeästi. Jaa palaute eri luokista eri kappaleisiin. Jos kirjoitat ranskalaisia viivoja, kirjoita kokonaisia lauseita tai mieluiten useita lauseita. Älä hyökkää palautteessa toista kohtaan! Toisaalta älä ota annettua palautetta itseesi - projektisihan on yhä kesken ja muokattavissa. Palautteen antajakin toisinaan ymmärtää väärin tai antaa virheellisiä ohjeita.
Huom. Katselmoinnin kohteen projekti saattaa elää tai päivittyä sillä aikaa, kun kirjoitat palautetta. Tämän vuoksi palautteeseen liitetään aika, jolloin projektin kävi lataamassa itsellensä. Muulla tavalla asiaan ei kiinnitetä tällä kurssilla huomiota, eikä katselmoijan tarvitse aktiivisesti yrittää tarkistaa, jos jokin asia onkin ehtinyt jo muuttumaan. Palaute voi siis olla osin jo vanhentunutta.