Thomas-van-Haastrecht / educom-webshop-basis

0 stars 0 forks source link

Het patroon van $valid op `true` zetten en dan steeds weer terug naar `false` is een slecht patroon #17

Open JeroenHeemskerk opened 8 months ago

JeroenHeemskerk commented 8 months ago

In de code hieronder begint $valid op false en als het een POST is wordt hij gelijk op true gezet. Om vervolgens bij iedere fout opnieuw op false gezet te worden https://github.com/Thomas-van-Haastrecht/educom-webshop-basis/blob/2dcd8442110aea5b3a205c4e82bd3217844351bf/contact.php#L11-L20

Dit patroon is erg fout gevoelig, als iemand een conditie toevoegd, kan er snel vergeten worden om de $valid op false te zetten.

Beter is het (zoals je ook in je PSD had gezet) om de $valid afhankelijk te laten zijn of er GEEN fouten zijn opgetreden, dus

   $valid = empty($titleErr) && empyt($nameErr) && /* ... */ empty($messageErr);
}