NickKoole / educom-webshop-database-1697464368

0 stars 0 forks source link

Spring de code na een `try {` iets verder in dan is zichtbaar waar het try block eindigd #14

Open JeroenHeemskerk opened 1 year ago

JeroenHeemskerk commented 1 year ago

Als je na een { de regels eronder iets verder inspringt (zoals hier bij deze try) dan wordt het beter leesbaar en is duidelijk waar het try-block eindigt https://github.com/NickKoole/educom-webshop-database-1697464368/blob/a2d07427295f5901b939e783e3734c0c065b3699/validations.php#L153-L166

 try { 
     $user = authenticateUser($email, $password); 

     if (!empty($user)) { 
         $name = $user['name']; 
         $valid = True; 
     } else { 
         $errMail = "Opgegeven emailadres is niet gekoppeld aan een gebruiker of incorrect wachtwoord"; 
     } 

 } 
 catch (Exception $e) { 
     $genericError = "Door een technisch probleem is inloggen helaas niet mogelijk op dit moment.<br>" . $e->getMessage();//$e moet eigenlijk weggeschreven worden naar een log 
 } 
JeroenHeemskerk commented 1 year ago

Overigens zou ik de $e->getMessage() niet aan de gebruiker laten zien, maar apart in een logError()

catch (Exception $e) { 
    $genericError = "Door een technisch probleem is inloggen helaas niet mogelijk op dit moment.";
    logError("Authenticatie faalt voor $email: " . $e->getMessage(); //$e moet eigenlijk weggeschreven worden naar een log 
 }