Katselmointi

[JuhQ]

Huomioita

Backend

• koodi tosi hyvin kirjoitettua ja strukturoitua
• middleware.bouncer nimetty hyvin, iski silmään heti index.js:ää tutkiessa
• parser.js tiedoston trimDbObject funktion voisi ehkä refaktoroida käyttämään mongoosen .toObject() metodia?
  • http://mongoosejs.com/docs/guide.html#toObject
  • tämän lisäksi sitä voisi ehkä refaktoroida myös näin:

    const trimDbObject = (obj) => {
    const {_id, ...rest} = obj.toObject();
    return {id: obj._id, ...rest}
    }

  • deleten käyttö tuntuu aina vähän pahalta, es6 myötä sen käyttöä pystyy melko ketterästi välttämään
  • lisäksi tuossa esimerkissäni ei manipuloida alkuperäistä objektia, vaan palautetaan uusi objekti ja alkuperäinen pysyy alkuperäisenä
• testit tosi laajat ja hyvät
• router tiedostoissa jänniä rivejä, kuten esim router/task.js rivi 48, miksi tämmöinen ratkaisu?
• mongoa käytetty tosi monipuolisesti :)

Frontend

• tiedosto on lowercasena, reactin kanssa on hyvää käytäntöä käyttää isoa alkukirjainta tiedostonimissä, mielellään myös niin että tiedoston nimi mätsää sen tarjoamaan komponenttiin. Esim form_error.js pitäisi olla FormError.js
• reactissa on hyvänä käytäntönä pidempien komponettimääritysten kohdalla rivittää komponentin kaikki propsit
  • eli

    <Route
    exact path={match.path}
    component={MaterialsContainer}
    />

    muotoon

    <Route
    exact
    path={match.path}
    component={MaterialsContainer}
    />

• kurssin oppeja käytetty hyvin
• reduxia käytetty hyvin
• .eslintrc tiedostoon jos lisäisi airbnb:n konffin niin iso kasa koodin tyylittely (ja jollain tasolla logiikkaan liittyvistä) ongelmista korjaantuisi

Sivuston käytettävyydestä

• sivusto erittäin käytettävä
• demodata huvittava! :D
• sivusto tuntuu selkeästi tarkoitettu mobiilissa käytettäväksi, mutta desktop versioon kannattanee myös panostaa?
  • esim desktopissa hamburger menun sijaan linkit voisi näyttää suoraan

Hyvän näköinen projekti kaikinpuolin!

[jrnn]

Suurkiitokset! Tuntuupa hyvältä saada palautetta! Ja erityisesti kiitokset spesifeistä korjaus- ja parannusvinkeistä.

Viittaamasi router/task.js:48 (eli irrallaan roikkuva rivi task._id) on hyvin kyseenalainen ratkaisu, tarkoitus on aiheuttaa virhe jos annetulla id:llä ei löytynyt mitään... aiheuttaa siis TypeErrorin jotka virheenkäsittelijä on myooppisesti ohjeistettu raportoimaan 404:na ( :D ). Fiksumpaa kai olisi kirjoittaa suosiolla auki mistä on kyse, vaikka

if ( !task ) throw new ResourceNotFoundError()

tjsp. ... en ottanut selvää saisiko saman jotenkin nätisti leivottua sisään jo mongoosen hakumetodiin

Frontend oli tarkoitus tehdä vain mobiiliin (sovelluksen käyttäjät ovat järjestään joko rakennustyömailla tai tien päällä), mutta ei uskallus riittänyt lähteä kylmiltään kokeilemaan React Nativea. Jos sovellus tulee aktiivikäyttöön niin kenties pääsen tekemään frontin puhtaalta pöydältä (mobiiliin siis), jolloin pitää kyllä ottaa toi AirBnB:n tyylisäännöstö käyttöön.

Vielä kerran kiitos vaivannäöstä!