search

Thomas-van-Haastrecht / educom-webshop-basis

0 stars 0 forks source link

Zet geen businessCode in presentatieCode. Let op dat je geen 'side'-effects codeerd #25

Open JeroenHeemskerk opened 10 months ago

JeroenHeemskerk commented 10 months ago

In de code hieronder wordt een bedankje getoond https://github.com/Thomas-van-Haastrecht/educom-webshop-basis/blob/01da25d0aa7e38694e1e7f5c91099cd92be23789/register.php#L16-L21 maar ook wordt de gebruiker aan de data-file toegevoegd. Dit is een onverwacht 'side'-effect, wat andere ontwikkelaars niet verwachten.

Het is beter om dit bovenin te doen in showRegisterContent

    function showRegisterContent() {
        $valsAndErrs = validation();
        if ($valsAndErrs['valid']) {
            addUser($valsAndErrs);
            displayThanks($valsAndErrs);
        } else {
            displayForm($valsAndErrs);          
        }
    }