[CODE] overview.js

[bommezijn]

Probeer de console.log(...) te vermijden als je de code pusht naar je main branch op github. Op dit moment als je naar de live variant gaat krijg je de fetchData Object te zien en dat wil je eigenlijk niet.

functie overview() haalt de data twee keer op. Met fetchData() en cleanData()
cleanData module heeft de import fetchData en roept fetchData() ook een keer op om data te manipuleren en een array terug te geven.

Op dit moment heb je een bug met je hero portraits, wrecking-ball portrait word niet opgehaald. Probleem hier ligt erin dat de naam een hyphen heeft maar de name heeft het probleem dat er geen hyphen in zit. Dit is natuurlijk zuur voor de grid.

Een oplossing zou zijn om je cleanNames uit te werken door hier dan een unieke case voor te maken in de loop. if (data.name === 'wreckingball'){ gekke regEX function die zoekt naar een combo en dan wanneer er een match is insert hypen }

Maar aangezien dit een enkel geval is en de API je hier niet bij helpt zou ik het gaan kijken als dit noodzaak is of een Could Do in de feature list

[bommezijn]

Het laatste punt, Vooral omdat alles werkt behalve de image omdat de API besluit om alles op 'wreckingball' te plaatsen behalve de image.

[bommezijn]

Ben er vergeten bij te zeggen:

Plaats comments bij de functies van wat het doet, gebruikt en wat er uit komt. Dit geeft de lezer en schrijver van de code inzicht van wat het zou moeten doen en hoe het gedaan kan worden.

Mijn Go To type of comments daarvoor is JSDocs

/**
@title Naam van functie
@description Wat doet de functie?
@param {TYPE VAR} NAAMVAR Beschrijving Variable
*/

@param zijn al je parameters die je gebruikt voor de functie, als je dit eenmaal doet krijg je bij VScode ook code highlighting hiervoor door intellisense