Skip to content

Peer feedback

Tom van den Berg edited this page Nov 4, 2021 · 2 revisions

De dag voor de eindbeoordeling was het tijd voor de peer-feedback. Ik kreeg en gaf feedback aan Melvin Idema.

Hij heeft een aantal issues achter gelaten in mijn repo.

For Loops

Ten eerste gaf hij aan dat ik nog ouderwetse for-loops gebruikte in mijn functie.

.then(function (response) {
  -      const arrayLength = response.data.standings[0].table.length
    -      for ( let i = 0; i < arrayLength; i++) {
    -        standingsPL.push(response.data.standings[0].table[i].team.name)
    -      }
  -    })

Hier had hij zeker gelijk in. Hij kwam ook met een oplossing om de for-loops te vervangen naar arrow functions, namelijk:

.then(response => standingsPL = response.data.standings[0].table.map(item => item.team.name))

De code werd hierdoor een stuk duidelijker en simpeler. Daarna heb ik mijn overige functions omgeschreven naar arrow functions, bijvoorbeeld de functions in mijn modules. Hierdoor is mijn code een stuk leesbaarder en minimaler geworden.

Codestijl in je module

De tweede issue sluit aan bij de eerste issue. Hij zei namelijk dat ik mijn modules een stuk simpeler kon coderen, door middel van arrow functions.

Zo zag het er eerst uit:

module.exports = function filterData(data) { // filter data from API file 
  return data.map((item) => {
    return item.replace('FC', '').trim()
  })
} 

En zo ziet het er uit na de feedback:

module.exports = data => data.map(item => item.replace('FC', '').trim())

Home 🏠

Data cleaning 🧽

Documentatie 👨🏻‍💻

Logboek 📒

Notities 📝

Evaluatie 💭

Clone this wiki locally