Full Stack projektin katselmointi

[Kaltsoon]

Full Stack projektin katselmointi

Terve @mriekkinen! Tässä lyhyt katselmointi Full Stack kurssin projektistasi. Katselmoinnissa esitetyt kommentit ovat parannusehdotuksia, joita voit halutessasi ottaa huomioon tässä tai tulevissa projekteissasi. Mitään muutoksia tähän projektiin ei siis ole pakko tehdä suoritusmerkintää varten.

Käytettävyys

Mitä tein?

Kokeilin hakua eri hakurajaimilla.

Kokemus

• Sovelluksen käyttöliittymä on käyttäjäystävällinen tyylikäs, hyvää työtä! 🥇
• Melko pieni yksityiskohta, mutta geneerisen "React App"-faviconin voisi muuttaa kuvaavammaksi
• Kätevää, että hakurajaimien muutokset muuttavat osoiterivin polkua, tämä tekee mm. hakujen jakamisen ja kirjanmerkkien tekemisen mahdolliseksi

Koodi

• Frontend-puolen koodin organisointia voisi ehkä vielä hieman hioa. Esim. komponentit components-hakemistoon, apu-funktiot utils-hakemistoon, jne. Näin relevantin koodin löytäminen on nopeampaa

• API-kutsuihin liittyvät operaatiot tuottavat usein jonkin verran toisteellista boilerplate-koodia. Esimerkiksi React Query -kirjaston avulla API-kutsut voi helposti abstrahoida yksinkertaisiksi, uudelleenkäytettäväksii hookeiksi

• Reduxin actioneiden typet kannattaa tallentaa const-tyyppisiin vakiomuuttujiin ja hyödyntää näitä muuttujia esim. reducereissa sen sijaan, että käyttäisi niitä suoraan merkkijonomuotoisena:

  const SET_STATUS = 'poiList/setStatus';
  
  // ...
  
  const poiReducer = (state, action) => {
    switch (action.type) {
      case SET_STATUS:
      // ...
    }
  };

  Tämä ehkäisee esimerkiksi kirjoitusvirheistä syntyviä bugeja ja helpottaa refaktorointia. Mallia voi katsoa Reduxin dokumentaatiosta

• Redux Toolkit tarjoaa todella hyviä työkaluja Reduxin käyttöön. Esim. createSlice-funktio vähentää reducerien boilerplate-koodia huomattavasti. Se käyttää sisäisesti Immer-kirjastoa, joten myös tilan päivittäminen helpottuu

• Backend-puolen koodi on selkeästi organisoitua, hienoa!

• Dockerin kanssa on hyödynnetty järkevästi multi stage buildia, niin sovelluksen image pysyy kompaktina

Kokonaisuus

Sovelluksesta löytyy toiminallisuus paikkojen hakuun monipuolisilla hakurajaimilla. Sen käyttö on helppoa, eikä katselmoinnin aikana ilmennyt käyttökokemusta heikentäviä bugeja, tai käytettävyysongelmia. Projektin koodi on suurimmilta osin selkeästi organisoitua ja sen laatu on erittäin hyvä. Teknologioita, kuten TypeScript ja Docker on hyödynnetty projektissa erinomaisesti. Hienoa työtä!

[mriekkinen]

Hei, kiitos arvokkaista kommenteista! Otan näistä opiksi ja hyödynnän niitä tulevissa projekteissa.