hanaalkadri / educom-webshop-basis

0 stars 0 forks source link

De functie test_input is op meerdere plekken gedefineerd, dit is niet D.R.Y. (Don't Repeat Yourself) #22

Open JeroenHeemskerk opened 1 year ago

JeroenHeemskerk commented 1 year ago

Deze functie staat in twee bestanden, dit is niet D.R.Y. (Don't Repeat Yourself) Als iemand een van de functie aanpast, zal hij niet zien dat er nog een andere versie bestaat.

https://github.com/hanaalkadri/educom-webshop-basis/blob/04dc111e0d495bf9f73f9bd95bd64c36478dbf88/contact.php#L73-L78 https://github.com/hanaalkadri/educom-webshop-basis/blob/04dc111e0d495bf9f73f9bd95bd64c36478dbf88/register.php#L70-L75 (idem in login.php)

Verplaats deze functie naar de index.php en verwijder uit contact.php, register.php en login.php!

Daarnaast is de functie test_input is in snake_case, terwijl de rest van de functies in camelCase zijn. Dit is niet consistent. Beter om de functie testInput te noemen.