Project Tech: Refactor - lottekoblens/matching-application GitHub Wiki

1. Function names should say what they do

UpdateData zegt niet welke data er ge-update gaat worden.



function updateData(id, liked) {}

Verandert in:

function likedAndVisitedToTrue

2. Remove dead code

const likedPeople = liked[0].id

Met likedPeople wordt niks gedaan, dus kan verwijdert worden.

3. Use meaningful and pronounceable variable names

const liked = await findAllPeopleLiked();

const liked zegt niet dat het over mensen gaat die geliked zijn, dus dat verander ik in:

const likedPeople = await findAllPeopleLiked();

4. Error handling

Bij de catch maak ik nu alleen gebruik van een res.redirect, maar ik geef niet de optie om te zien wat de error is. Dus ik voeg hierbij een console.log toe.

5. Formatting: use consistent capitalization

Bij de eerste functie moet all ook met een hoofdletter om consistent te zijn.