Fullstack-harjoitustyön katselmointi

[ShootingStar91]

Fullstack-harjoitustyön katselmointi

Tämä on katselmointi fullstack-harjoitustyöstäsi. Sinun ei tarvitse toteuttaa mahdollisia parannusehdotuksia saadaksesi suorituksen työstäsi.

Käyttökokemus

• Sovellus on hauska arvauspeli.
• Se on helppokäyttöinen ja toimii.
• Pelissä on paljon ominaisuuksia, kuten kartta ja erilaisia vihjeitä.
• Ominaisuudet on toteutettu hyvin: ne asettuu hyvin ruudulle ja peliä on näppärä pelata.
• Ehkä vielä olisi ollut kiva jos maan voisi arvata klikkaamalla sitä suoraan kartasta.
• Rekisteröityminen ja kirjautuminen toimi myös ja niissä on hyvät virheviestit.
• Kirjautuneena pelit myös tallentuu ja niitä voi myöhemmin katsella.

Koodi

• Koodi on selkeää sekä hyvin jäsennelty eri kansioihin ja tiedostoihin
• Hyvin käytetty Typescriptiä ja muita teknologioita kuten Material UI:ta
• Koodissa sekä frontin että palvelimen puolella selkeä parannuskohde on virheenkäsittely:
  • Frontissa service-kansiossa requestien kohdalla on toistettu samaa koodia jossa katsotaan, palauttiko request virheen ja käsitellään se samalla tavalla. Tätä toisteisuutta voisi poistaa joko refaktoroimalla logiikkaa tai käyttämällä jotain ulkoista kirjastoa joka hoitaa tämän. Requestien tekemiselle ja virheen tarkistamiselle voisi esimerkiksi olla yksi funktio, jota sitten käytetään eri funktioista eri parametreilla joissa määritellään myös metodi.
  • Backendissä voisi käyttää express-async-errors -npm-kirjastoa, jolloin virheitä ei tarvitse jokaisessa service-funktiossa napata try-catchillä, vaan virheenkäsittelijä osaa napata ne automaattisesti. Tämä on esitelty kurssimateriaalissa.

Yleisesti

Sovelluksella on hauska idea, ja se on toteutettu hyvin. Bugeja ei löytynyt, ja peliä on hauska pelata. Pelissä on myös paljon ominaisuuksia. Koodi on selkeää ja hyvin organisoitua, ainoa selkeä parannuksen aihe on virheenkäsittelyn luoma toisteisuus. Erinomainen projekti, hienoa työtä! 💎