search

lacolegiala / movie-app

1 stars 0 forks source link

Full Stack projektin katselmointi #1

Open Jakousa opened 3 years ago

Jakousa commented 3 years ago

Full Stack projektin katselmointi

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?

Selailin elokuvia

Etsin elokuvan hakutoiminnallisuudella

Kokemus

Todella hyvän näköinen!

Napit nyt vähän old-school, mutta kaikki toimii niin sulavasti että napit meinasivat unohtua.

Favicon unohtui vaihtaa reactin omasta.

Koodi

Tämä ei näytä sellaiselta asialta että sitä kuuluisi tallentaa versionhallintaan:

https://github.com/lacolegiala/movie-app/blob/8841d08fd8ce11d625294d5bb972b08338227e25/src/utils/tmdbApiClient.ts#L7

Miltä kuulostaisi switch casen tilalle jotain muuta: https://github.com/lacolegiala/movie-app/blob/8841d08fd8ce11d625294d5bb972b08338227e25/src/components/GenreList.tsx#L41-L82

vs

const idToGenreMap = {
  28: action,
  12: adventure,
  16: animation,
  ...
}

ja

idToGenreMap[id]

Muuten koodi on hyvälaatuista! Hienoa!

Kokonaisuus

Hienot sivut tietokannan sisällön selailuun. Pidän erityisesti siitä että sivut vaikuttavat mobiiliyhteensopivilta. Olisi kiva jos lisäät myös youtube videon embedauksen suoraan sivuillesi ettei tarvitse klikkailla linkkejä.

lacolegiala commented 3 years ago

Kiitos katselmoinnista ja parannusehdotuksista! Aion parannella projektia vielä jatkossakin, esimerkiksi mainitsemasi nappien ulkonäön parantaminen ja YouTube-videon upottaminen sovellukseen ovat jo to do -listalla.

Apikey on muutenkin nähtävillä selaimen network-tabissa, joten en näe erityistä syytä poistaa sen arvoa koodista. TMDB:llä ei tietääkseni ole muita autentikointitapaa, joten toistaiseksi pysyn tässä ratkaisussa.

Switch casen vaihtaminen objektiksi ei olisi käynyt itselläni mielessä, kiitos ehdotuksesta! 👍