hanaalkadri / educom-webshop-basis

0 stars 0 forks source link

De code om een menu te maken is niet D.R.Y. #26

Open JeroenHeemskerk opened 1 year ago

JeroenHeemskerk commented 1 year ago

De code om het menu te maken bevat veel herhaling: https://github.com/hanaalkadri/educom-webshop-basis/blob/04dc111e0d495bf9f73f9bd95bd64c36478dbf88/index.php#L91-L102

Het stuk <li> ... </li> wordt steeds herhaald met als enige verschil de pagina waar hij naar toe moet en de tekst op de button.

maak een functie showMenuItem($link, $label) die het <li> ... </li> stuk neerzet zodat deze code kan worden:

   showMenuItem('home', " Home ");
   showMenuItem('about', " About ");
   showMenuItem('contact', " Contact ");
   showMenuItem('register', "Register ");
JeroenHeemskerk commented 1 year ago

De huidige code mist een aan het eind

JeroenHeemskerk commented 1 year ago

Waarom heeft de contact menu optie een id?