synesteesia / reseptisovellus Goto Github PK
View Code? Open in Web Editor NEWLicense: MIT License
License: MIT License
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! :)
A declarative, efficient, and flexible JavaScript library for building user interfaces.
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google ❤️ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.