[POO] is_object() ou objet vide

Version imprimable

14/05/2009, 10h32
lerieure

[POO] is_object() ou objet vide
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:

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 !
14/05/2009, 14h06
mathieu

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

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

Code:

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;
 
  }
 
  // ...
 
}

14/05/2009, 14h52
lerieure

Exemple

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
14/05/2009, 15h48
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.
14/05/2009, 16h13
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
14/05/2009, 16h17
mathieu

Citation:

Envoyé par 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.

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
14/05/2009, 16h20
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:

@$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
14/05/2009, 16h51
lerieure

Merci beaucoup tu m'as bien aidé =)
14/05/2009, 20h22
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:

1 2 3 $monObjet1 = "o_auteur"; $monForm->{$monObjet1}->get_name();
Immagine en ayant différent "o_auteur" dans un tableau sur lequel tu boucle...