RogierBe / educom-webshop-basis-1699355256

0 stars 0 forks source link

Deze 2 functies lijken heel erg op elkaar, dit is niet DRY, maak er 1 functie van #20

Open JeroenHeemskerk opened 10 months ago

JeroenHeemskerk commented 10 months ago

De twee functies hieronder lijken heel erg op elkaar https://github.com/RogierBe/educom-webshop-basis-1699355256/blob/f21e2261a5553f3bcb6300204e98bf28e022e76f/data/dataRead.php#L3-L17 Het verschil zit hem alleen in de 'filemode' waarin je hem opent.\

Maak hier 1 functie van waarbij je de mode meegeeft

function openFile($mode){ 
   $dataDir = "C:/xampp/htdocs/educom-webshop-basis-1699355256/data/"; 

   $userFile = fopen($dataDir. 'users/users.txt', $mode); 
   return $userFile; 
 }
JeroenHeemskerk commented 10 months ago

Overigens wordt de $dataDir in de code hierboven 3x gedefinieerd, dit is ook niet nodig. En gebruik RELATIEVE paden en NOOIT beginnen met C:/ omdat als de website wordt ge-deployed hij NOOIT op dezelde plek staat.

Dus $userFile = fopen('data/users/users.txt', $mode);