search

JBZonda / educom-webshop-oop

0 stars 0 forks source link

Zet geen business logica in de presentatie laag #20

Open JeroenHeemskerk opened 1 year ago

JeroenHeemskerk commented 1 year ago

https://github.com/JBZonda/educom-webshop-oop/blob/6e2db592597419c020bb1d80cb1477340da43a5c/webshop.php#L17-L23 https://github.com/JBZonda/educom-webshop-oop/blob/6e2db592597419c020bb1d80cb1477340da43a5c/webshop.php#L36-L42

De functie isUserLoggedIn() is een business logica functie, beter is het om in de $data een boolean allowedToBuy of allowedToOrder te zetten in een business functie en daar de if op te laten afgaan

Ook get_cart() is business logica, beter is het om de cart dan in de $data te zetten

JeroenHeemskerk commented 1 year ago

Als we straks voor OOP alle presentatie logica in Views gaan zetten zal dit ook een probleem gaan worden!

JeroenHeemskerk commented 1 year ago

Bovenstaande 2 functie zijn identiek*, dit is niet D.R.Y. * op de return pagina na dan

Maak er een functie van die de return pagina als parameter krijgt.