Discussion :

[laurentSc]

Bonjour,

je souhaite contrôler le type des arguments passés au constructeur de ma classe . Pour éviter un message d'erreur si l'utilisateur met des paramètres d'un type non accepté, je pensais mettre un try-catch. Mais ça me génère un message d'erreur. Pourquoi ?

Code php :

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
 
<?php
declare(strict_types=1);
namespace Classes\Olivier;
 
class Country
{
  // no setter as the constructor is sufficient
    private $name;  //string
    private $region;  //string 
 
      /* Constructor
      */
    public function __construct(string $name, string $region='') //the 2nd parameter is optional
    {
      try {
        $this->name=$name;
        $this->region=$region;
      }
      catch {
        echo "format incorrect<br/>";
      }
    }

Parse error: syntax error, unexpected '{', expecting '(' in C:\wamp64\www\ticket2\Classes\Olivier\Country.php on line 20

Je suis en PHP7.3.5

[Invité]

Bonjour,

• PHP - Les exceptions

[rawsrc]

Salut,

Pour éviter un message d'erreur si l'utilisateur met des paramètres d'un type non accepté

si le typage des variables te pose un problème, je ne vois pas pourquoi tu ne le retires pas ?
D'ailleurs, je ne comprends pas trop ton problème et personnellement je ne retirerais pas le typage

Pour ton apprentissage :
Un constructeur doit éviter autant que possible de tirer des exceptions.
S'il le fait, c'est l'instanciation de la classe qui doit être encapsulée au sein d'un bloc try ... catch ... externe.
Un constructeur ne peut avoir de return sauf un return (void).

Dis-moi, tu as avancé sur la POO pendant ton sommeil que tu sois confronté aujourd'hui à ce genre de problématique complexe ?
Ce genre de problème ne se présente que quand tu fais de l'architecture avancée.

[laurentSc]

Bonjour,

je ne suis dispo pour vous répondre que maintenant.

Déjà merci à Jérôme car bien que je sache qu'on peut faire des try-catch, je l'avais encore jamais fait...

Ce que je voudrais, c'est imposer le type des variables (d'où le declare(strict_types=1);), et que si l'utilisateur essaye d'instancier la classe en passant des variables d'un mauvais type, ça affiche un message d'erreur mais ne plante pas le script.

Je m'attendais à un message d'erreur avec ce code :

Code php :

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

1
2
3
4
5
6
7
 
try{
   $country1=new Country(33); echo "country:". $country1."<br/>";
}
catch (Exception $e){
    echo "format incorrect<br/>";
  }

country.php :

Code php :

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
 
<?php
declare(strict_types=1);
namespace Classes\Olivier;
 
class Country
{
  // no setter as the constructor is sufficient
    private $name;  //string
    private $region;  //string 
 
      /* Constructor
      */
    public function __construct(string $name, string $region='') //the 2nd parameter is optional
    {
        $this->name=$name;
        $this->region=$region;
        echo "type=".gettype($name)."<br/>";
    }
...

mais non, et le type affiché est string et non int. Pourquoi ?

[rawsrc]

Salut,

t'as juste oublié la directive : declare(strict_types=1); au début du script contenant le try.
Cette directive doit être OBLIGATOIREMENT présente sur tous les scripts pour que le moteur l'applique.

Si tu l'ajoutes, voici ce que tu obtiens :

Fatal error: Uncaught TypeError: Argument 1 passed to Classes\Olivier\Country::__construct() must be of the type string, int given, called in

Et là, ça fonctionne comme attendu

[laurentSc]

Merci pour l'explication sauf que l'attendu, c'est pas une fatal error de PHP mais que le script détecte le mauvais type et affiche un message d'erreur et que le script ne plante pas...

[rawsrc]

le moteur détecte bien le mauvais type et il plante ce qui est tout à fait normal. C'est ce qui assure la cohérence d'une part et un niveau de sécurité accru d'autre-part.
Cette rigidité oblige le développeur à une très grosse rigueur et c'est à lui de s'assurer que ce qu'il présente en entrée de fonction correspond parfaitement à ce qui est attendu.
Le moteur ne fera pas ce travail à ta place : si les types ne correspondent pas : le jeu est terminé.
Donc en amont, soit tu t'assures que les types de données collent parfaitement, soit tu les castes explicitement mais dans tous les cas, s'il y a un problème à posteriori, ça ne sera uniquement que de la faute du développeur.

Bref, codage autrement plus propre et sécurisé.