JBZonda / educom-webshop-oop

0 stars 0 forks source link

Gebruik Excepties niet voor fouten die een gebruiker maakt, maar voor onverwachte situaties #23

Open JeroenHeemskerk opened 1 year ago

JeroenHeemskerk commented 1 year ago

In de upload functies worden veel excepties gebruikt, dit is eigenlijk niet de bedoeling https://github.com/JBZonda/educom-webshop-oop/blob/f1ad947d7bd815ee292adfe7eab0cce20c8b1885/models/ShopModel.php#L39 https://github.com/JBZonda/educom-webshop-oop/blob/f1ad947d7bd815ee292adfe7eab0cce20c8b1885/models/ShopModel.php#L43 https://github.com/JBZonda/educom-webshop-oop/blob/f1ad947d7bd815ee292adfe7eab0cce20c8b1885/models/ShopModel.php#L48 https://github.com/JBZonda/educom-webshop-oop/blob/f1ad947d7bd815ee292adfe7eab0cce20c8b1885/models/ShopModel.php#L54

Beter kan je een $this->errors['image'] zetten en dan false retourneren.