Project

Code

Vanaf lijn 13 in renderQuotetoHTML kun je misschien nog variabelen. Maar als je het maar 1 keer hoeft te doen twijfel ik ook eraan of je een variabel zou moeten maken ervoor. https://github.com/DustinSchouten/FDND-Web-Design-Quotes-App/blob/60acd1108df857343f3669813d87b3881588663d/modules/renderQuoteToHTML.js#L13
https://github.com/DustinSchouten/FDND-Web-Design-Quotes-App/blob/60acd1108df857343f3669813d87b3881588663d/modules/renderQuoteToHTML.js#L26 Je kunt dit ook veranderen naar een short-circuit:
```
   avatarImgElement.src = (avatarImgURL ||  'image_skeleton.png');
```
Dan hoef je geen if else te gebruiken. Voor de leesbaarheid kun je een regel commentaar erbij zetten en dan is het net zo leesbaar.
In states.js gebruik je voor meerdere functies dezelfde elementen. Bijvoorbeeld: Deze functie https://github.com/DustinSchouten/FDND-Web-Design-Quotes-App/blob/60acd1108df857343f3669813d87b3881588663d/modules/states.js#L3 en Deze functie: https://github.com/DustinSchouten/FDND-Web-Design-Quotes-App/blob/60acd1108df857343f3669813d87b3881588663d/modules/states.js#L9 Gebruiken allebei: https://github.com/DustinSchouten/FDND-Web-Design-Quotes-App/blob/60acd1108df857343f3669813d87b3881588663d/modules/states.js#L4 Deze zou je 1 keer als variabelen kunnen declareren en dan per functie oproepen on stijl te wijzigen.