Fullstack-harjoitustyön katselmointi

[ShootingStar91]

Fullstack-projektin katselmointi

Tämä on katselmointi harjoitustyöstäsi. Sinun ei tarvitse toteuttaa tässä mainittuja parannushedotuksia saadaksesi suorituksen.

Käyttökokemus

• Kerrassaan upea ja uniikki ulkoasu
• Rekisteröityminen ja kirjautuminen toimi hyvin. Latausikoni requestin vastausta odotellessa olisi vielä hyvä idea.
• Ominaisuudet toimii hyvin ja on helppoja käyttää.
  • 'Needin' aktiivisuutta säätelevä toggle on hieman harhaanjohtava, kun sen molemmin puolin on ikonit muokkaamiselle ja poistamiselle. Ei ole selvää, että mitä toggle tekee, ennenkuin sitä kokeilee.
• Tekstikentissä on hyvin validaatioita joista tulee notifikaatioita, sekä esimerkiksi duration-kenttään ei voi syöttää muuta kuin numeroita.
• Sivusto toimii hyvin myös puhelimella. Valikko siirtyy alas ja ominaisuudet on helppoja käyttää. Hienoa!

Koodi

• En tunne Vuea, joten en osaa kommentoida sille spesifejä asioita. Koodi kuitenkin näyttää pääosin hyvältä. Tiedostot on organisoitu loogisiin kansioihin.
• Store-kansion tiedostoissa on paljon toisteista koodia, esimerkiksi virheenhallinnan sekä header-objektin osalta. Nämä voisi hoitaa jotenkin keskitetysti.
• Backendin petController.js-tiedostossa on esimerkkejä kahdesta parannettavasta asiasta:
  • Virheenhallinta - npm-paketti "express-async-errors" on kätevä tapa poistaa try-catch-koodi. Sen avulla esim. tietokantatoiminnoista tulevat virheet napataan automaattisesti ja lähetetään virheenkäsittely-middlewarelle.
  • Validaatiot (esim. addNewNeed ja addNewRecord -funktioissa) ovat todella pitkiä ja toisteisia. Tähän on kehitetty kirjastoja, ainakin 'zod' ja tietääkseni 'yup' ovat käteviä validaatioiden kirjoittamiseen.
• Muuten projektissa on käytetty fiksusti sopivia kirjastoja.
• Backend on organisoitu hyvin, ja middlewareja on käytetty tehokkaasti

Yhteenveto

Projekti on näppärä, hyvin tyylitelty ja helppokäyttöinen sovellus lemmikin tarpeiden seuraamiseen. Kaikki ominaisuudet toimii, sovellus on huoliteltu ja käyttökelpoinen myös mobiililla. Hienoa työtä! :gem:

[yumeangelica]

Kiitos hyvästä ja rakentavasta palautteesta. Tulen ottamaan nämä mainitsemasi parannusehdotukset huomioon tulevissa päivityksissä.