Full Stack projektin katselmointi
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?
Kirjauduin sisään annetuilla tunnuksilla
Aloitin pari challengea, keskeytin niistä toisen
Kokemus
Siistin näköinen appi. Ei mitään turhaa.
Login nappi voisi olla näkyvämmin
"You can edit past challenge days but you cannot alter future days in advance." ei taida pitää paikkaansa? En löytänyt painiketta muuttaa edellisiä päiviä. Toisaalta minusta tämä että voi muokata vain tätä päivää on sopiva tapa käyttää sovellusta haasteen kanssa.
Olisi kiva nähdä kuinka monta päivää jäljellä kun toteutin tämän päivän haasteen.
Koodi
Koodin on selkeästi jaettu tiedostoihin. Lähes kaikki löytyy sieltä mistä odottaisikin.
Tässä tiedostossa näyttää olevan joko paljon copy-pastea tai kaksi eri komponenttia: https://github.com/eevaalanko/fullstack2020-project/blob/8009eb6dcd3e3ce9e124bbdad824af7b693d4192/frontend/src/components/ChallengeList.js
Voisit kokeilla näyttämislogiikan siirtämistä komponenttiin itseensä. Eli sen sijaan että tarkistat onko activeChallenge parent komponentissa. Siirrätkin sen "ActiveChallengeComponent" sisälle laittamalla komponentin alkuun rivin
if (!activeChallenge) return null
Tällöin logiikka on selkeä: Jos haaste ei ole aktiivinen ei aktiivisen haasteen komponenttia ikinä näytetä.
Toki sinulla on tuossa linkkaamassani kohdassa myös muita solmuja ratkaistavana ennen kuin voit poistaa logiikkaa ChallengeComponentista.
Bäkkärin index.js on aika pitkä, voisikohan sitä jotenkin jakaa pienempiin osiin?
Kokonaisuus
Simppeli projekti hienosta ideasta. Hyvää työtä! Koodista oli vaikea löytää mitään nipotettava.