search

Thomas-van-Haastrecht / educom-webshop-basis

0 stars 0 forks source link

Tip voor de `showFormField` functie #24

Open JeroenHeemskerk opened 10 months ago

JeroenHeemskerk commented 10 months ago

Naar aanleiding van #20 heb je deze functie gemaakt https://github.com/Thomas-van-Haastrecht/educom-webshop-basis/blob/01da25d0aa7e38694e1e7f5c91099cd92be23789/index.php#L146-L162

Ik zou echter geen case 'text': maken maar hier de default: case van maken omdat je dan alle andere types (zoals bijv. "password" of "email") met dezelfde functie kan maken, omdat dit allemaal <input>'s zijn

 switch ($type) { 
     default: 
         textField($id, $label, $type, $valsAndErrs); 
         break; 
     case 'radio': 
         radioField($id, $label, $type, $valsAndErrs, $options); 
         break; 
     case 'select': 
         selectField($id, $label, $type, $valsAndErrs, $options); 
         break; 
     case 'textarea': 
         textAreaField($id, $label, $type, $valsAndErrs, $options, $placeholder); 
         break; 
 }