Dynamo-projektin 2. koodin katselmointi

Aika: 26.11.2007 14:15-15:33
Paikka: Jyväskylän yliopisto, AgC223.1
Laadittu 27.11.2007

Läsnä:

Tilaaja:

• Riku Ahonen
• Olli Alanen

• Tero Hätinen
• Joni Purojärvi
• Antti Pyykkönen

• Juha Huikari
• Jukka-Pekka Santanen

Katselmointi

Ryhmä oli lähettänyt edellisen katselmoinnin jälkeen muokatun DHCP-aiheen lähdekoodin ja virtuaalisen vierasagentin prototyypin ensimmäisen version lähdekoodin edellisenä perjantaina projektiorganisaation sähköpostilistalle. Näin kaikilla oli mahdollisuus tutustua koodiin ja tehtyihin muutoksiin jo etukäteen.

Katselmoinnissa Purojärvi esitteli oleellisia kohtia lähdekoodista. Katselmoinnin aikana keskustelu oli vapaata.

Huomioita ja parannusehdotuksia

Katselmoinnin aikana tuli esiin seuraavat huomiot ja parannusehdotukset:

• Funktioden kommentointi tulee laatia siten, että niistä saadaan tuotettua dokumentaatio doxygen-ohjelmalla.
• Lähdekoodin sisennykset tulee yhdenmukaistaa ja vaihtaa tabulaattoreiden sijaan kaksi välilyöntiä.
• Liian pitkät rivit tulee katkaista.
• atm_mobiles ja selected -muuttujien nimet tulee vaihtaa kuvaavampiin.
• Virtuaaliseen vierasagenttiin tulee koodata tietorakenne ja oma aliohjelma, joka hoitaa asetustiedoston luvun.

Katselmoinnin aikana tuli esiin seuraavat kehitysideat, joita tuskin tullaan toteuttamaan projektin aikana, mutta tullaan dokumentoimaan jatkokehitysideoihin.

• Sisäkkäiset monimutkaiset ehtolauseet voisi siistiä.
• Voisi toteuttaa tarkastuksen, joka tutkii onko asetustiedostosta saatu interface validi.
• SPI-tarkastuksen voisi muuttaa asetustiedostosta luettavaksi realmin tarkastukseksi.

Seuraava palaveri

Katselmoinnissa sovittiin seuraavan palaverin ajankohdaksi 5.12.2007 klo 9:30.