Discussion :

[kevin254kl]

Bonjour à tous,

Pourriez vous m'aider s'il vous plaît?
Je remarque que dans certains code il-y a rarement des exceptions par exemple pour ce projet laravel https://github.com/bestmomo/laravel5...ree/master/app ou encore la démo symfony exemple avec ce controller https://github.com/symfony/demo/blob...Controller.php la fonction search.

Ma question est donc quand utiliser les exceptions et les try catch ? Dans le cas de ces projets pourquoi y'en a t-il pas
Exemple en laravel ou je dois refactorer une fonction ici le try catch est justifier d'après vous?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
 try {
            $pro = $pro::findOrFail();
           $contact = $this->contact->findOrFail($pro->id);
          $templateList = $this->smsRespository->getTemplateList();
 
 
        $form = $this->createForm(ProSmsType::class, null, [
            'action' => $this->getUrlAdminMemberSms($this->request->get('id')),
            'attr' => ['class' => 'form-callback'],
            'sms-template-list' => $templateList
        ]);
 
               $form->handleRequest($request);
                if ($form->isValid() && $form->isSubmitted()) {
                    $data = $form->getData();
                    $cellphone = $contact->cellPhone;
                    if (!empty($cellphone)) {
                         event(new ProSmsEvent($pro, $data['sms_template_content']));
                        return redirect('admin::pro::view', ['id' => $pro->id]);
                    } 
                     throw new \Exception("Problème avec le numéro");
 
                }
        } catch (Exception $e) {
            return response()->json(['messages' => [['title' => 'Erreur : ' . $e->getMessage()]]], 500);
        }

Merci

[insaneness]

Bonjour,
Les exceptions sont là pour remonter une erreur critique empêchant la bonne exécution du programme.
Il faut les utiliser sur des bouts de codes sensible et requis pour la suite d'une tâche.

Ex: Appel à une API externe pour obtenir des datas qui seront utilisées par la suite. Si l'appel API se passe mal alors on renvoie une exception.

Ce ne sera pas le cas lors d'erreurs de validation de formulaire par exemple.

[Celira]

Cela dit, tu peux très bien définir tes propres exceptions que seront différentes des Exception de base, et qui te permettront une gestion plus fine. PHP: Exceptions étendues - Manual

Exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
// on définit une classe pour les erreurs de validation
class MyValidationException extends Exception {
 
    public function __construct($field, $message, $code = 0, Exception $previous = null) {
 
        // on mémorise le champ
        $this->_field = $field;
        // on construit l'exception de base
        parent::__construct($message, $code, $previous);
    }
 
    public function getMessageWithField() {
        return "Erreur sur le champ {$this->_field} : ".$this->getMessage();
    }
}

Et on s'en sert pour les problèmes de validation :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
try {
    $pro = $pro::findOrFail();
    $contact = $this->contact->findOrFail($pro->id);
    $templateList = $this->smsRespository->getTemplateList();
 
 
    $form = $this->createForm(ProSmsType::class, null, [
        'action' => $this->getUrlAdminMemberSms($this->request->get('id')),
        'attr' => ['class' => 'form-callback'],
        'sms-template-list' => $templateList
    ]);
 
    $form->handleRequest($request);
    if ($form->isValid() && $form->isSubmitted()) {
        $data = $form->getData();
        $cellphone = $contact->cellPhone;
        if (!empty($cellphone)) {
            event(new ProSmsEvent($pro, $data['sms_template_content']));
            return redirect('admin::pro::view', ['id' => $pro->id]);
        } 
        throw new \MyValidationException("Numéro", "Le champ doit être renseigné");
 
    } 
} catch (MyValidationException $e) {
    // traitement particulier pour les erreurs de validation : Erreur 400 (Bad Request)
    return response()->json(['messages' => [['title' => 'Erreur : ' . $e->getMessage()]]], 400);
} catch (Exception $e) {
    // les autres Exception (genre plantage technique) :  Erreur 500 (Internal Server Error)
    return response()->json(['messages' => [['title' => 'Erreur : ' . $e->getMessage()]]], 500);
}

[kevin254kl]

Merci pour vos réponses

Je fais moi même mes exceptions, mais la question c'est quand cela vaut le coup d'être utilisé?
Après recherche et d'après cette article très petinant https://culttt.com/2014/04/09/use-exception/ et comme dit plus haut c'est pour cause exceptionnel.
Par exemple dans le code que j'ai donné, le try catch apporte t-il réellement quelque chose? Ici si je n'utilise pas le try catch cela me donne ça:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 public function create(Pro $pro, SmsTemplateRepository $smsRespository, ContactRequest $request)
    {
        $pro = $pro::findOrFail(1, ['cellPhone', 'pro_id']);
        $contact = $this->contact->findOrFail($pro->id);
        $templateList = $this->smsRespository->getTemplateList();
 
 
        $form = $this->createForm(ProSmsType::class, null, [
            'action' => $this->getUrlAdminMemberSms($this->request->get('id')),
            'attr' => ['class' => 'form-callback'],
            'sms-template-list' => $templateList
        ]);
        $form->handleRequest($request);
 
        if ($form->isValid() && $form->isSubmitted()) {
            $data = $form->getData();
            event(new ProSmsEvent($pro, $data['sms_template_content']));
            return redirect('admin::pro::view', ['id' => $pro->id]);
        }
 
        $data = $this->format($pro, $contact);
        return view('admin.pages.builder.sms', ['form' => $form->createView(), 'proId' => $pro->id, 'data' => $data]);
    }

Dans ce code c'est vraiment pertinent d'utiliser un try catch (c'est du laravel) sachant que finalement on a pas d'accès tier juste des accès en bdd et un event?

Merci

[kevin254kl]

Je m'auto répond,

Comme expliqué une exception doit-être exceptionnel et donc une action qui peut ce produire occasionnellement.
Ici on peut nettoyer le script par exemple avec les findOrFail qui gère l'exception.

[ABCIWEB]

Je m'auto répond,

Comme expliqué une exception doit-être exceptionnel et donc une action qui peut ce produire occasionnellement.

Heu, c'est très vague comme définition et peut prêter à confusion, gérer des exception ne doit pas être considéré comme exceptionnel, c'est même conseillé comme bonne pratique de développement.

Ici on peut nettoyer le script par exemple avec les findOrFail qui gère l'exception.

...Mais il est vrai que quand on utilise un framework, la plupart des exceptions sensibles sont déjà gérées en interne et dans ce cas il est inutile de les doubler. Cela dit en cas de doute, mieux vaut doubler cette gestion des exceptions que de n'en avoir aucune, car la non gestion des erreurs peut dans certains cas rendre un code plus vulnérable au piratage.

Cela permet aussi d'envoyer des messages au visiteur pour lui signaler des erreurs de saisie ou une bdd momentanément indisponible par exemple, plutôt que d'avoir un formulaire qui ne fonctionne pas sans savoir pourquoi. Donc dire que que la gestion des exceptions doit-être exceptionnelle n'est pas une bonne réponse générique.