Closed JeroenHeemskerk closed 1 year ago
Let op dat je net as in issue #23 geen datavelden buiten de code hebt staan
Ook is het overstandig om de HELE user bestand iedere keer in te lezen als je wilt controleren of een user bestaat. https://github.com/Corby7/educom-webshop-basis/blob/01d0932819c1f6ece226298bab1b6acd09cf030e/filerepository.php#L6-L37
Wat als er 10.000 users zouden zijn, dan neem je voor 10.000 users alle data in geheugen. Dit weten Hackers goed te gebruiken.
Beter is het om in de findUserByEmail functie de fopen, de fclose en de while(!feof(...)) te hebben staan
findUserByEmail
fopen
fclose
while(!feof(...))
Let op dat je net as in issue #23 geen datavelden buiten de code hebt staan
Ook is het overstandig om de HELE user bestand iedere keer in te lezen als je wilt controleren of een user bestaat. https://github.com/Corby7/educom-webshop-basis/blob/01d0932819c1f6ece226298bab1b6acd09cf030e/filerepository.php#L6-L37
Wat als er 10.000 users zouden zijn, dan neem je voor 10.000 users alle data in geheugen. Dit weten Hackers goed te gebruiken.
Beter is het om in de
findUserByEmailfunctie defopen, defcloseen dewhile(!feof(...))te hebben staan