artoh
commented
4 years ago Kiitos kommenteistasi (ja aivan erityiskiitos aikaisesta katselmoinnista, nyt ei tule kiirettä muokata tehdä vielä vähän muokkauksia).
Pitää harkita tuo liitostaulun nimeäminen tulevissa projekteissa. Tosin ei oikein taivu taulun liittämiseen itseensä (esim. tässä projektissa lapsi-vanhempi -suhteen taulussa Huoltajuus, Henkilo_Henkilo olisi pikkasen outo ;)
Kiitokset näistä ja menestystä omaan projektiisi!
ArtoH
Projekti ladattu 13.10. n. klo 13
Dokumentaatio
Todella kattava ja selkeä dokumentaatio. Sovellus tuntuu olevan aika hyvällä mallilla, ja jossakin taisikin lukea, että kaikki suunniteltu toiminnallisuus onkin jo toteutettu. Varsinaisten erillisten dokumentaatiotiedostojen lisäksi koodin ymmärtämistä helpottivat pienet kommentit funktioiden/metodien alussa.
Yleisesti koodia läpikäydessä kaikki näyttää olevan aika hyvin järjestetty niin, että samaan asiaan liittyvät funktiot on koottu samaan paikkaan ja eri toiminnallisuudet eroteltu kansioihin ja tiedostoihin.
Modelit
Kommentit nopeuttavat hyvin lukemista
Joissakin SQL -kyselyissä on kirjoitettu ns. Keywordit kokonaan isoilla kirjaimilla, kun taas joissakin on koko kysely pelkillä pienillä. Pieni juttu, mutta koodi on kuitenkin jonkin verran selkeämpää jos käytetään samaa kirjoitusasua ja vielä (oma mielipide) mielellään sitä "all-caps" -tyyliä keywordeissä, jolloin keywordit ja sitä kautta kyselyn rakenne erottuu selvemmin.
Kontrollerit
Tietokanta
Näyttää omaan silmään järkevältä, näyttää olevan useampikin liitostaulu.
Liitostaulujen nimet ovat kyllä järkevät siinä mielessä, että ne kuvaavat juuri sitä mitä taulutkin, mutta itse suosin nimenä yhdistettävien taulujen nimien yhdistelmää, jolloin aina liitostaulun nimestä käy heti selväksi, mihin muihin tauluihin se liittyikään, vaikkei muusta tietokannan tai sovelluksen rakenteesta olisi mitään tietoa. Esim. vaikkapa kuvitteelliset taulut Henkilö ja Tehtävä, tilanteessa missä Henkilöllä voi olla monta tehtävää ja Tehtävällä monta Henkilöä (eli monta tekijää), nimeäisin liitostaulun "henkilötehtävä". Tällöin myös mahdollisesti ison ja monimutkaisen tietokantakaavion lukeminen voi helpottua, kun liitostaulut näkee nopealla vilkaisulla pelkistä nimistä erilaisen nimiformaatin (lähinnä -merkin) ansiosta.
Käyttökokemus
Käyttöliittymä on selkeä ja siinä käytetyt ikonit saavat sen tuntumaan yksinkertaisemmalta verrattuna pelkkään tekstiin, esim. "Poista" vs. roskakori-ikoni yms.
Sovellus on aika laaja, joten joitain toiminnallisuuksia varmaan jäi huomaamatta/testaamatta, mutta ne jotka tuli käytyä läpi näyttävät toimivan ihan oikein.