Ohjelmistotuotantoprojekti Kasi TARKASTUSPÖYTÄKIRJA 28.2.2007 Aika ja paikka 28.2.2007 klo 14.15 Sali CK109 Tietojenkäsittelytieteen laitos (Exactum) Gustaf Hällströmin katu 2b, Helsinki Osallistujat Tuukka Palomäki Lauri Holmas Kestilä Veli-Pekka, puheenjohtaja Lahtinen Joni Penttilä Markus Sarin Antti-Pekka (saapui n. klo 16.30) Ilkka Tikkala Ilari Moilanen, ohjaaja ja sihteeri Poissa Ei poissaolijoita 14.16 Kokouksen aloitus Tyyppimerkinnät V selvä vakava virhe (asia ilmaistu väärin) T tulkinnanvarainen virhe (epäselvää tekstiä) P puute (jotain puuttuu) Nro Luku Sivu Kohta Tyyppi Kuvaus 1 4.1 5 taulukko T Boolean tietotyyppi ei ole esim Oraclen natiivityyppi 2 kuvaus T pääkäyttäjä on väärä termi 3 taulukko T || 4 4 - T Luku 4.1 pitäisi siirtää kuvan 2 jälkeiseen osaan 5 alin kappale T "Seuraavalla sivulla" -> "Kuvassa 2" 6 6 kuva,userbalance V ub_username ei ole oikeasti int 7 kuva,log T "PK" pois 8 4 7 - T "auto increment" pitää muuttaa monessa alikappaleessa toiseen muotoon 9 4.2 taulukko T Datetime tietotyyppi ei ole esim postgren tukema 10 4.3 taulukko V f_fpid ei vastaa kuvaa 2 11 V f_fpid kuvaukseen merkintä "pääavain" 12 V fp_username ei vastaa kuvaa 2 13 4.4 taulukko V fp_fingerprintid ei vastaa kuvaa 2 14 4.5 8 kuvaus T kappaleen toinen lause kaipaa korjausta 15 4.6 kuvaus T kappaleen toinen lause kaipaa korjausta 16 P "tuontiarvo", "asiakaskäyttöliittymä", "aktiivikäyttö" jne. Sanat eivät ole sanastossa tai ne ovat virheellisiä 17 taulukko T tietotyypit blob ja datetime eivät ole kaikkialla tuettuja 18 4.7 kuvaus P (ks kohta 16) 19 4.8 9 kuvaus T "ostoarvo" ei ole sanastossa 20 4.9 kuvaus T taulun tarkoitus ei ole selvä kuvauksen perusteella 21 4.10 10 taulukko V in_cancelevent on mahdollisesti turha 22 4.11 kuvaus T mikä on "mielekäs järjestys" 23 4.12 11 taulukko T log_tableid ja koko taulun logiikka on epäselvästi ilmaistu 24 4 11 kuva V pois 25 5.1 12 kappale 1 T viite kuvaan puuttuu 26 kappale 2 T selitä tarkemmin "riippumaton" käsite 27 kappale 4 T "Se on yksi ohjelman yksinkertaisimmista..." tarpeetonta selitystä 28 T Authentication komponenttia käytetään myös sisäänkirjauksessa 29 kappale 5 T "Asetustiedoston tullessa komentori..." liian yksityiskohtaista 30 P "settings-luokkia" ei ole sanastossa 31 kappale 6 P "JDBC-rajapinta" ei ole sanastossa 32 viimeinen kappale T viimeisiä virkkeitä muutettava selkeämmiksi 33 T viimeisten virkkeiden sisältö kirjoitettava tarkemmin auki 34 5.2 14 kuvaus T korjaa viimeinen lause 35 5.2.1 - T koko luku siirretään myöehmmäksi eli käydään ensin läpi sellaiset luokat jotka eivät viittaa muihin luokkiin 36 alaotsikko V "Metodit" -> "Muuttujat" 37 familynamen kuv. T korjaa ensimmäinen lause 38 fingerprints kuv. T pikkurilli ja pekku väärässä järjestyksessä 39 15 defaultlang. kuv. T koko kuvaus muutettava. lisäksi sana "vaihtoehtoja" -> "arvoja" 40 Metodit aliluku T "Muut metodit ovat" pois koko kappale 41 16 setBalances T Liian monimutkainen toimpenpide GUIn hoidettavaksi? 42 5.2.2 17 FINNISH, SWE... T voisi tehdä oman erillisen luokan kielille 43 getBalance T korjaa lause 44 5 5.2.3 ja muita T onko main paras paketti näille luokille 45 5.2.3 kuvaus T "pohjana" -> "yliluokkana" 46 18 getGroup T muuto metodin nimi kuvaavammaksi 47 5.2.5 19 kuvaus T lisätään tieto perimisestä 48 5.2.6 20 kuvaus T poista virkkeen loppu 49 5.2.7 21 kuvaus V "jonka voi perua" -> "joka on peruttu" 50 timedate T muoto pitää speksata jossain 51 5.2.8 Muuttujat aliluku T sulut pois "(sekunneissa)" ympäriltä 52 22 finnish,swe... T taulukkoon 53 5.2.9 23 texttable T muokkaa nimeä 54 texttable kuvaus T avaa kappaleen teksti 55 Language T korjaa konstruktori 56 5.2.10 kuvaus V poista lause "ja lisäksi kuinka usein kantaan..." 57 24 databaseurl,dbu... T mitä kaikkea tietoa on osoitteessa mukana ja mitä ei 58 DBQuery T korjaa luokan rakenne 59 5.2.11 25 - V poista koko kappale 60 5.2.11 T hakasulut vai kysymysmerkki vai mitä 61 5.2.12 cachesize T pois 62 licencepath kuv. T poista turha lause keskeltä 63 5.3 27 kuvaus T selitä termejä enemmän 64 P viite kuvaan puuttuu 65 5.3.1 kuvaus T selvennä kappaleen viimeistä lausetta 66 28 stopForProblem T selvennä kuvausta 67 5.3.2 authenticateUser T tässä ja tulevaisuudessa boldaa true ja false arvot ja parametrit ja esim null arvo 68 importProduct T tässä ja alemmassa metodissa muuta "tavara" -> "tuote" 69 29 deleteFingerprint T viimeinen virke on virheellinen 70 cancelProductEx... T viimeinen virke on virheellinen 71 5.3.3 31 saveFingerprint V "image" -> "data" 72 reconnectDatab. T kuka tekee uudelleenyritykset? tämä luokka vai joku muu 73 5.3.4 kuvaus T korjaa viimeinen lause 74 32 getFingerPrintS.. V metodin nimen keskellä "P" -> "p" 75 5.3.5 kuvaus T tunnistus tehdään TKTL käyttäjätunnuksia vasten 76 authenticateUser T "parametreikseen" -> parametreinaan. Boldaa true ja false 77 5.3.6 - V addFingerprint metodi ja sen kuvaus puuttuvat 78 5.3.7 authenticatedF... V "void" -> "boolean" ja korjaa kuvaus 79 6 33 - V Pitää selittää tarkemmin arkkitehtuuria esim. mitä inludoidaan ja milloin jne 80 6.1.1 kuvaus T Molemmat ensimmäisen kappaleen lauseett kaipaavaat korjausta. 81 6.1.2 kuvaus T Voisi kertoa että hakemistorakenne esitellään myöhemmin 82 connection T Tarkennetaan mitä kansiossa on tarkalleen 83 34 product.php T Mistäkautta tälle sivulle tullaan? 84 editP. addP. jne T Katoaako tuotelista/käyttäjät tuottamat taulukot tms kun näille sivuille tullaan? 85 removeProduct.php V tuotteita ei poisteta oikeasti kannasta 86 T Jos tuotetta ei halutakkaan poistaa niin mitä sitten tapahtuu? 87 35 user.php T Miten tälle sivulle päästään? 88 addUser.php V koko tämä alikappale poistetaan dokkarista 89 editU. remov. jne T (ks. kohta 84) 90 report.php T (ks. kohta 83) 91 editU. remov. T Onko tunnisteena username kuten mainittua? 92 - P Pitäisi olla haku jolla hakea käyttäjiä 93 6.1.3 36 kuvaus V includataan ei ole sana 94 6.1.3 - P Kaikista luokista voisi kertoa missä ne sijaitsevat 95 6.1.4.1 kuvaus V "fi.php", se.php" ja "en.php" kolmikirjaimiseen muotoon (eli fin.php jne) 96 37 kappale 2 P lisää "swe.php" 97 - T ruotsia dissataan jatkossa paljon 98 T pitää selittää missä ja miten kielivalinta tehdään 99 38 Metodit kappale T mitä tehdään jos virheilmoitus tulee? 100 6.1.4.2 - P sql-injection hyökkäyksiin varautumisesta pitää selostaa 101 39 executeSelect jne T selitä auki lauseet "komennen suoritukset PHP:lla voivat poiketa toisistaan" 102 6.1.4.3 40 - V koko luokka suunnitellaan uudestaan uuden tietokannan mukaiseksi 103 6.1.4.4 41 - V osa tämän luokan metodeista pitäisi laittaa erikseen (esim erilliseen luokkaan) jotka käsittelevät tämän luokan ilmentymiä 104 42 Metodit T mistä setter metodit saavat muuttujansa. Pitää selittää enemmän 105 6.1.4.5 44 - V (ks. kohta 103) 106 45 Metodit T (ks. kohta 104) 107 loadGroupUsers V tämä metodi kokonaan pois(?) 108 6.1.4.6 46 Metodit T (ks. kohta 104) 109 7.1.1 47 kuvaus T mitä virheilmoitukselle tehdään? 110 48 Metodit P lisätään toinenkin konstruktori 111 displauPopup P käytettyjä sanoja ei selitetä tässä tai sanastossa 112 7.1.2 49 kuvaus T annetaanko parametrina luokka vai rajapinta 113 kuvaus (kappale 2) T poista sana "metodeja" ja muokkaa 114 P kun nyt kerta exceptioneja aletaan luettelemaan niin kerrotaan sitten ne kaikki kerralla 115 Metodit T (ks. kohta 112) 116 7.1.3 50 kuvaus T (ks. kohta 112) 117 51 Metodit T (ks. kohta 112) ja tästä dokkarin kohdasta eteenpäin asiaa ei enää merkitä virheenä 118 7.2.1 52 - T kuva ja teksti eivät täsmää -> poistetaan kuvasta osia 119 7.3 kuvaus T mitä ovat "asetustiedostot"? 120 - V tietokantakaavio pitää korjata 121 7.3.1 53 Muuttujat T mikä Properties olion tarkka luokkapolku on? 122 7.4.1 54 - P metodi puuttuu Päätös: Dokumentti hyväksyttiin muutoksin. 16.02 Kokouksen lopetus