Full Stack projektin katselmointi

[Kaltsoon]

Katselmointi

Terve @larilofman! Tässä lyhyt katselmointi Full Stack kurssin projektistasi.

Käytettävyys

Mitä tein?

• Rekisteröidyin uutena käyttäjänä
• Muokkasin käyttäjätietojani
• Lisäsin uuden listan
• Lisäsin, muokkasin ja poistin listattavia asioita
• Kirjauduin ulos

Kokemus

• Harvoin näkee projektissa yhtä selkeää README:tä
• Sovelluksella on tyylikäs ja käyttäjäystävällinen käyttöliittymä, hyvää työtä! 🥇
• Hieno favicon!
• Automaattinen sisään kirjautuminen rekisteröitymisen jälkeen, on erittäin kätevää
• Lomakkeissa on virhetilanteissa selkeät virheviestit
• Kaikkien sovelluksen toimintojen käyttö on intuitiivista ja helppoa, moitittavaa on vaikea löytää

Koodi

• Frontend-puolen koodi on selkeästi organisoitua, hyvä!
• Tämä tiedosto kuuluisi varmaan .gitignore-tiedostoon?
• services-hakemistossa on hyviä abstraktioita API-kutsuihin
• Reduxin actioneiden typet kannattaa tallentaa const-tyyppisiin muuttujiin ja käyttää niiden arvoja sen sijaan, että käyttää niitä merkkijonomuotoisena. Tämä ehkäisee kirjoitusvirheistä syntyviä bugeja ja helpottaa refaktorointia. Mallia voi katsoa Reduxin dokumentaatiosta
• Reduxin actioneita kannattaa luoda nk. action creator-funktioiden avulla sen sijaan, että käyttäisi suoraan objekteja dispatch-kutsun yhteydessä. Kun actionin luonti on abstraktoitu funktion avulla, sitä on helpompi muuttaa tulevaisuudessa ilman suurempaa refaktorointia
• Syvien tietorakenteiden immutable päivittäminen Reduxin reducereissa puhtaalla JavaScriptilla on melko työlästä. Immer-kirjasto helpottaa tätä huomattavasti
• Redux Toolkit tarjoaa todella hyviä työkaluja Reduxin käyttöön. Esim. createReducer-funktio vähentää reducerien boilerplate koodia huomattavasti. Se käyttää sisäisesti Immer-kirjastoa, joten myös tilan päivittäminen helpottuu
• API-kutsuihin liittyvät operaatiot tuottavat Reduxin kanssa usein jonkin verran toisteellista boilerplate-koodia. SWR kirjaston avulla API-kutsut voi helposti abstraktoida yksinkertaisiksi hookeiksi
• Hyvä idea käyttää Formik-kirjastoa lomakkeiden tilanhallinnan kanssa
• Myös backend-puolen koodi on selkeästi organisoitua, hienoa!
• Backendissäkin löytyy erittäin hyviä abstraktioita services-hakemistossa

Kokonaisuus

Kätevä sovellus, jonka toiminallisuus on toteutettu moitteettomasti. Koodi on kaikilta osin selkeästi organisoitua ja pääosin laadukasta. Koodin ylläpidettävyyttä parantaa huomattavasti hyvät abstraktiot sekä frontend-, että backend-puolella. Teknologioita, kuten TypeScriptiä on käytetty järkevästi. Hyvää työtä!

[marimetsola]

Heippa @Kaltsoon ! Kiitos katselmoinnista, kehuista ja vinkeistä. Jahka saan tuon kymppiosan tehtyä ja kurssin taputeltua taidan syventyä refaktoroimaan tuota state management hirvitystä neuvojesi perusteella. Kenties repon kehtaa sitten muuttaa julkiseksi ja jopa esitellä työpaikkoja hakiessa.