Discussion :

[lerieure]

Bonjour,

Voila j'ai une classe qui a pas mal d'attribut objet.
Dans mon projet ces objet ne sont pas obligatoire donc ils peuvent ne pas etre instanciés .. ainsi je me retrouve a faire plein de

Code :

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

1
2
3
if( is_objet(MonObjet->getMonAutreObjet()){
 a= MonObjet->getMonAutreObjet()->getA();
}

Ma question est la suivante : est ce qu'il est mieux de créer des objets vides ou de faire tous les test if(is_object(..)){...} ?

Merci d'avance !

[mathieu]

montre nous un exemple d'attribut qui n'est pas toujours instancié

[keaton7]

Ca manque d'exemple effectivement, alors je me lance dans une suggestion au hasard :

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
 
 
class Element {
 
  // Proprietes
  $objet_1 = NULL;
  $objet_2 = NULL;
  $objet_3 = NULL;
  $objet_4 = NULL;
  $objet_5 = NULL;
 
  public function __construct() {
 
    // ton constructeur
 
  }
 
  public function actionElement {
 
    if ($this->objet_1) return TRUE;
    else return FALSE;
 
  }
 
  // ...
 
}

[lerieure]

Tout bêtement j'ai une classe FORMULAIRE qui prend en private o_auteur

o_auteur est un objet d'une classe USER

imaginons que le formulaire peut etre anonyme et que par exemple il est possible de ne pas remplir le champ qui aurait instancier l'objet o_auteur qui aurait fait x traitements pour x raisons.

lors de l'enregistrement du formulaire dans la base si je veux enregistrer son nom,
je ferais quelque chose du genre : $MonForm->get_o_auteur()->get_name()
ainsi si le champ n'est pas rempli, j'ai une magnifique fatal error qui me dit que j'utilise une méthode d'une classe sur un non - objet.

Dès lors je suis obligé d'ajouter une condition avec is_objet() pour savoir si $MonForm->get_o_auteur() est un objet ou j'instancie l'objet quoi qu'il arrive sauf que si le champ n'est pas renseigné, j'instancie un objet vide.


ça m'arrive X fois dans mon projet tellement que quand je dois enregistrer je fais presk un if(is_object($x)) à chaque champ.


Donc ma question était qu'est ce qui était le mieux : objet vide ou faire des conditions is_object()

J'espere que j'ai été assez clair ..

Merci d'avance

[keaton7]

Je ne me suis jamais vraiment trouve dans cette situation, mais peut-etre pourrais-tu creer un auteur anonyme,et l'instancier au debut de ta classe de la meme maniere que tu appelles un auteur existant.

[lerieure]

oui alors mon objet anonyme serait un objet null sinon il serait plus anonyme =)
Mon cas est un Exemple. Il y a beaucoup d'autre exemple..
Sans avoir vraiment l'exemple concret le mien était beaucoup trop long a expliquer .., J'aurais aimé savoir si dans l'absolu c'était mieux de faire des is_object() ou des objets vide .. en terme de performance.

voila Merci beaucoup pour votre participation

[mathieu]

Je ne me suis jamais vraiment trouve dans cette situation, mais peut-etre pourrais-tu creer un auteur anonyme,et l'instancier au debut de ta classe de la meme maniere que tu appelles un auteur existant.

ou encore mieux, tu pourrais créer deux classes UtilisateurNormal et UtilisateurAnonyme qui héritent de la classe USER et dans l'attribut "auteur" tu pourras mettre une instance d'une de ces 2 classes

[keaton7]

Mon experience sera insuffisante pour te donner une reponse formelle. Cependant, la logique laisse penser qu'instancier un objet est plus lourd que de ne pas le faire et is_object() ne devrai pas s'allouer plus de memoire que ca ... Apres, il est clair que programmer en objet en PHP, c'est deja un sacrifice de performances, alors dans cette voie, il est parfois preferable de preferer une logique stricte plutot qu'un code plus sale qui va plus vite ...

La rigueur, c'est les elements vides, et les perfs, c'est les conditions partout. Je pense cependant qu'il doit exister une 3eme solution qui serait d'utiliser "l'operateur de controle d'erreur" (@). Ca eviterai la fatal error ...

Quelque chose du genre :

Code :

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

@$MonForm->get_o_auteur()->get_name()

Si tu choisis cette solution, c'est a double tranchant, il te faudra gerer les cas d'erreurs par toi meme.

Un petit lien :
http://uk2.php.net/manual/fr/languag...rorcontrol.php

[lerieure]

Merci beaucoup tu m'as bien aidé =)

[yannux]

L'utilisation d'un @ devant est une solution plutôt crade.... :s

Pourquoi ne pas avoir un système, avec un tableau qui contiendrais le nom de chaque objet créer, tu boucle dessus si il n'est pas vide et tu appel les objets dedans.....

Php permet de faire quelque chose du genre :

Code :

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

1
2
3
 
$monObjet1 = "o_auteur";
$monForm->{$monObjet1}->get_name();

Immagine en ayant différent "o_auteur" dans un tableau sur lequel tu boucle...