RaduTodor / TestingPHP

A repository for teaching purposes.
0 stars 0 forks source link

Unnecessary code #5

Open sica07 opened 5 years ago

sica07 commented 5 years ago

https://github.com/RaduTodor/TestingPHP/blob/0a5162811182348031e94ea9a25087a685fa22f6/app/classes/EditClass.php#L21-L27 Problemele cu codul de mai sus sunt urmatoarele:

  1. Daca nu se realizeaza update-ul, PDO v-a arunca o exceptie (lipsesc anumite date mandatorii, ori orice altceva), deci trigger_error nu este necesar. In cel mai rau caz aici ar fi fost nevoie de un try catch, nicidecum de un trigger_error. PHP-ul are un sistem foarte bun de error handling asa ca situatiile in care trebuie ca developerul sa foloseasca trigger_error sunt extrem de rare.

  2. Si daca nu s-ar genera o exceptie si s-ar rula functia de trigger_error, tot un s-ar executa codul mai departe, deci else-ul este inutil.