sofiewillemsen / educom-webshop-database

0 stars 0 forks source link

Het is beter om in datalaag functie geen data uit $_POST te halen, hiermee worden ze minder herbruikbaar #13

Open JeroenHeemskerk opened 2 years ago

JeroenHeemskerk commented 2 years ago

In diverse datalaag functies: https://github.com/sofiewillemsen/educom-webshop-database/blob/95f39eb03794fe00f464c85a3883e3461c2d8a99/changepassword.php#L48-L56 https://github.com/sofiewillemsen/educom-webshop-database/blob/95f39eb03794fe00f464c85a3883e3461c2d8a99/validate.php#L74-L81 https://github.com/sofiewillemsen/educom-webshop-database/blob/95f39eb03794fe00f464c85a3883e3461c2d8a99/validate.php#L110-L113 https://github.com/sofiewillemsen/educom-webshop-database/blob/95f39eb03794fe00f464c85a3883e3461c2d8a99/validate.php#L126-L127 https://github.com/sofiewillemsen/educom-webshop-database/blob/95f39eb03794fe00f464c85a3883e3461c2d8a99/validate.php#L136-L138

haal je de data uit $_POST, dit maakt de functies minder geschikt om hergebruikt te worden, je kan beter de data als parameter meegeven aan de functie. Bovendien doe je geen filtering en trim meer op deze velden, wat ik wel zou adviseren

function changePassword($email, $password, $newPassword) {  } 
function findUserByEmail($email) {  }
function checkRegisterUser($email) {  }
function checkRegisterPassword($password, $password2) {  } // Noem deze checkRepeatPassword, dan kan je hem ook bij changePassword gebruiken