AchrafReyani / educom-webshop-basis

0 stars 0 forks source link

Je herhaalt hier code, die je eerder al hebt uitgevoerd, dit is niet D.R.Y. #23

Open JeroenHeemskerk opened 5 months ago

JeroenHeemskerk commented 5 months ago

In de else if cases hieronder https://github.com/AchrafReyani/educom-webshop-basis/blob/80b3459e237fd77cf872cbc2e0e6dddcb09ad8c3/contact.php#L41-L43 https://github.com/AchrafReyani/educom-webshop-basis/blob/80b3459e237fd77cf872cbc2e0e6dddcb09ad8c3/contact.php#L47-L51 https://github.com/AchrafReyani/educom-webshop-basis/blob/80b3459e237fd77cf872cbc2e0e6dddcb09ad8c3/contact.php#L54-L56 Worden een aantal checks die je eerder hebt gedaan herhaalt: https://github.com/AchrafReyani/educom-webshop-basis/blob/80b3459e237fd77cf872cbc2e0e6dddcb09ad8c3/contact.php#L23 https://github.com/AchrafReyani/educom-webshop-basis/blob/80b3459e237fd77cf872cbc2e0e6dddcb09ad8c3/contact.php#L27 https://github.com/AchrafReyani/educom-webshop-basis/blob/80b3459e237fd77cf872cbc2e0e6dddcb09ad8c3/contact.php#L31 https://github.com/AchrafReyani/educom-webshop-basis/blob/80b3459e237fd77cf872cbc2e0e6dddcb09ad8c3/contact.php#L39 etc. Hierdoor staat de check op twee plekken, dit is niet D.R.Y (Don't Repeat Yourself) en kan ervoor zorgen dat iemand een veld toevoegt en hier mist dat hij in al deze else cases ook iets moet toevoegen

Beter is het om de 'valid' vlag af te laten hangen of er geen eerdere 'errors' zijn gevonden:

  $valid = empty($pronounError) && empty($nameError) && ... && empty($messageError);