Koodikatselmointi

[AnnaKuokkanen]

Koodi ladattu 02.06 klo 20.40

Koodin rakenne

Sovelluksen rakenne on selkeä. Tietokantataulujen määrittely ei tapahtunut omissa tiedostoissa, mikä olisi kasvattanut tiedostojen ja kansioiden määrää, vaan ne on jaettu sopiviin kokonaisuuksiin, mikä parantaa luettavuutta. Metodit vaikuttavat sopivan pituisilta. Niiden sisällä ei ole myöskään pitkiä rivejä. Muuttujat on nimetty selkeästi ja koodia on helppo lukea. En huomannut koodissa myöskään toisteisuutta.

Toiminta

Testattu seuraavia toiminnallisuuksia:

• luotu kaksi käyttäjää
• yritetty käyttää uudestaan samaa käyttäjänimeä sekä luoda käyttäjä tyhjillä kentillä
• muokattu käyttäjien tietoja ja lisätty niille projekteja
• lisätty ja poistettu käyttäjien taitoja sekä tarkasteltu niitä toisen käyttäjän näkymästä
• testattu projektin muokkaamista, poistamista ja muiden lisäämistä projektiin
• poistettu ja luotu käyttäjä uudestaan samalla käyttäjänimellä
• haettu käyttäjiä taitojen perusteella

Parannusehdotuksia ja jatkokehitysideoita:

• info muiden käyttäjien näkyville
• omat tiedot pois näkymästä, missä etsitään ihmisiä
• käsitellä kyselyt niin, että suuret kirjaimet ja välilyönnit eivät haittaa hakua
• hakuun voisi lisätä parametreja, esim. jotta ihmisiä voisi hakea sekä taidon että nimen perusteella
• ei voi lisätä kokemusta, joka on kestänyt 0 vuotta ja 0 kuukautta

Sovelluksen pääpiirteet toimivat hyvin ja käyttäjä voi jo hyödyntää sitä. Muutamia huomioita ja ehdotuksia:

Kokemusta lisätessä vuodet/kuukaudet voivat mennä negatiiviseksi ja tällöin kun kokemusta yrittää lisätä, tulee “Internal Server Error”. Käyttäjä voi myös lisätä taidon, josta ei ole ollenkaan kokemusta. Nämä molemmat tapaukset kannattaa estää käytettävyyden nimissä.

Projektia ei voi vielä hallita eikä ihmisiä lisätä projekteihin (Internal Server Error). Projektin hallintaan liittyen kannattaa tarkistaa kysely find_skills_by_project sekä tiedosto projects.html. Henkilöiden lisäämisessä virhe voi myös olla tiedostossa accounts.html.

Haku on tällä hetkellä case sensitive ja myös esim. välilyönti alussa/lopussa vaikuttaa hakuun. Käyttäjän kannalta olisi hyvä, jos tästä voisi päästä eroon muokkaamalla tietokantakyselyä.

Itse käyttäjä on listassa, kun etsitään muita ihmisiä. Olisi hyvä, jos käyttäjälle näytettäisiin vain muita käyttäjiä, vaikka tämä on myös makuasia.

Tällä hetkellä kun käyttäjä päivittää itsestään infoa, se näkyy vain hänelle itselleen (tai ainakaan en nähnyt infoa toisesta käyttäjästä). Voisi olla kiva, jos voisi nähdä infoa muista käyttäjistä.

Muihin bugeihin en onnistunut törmäämään. Poisto, listaus, lisäys ja muokkaus toimivat hyvin.

Yleisesti

Kaiken kaikkiaan koodi vaikuttaa laadukkaalta ja työ on hyvällä mallilla. Sovellus näyttää hyvältä ja on hyödyllinen. Dokumentaatio on myös ajan tasalla.

[MiguelSombrero]

Kiitoksia hyvistä kommenteista!