lydiavangammeren / educom-webshop-database

0 stars 0 forks source link

Er staat een fgets in de file op een plek waar het niet triviaal is, documenteer deze #12

Open JeroenHeemskerk opened 1 year ago

JeroenHeemskerk commented 1 year ago

onderstaande fgets op regel 7 staat op een plek die niet logisch lijkt zeker omdat je niets lijkt te doen met de $line https://github.com/lydiavangammeren/educom-webshop-database/blob/772ca8f609e22dd8d41900366131c87269093c2e/file_repository.php#L5-L9

de fgets is echter wel heel bewust daar neergezet, zet er in commentaar achter waarom, dat voorkomt dat iemand het per ongeluk weghaald.

lydiavangammeren commented 1 year ago

Eerlijk gezegd had ik dit ergens gelezen, en heb ik het overgenomen. Wat is precies het nut van de fgets daar ook al neer te zetten?

JeroenHeemskerk commented 1 year ago

Om de 'header' regel in te lezen

lydiavangammeren commented 1 year ago

De |email|name|password bedoel je, toch?

JeroenHeemskerk commented 1 year ago

Ja dat bedoel ik