synesteesia / reseptisovellus Goto Github PK

Vertaisarviointi 13.4.2021 https://github.com/synesteesia/reseptiSovellus

Sovelluksen ulkonäkö näyttää tosi kivalta! Myös kuvaamasi toiminnallisuudet toimivat hyvin.
Tuntui ehkä vähän hassulta kirjoittaa valmistusohjetta ennen kuin on listannut tarvittavat raaka-aineet;
pitäisikö nämä olla toisinpäin? Eli ensin tallennetaan reseptin nimi, sitten listataan tarvittavat ainekset
ja siihen alle valmistusohje?

Tuossa missä kaikki reseptit listataan niin mitä ensimmäisessä kolumnissa (jossa nyt lukee vain todo ja risuaita)
olisi tarkoitus lukea? Olisiko esimerkiksi kesto tai vaikeusaste?

Nyt pystyy lisäämään myös tyhjiä rivejä reseptiin - jonkinlainen käsittely tarvitaan tähän. Lisäksi nyt
kuka vaan käyttäjä pystyy vain poistamaan esim kaikki reseptit - tämä ei ehkä hyvä idea. Pohdin myös reseptien
"omistajuutta" ja sitä että kuka vaan voi poistaa reseptistä ainesosia...Toki sanoit ettet ollut vielä
tietoturvaan liittyviä asioita pohtinut niin toivottavasti kommenttini auttavat siinä.

Tietokannan rakenne ja taulujen sisällöt vaikuttavat hyvältä paitsi että taulu userrecipes vaikuttaa
turhalta? Eikö käyttäjän reseptit voitaisi listata esim users-tauluun? Tai recipes-tauluun
tarvittava logiikka eli sinne käyttäjät jotka sen valinneet itselleen? Kannattaa ehkä miettiä tuota toiminnallisuutta
ja tuon taulun tarpeellisuutta.

Listaamissasi TODO-ominaisuuksissa on hyviä ideoita; sen sijaan laajennusideat vaikuttavat ehkä jo
liialta (mielestäni) tähän harkkatyöhön.

Vaikka olit kommentoinut, että koodisi on sekavaa vielä, en kyllä itse näin arvioisi, toki en mikään
suuri asiantuntija tässä ole. Mielestäni oli helppo tutkia koodia ja siitä havaita sovelluksesi rakenne.

Sovellusta käytettäessä kaikki toimi hyvin. Pidin noista oletko varma -varmisteluista, niin tietoa ei häviä vahingossa. Ehkä lisäisin rekisteröitymiseen vielä toisen salasanakentän, jos sattuu näppäilyvirheitä, tosin ensimmäisessä vertaisarvioitavassa oli myös vain yksi salasanakenttä. Ohjeen muokkauksessa saattaisi riittää pelkkä ohjeen muokkaustoiminto, uusi ohje -kohta hieman hämmensi, kun ohje oli jo olemassa. Tuota ehkä hieman miettisin. Toimisi kyllä hyvin myös jos aineksien lisäämisen ja ohjeen lisäämisen voisi toteuttaa samassa näkymässä, se on varmaankin tuossa ollut mietinnän alla.
Koodissa skeema oli asianmukainen ja selkeä, samoin html -tiedostot olivat ulkoasultaan todella selkeitä lukea. Loginia hetken etsin ja sieltähän se indexistä löytyikin ihan ok. Routes -tiedostoon olisin ehkä kaivannut polut keskitetysti, josta sitten kutsujen kautta toiminnot haetaan eri moduuleista, tämä voisi auttaa myöhemmin ylläpidossa. Näinkin koodi oli kyllä mielestäni erittäin selkeä ja helppolukuinen. Session -oliota oli käytetty mielenkiintoisesti, tästä tavasta täytyy ottaa itsekin selvää.
Kaiken kaikkiaan kiva sovellus, tekee mieli ottaa käyttöön! :)