Potkuri-projekti	TARKASTUSPÖYTÄKIRJA
			4.11.2008


 
Tarkastuksen aika ja paikka 4.11.2008
klo 12.49- salissa A319
 
Osallistujat:
Sampo Lehtinen (ohjaaja, toimi tilaisuudessa sihteerinä)
Timo Tonteri (pj)
Tobias Rask
Eero Vehmanen
Jesse Paakkari
Veera Hoppula
Mikko Kuusinen
 
Tarkastettu tuotos: suunnitteludokumentin 3. luku sivut 4-13. 
Tyyppimerkinnät: V selvä virhe, T tulkinnanvarainen virhe, P puute. 

Todetut puutteet (puutteen toistumista ei välttämättä ole merkattu, vaikka 
toistuminen olisi todettu): 

V 3 The plane routes. Pitäisi olla arrival tree. 

V 3.1 Linkitetty lista. Pitäisi olla taulukko. 

V 3.1 No information of direction is required. 

T 3.1 A* algorithm. Kirjoitusasu vaihtelee pitkin dokumenttia. 

P 3.1 Puuttuva metodi. getOutherArc tms. 

T 3.1 The-artikkeli esiintyy vaihtelevasti metodien nimien edellä. 

T 3.1 Metodien parametrit kuvattu eri metodeissa eri tavalla. 

V 3.1.8 the updated grap is all went ok. is -> if. 

T 3.3 mahdollisesti muuallakin. Tekstissä metodien nimet, tms. hakasten 
sijasta 
kirjoitusasultaan kursiiviksi. 

P 3.3 kolmasrivi konstruktorin kuvaus vajaavainen. 

P 3.3.1 Taulukon alaviiva puuttuu. 

T 3.3.1 flagSystemTimer kuvaus huono. 

T 3.3 2. kpl, 2. rivi puhutaan parametreista vaikka pitäisi ilmeisesti 
olla 
datacollection. 

V 3.3 CalculationControl nimi eroaa toteutuksessa olevasta nimestä 

V 3.3.2 viimeinen lause: sets is -> sets it 

V 3.3.2 puhutaan member variableista vaikka pitäisi ilmeisesti olla olio 
tai 
luokkamuuttuja. 

V 3.4 kuvailtu liian suppeasti 

V 3.4 pitäisi olla yhtä tasoa syvemmällä eli tarkastushetkellä 3.3.3. 

V 3.5 pitäisi ilmeisesti olla set TimerFlag 

V 3.5 pitäisi olla tasoa syvemmällä 

V 3.6 pitäisi olla tasoa syvemmällä. 

V 3.6 kuvailu kirjoitettava paremmaksi, ei ranskalaisten viivojen 
kokoelmaa. 
Muiden metodien kirjoitusasun mukaiseksi. 

T 3.6 succession pitäisi ilmeisesti olla success. 

T 3.6.1 turhat heittomerkit falsen ja truen ympärillä. 

P Yleisesti pitäisikö luokan tai paketin edellä olla asiaa selventävä 
kuva. 

V 3.6.1 metodi mahdollisesti väärässä luokassa. 

T 3.7.1 laatikon pohjalla ylimääräinen viiva. 

V 3.7.2 viimeinen rivi ArrayList of Vertx-sanan monikko ei vectors. 

T 3.7.3 käytettävä heuristiikka pohdinnan alla. 

T 3.7.3 The priority valjue of vertex n in ... tarvitsee ehkä muutosta 

T yleisesti muuttujien nimet kursiiville. 

V 3.8 väärällä tasolla. 

T 3.8 1. numeroitu kohta 3. rivi viimeinen sana 

T 3.8 hassu Z-kirjain 

T 3.8 Halfs -> halves 

V 3.8 metodi olettaa, että yksi polun solmuista olisi mergepointti. 
Huomioitava, 
ettei ristiriidassa minkään kanssa. 

V 3.8 sisältää mahdollisesti kirjoitusvirheitä, joiden kohtaa ei muistettu 
katselmoinnissa. 

V 3.10 closed list -> muutettava available muuttujan arvoa. 

V 3.11 metodin nimi 

T 3.11 locate sana väärin. 

V 3.13 kuvailtu liian suppeasti. 
Tarvitaanko sivunvaihtoja pakettien tai luokkien jälkeen. 

T 3.14.4 allready --> already 

T 3.14.4 sisäkkäisiä sulkeita. 

T 3.14.4 lyhenteet sanastoon. 

T 3.14.4 paluuarvon kuvaus. 

T 3.14.4 knowledle -> knowledge 

V 3.16 kuvaus flight path -> arrival tree