[POO] Singleton ou référence ?

Salut à tous !

Je me pose actuellement une question, concernant les objets transversaux tels que l'objet gérant la base de données, classes de log etc ...

Comment passer ces objets aux autres objets de mon appli ? Par référence ou en utilisant le pattern design singleton ?

Par exemple, pour un objet user, faut-il créer une variable membre et passer par référence l'objet gérant la BDD lors de l'instanciation du user ? (accessible donc par $this->db a chaque fois que l'on veut envoyer une requête dans l'objet user).La classe de BDD serait donc instanciée une fois et passée par référence a chaque fois.

OU

Faut-il a chaque fois que l'on veut passer des requêtes dans une méthode la classe user commencer par instancier la classe gérant la BDD en utilisant un pattern design singleton ? (du type : $db = MySQL::singleton($params) ) La classe de BDD serait donc instanciée (singleton) dans chaque méthode de chaque objet qui a besoin de passer des requêtes.

Ou alors c'est strictement la même chose ? En terme d'écriture, personnellement je préférerai utiliser l'instanciation de singleton a chaque fois, c'est pour ça qu'existe ce type d'instanciation je pense. Par contre en terme de performance je ne sais pas ce que ca donne, si beaucoup de méthodes s'enchainent, je ne sais pas comment est gérée l'instanciation par singleton, si c'est aussi rapide d'accès qu'un travail sur la référence.

Peut etre encore je suis complètement dans le faux, et qu'il ne faut pas procéder de la sorte, j'écoute vos conseils avisés.

Merci beaucoup et bonne fin de journée

ZeGuizmo

Avec la fonction recherche j'ai raté ce post, qui figure quelques lignes en dessous du mien :

http://www.phpcs.com/infomsg_GESTION-USER_1115107.aspx#4

Ici neige d'hiver propose d'instancier par singleton puis de le passer dans une variable $user->db
Est-ce meilleur que de passer une reference vers l'objet de base de données au constructeur du user ?

J'imagine que c'est meilleur que d'instancier par singleton a chaque méthode, mais je n'en suis pas certain.

Car il y a un problème pratique, je trouve, a créer une variable membre 'db' pour chaque objet qui necessite une connection, c'est le debug. Quand on fait un joli print_r de notre objet user pour voir un peu quelle connerie on a pu faire pour qu'il s'appelle alfred au lieu de marcel, et qu'on utilise des objets debase de données un poil conséquents (type MDB2 de pear) ben le print_r prend des dizaines de pages. Quand il y en a qu'un, ca va, mais quand on travaille sur une liste de quelques dizaines de user, imagnez un peu les dégats.

Donc je reformule ma question, après tout ça :

En terme d'éthique et de performance, puis-je sérieusement instancier un objet qui peut être assez important (type gestion de base de données) par singleton à chaque méthode qui en a besoin, ou cela ne se fait pas et on préférera utiliser une variable membre de l'objet contenant la base de données.
Si la deuxieme solution est la bonne, est-il mieux de passer l'objet par référence au constructeur de classe, ou vaut-il mieux l'instancier dans le constructeur de classe en singleton ?

Merci, et désolé si je ne suis pas tres clair.

ZeGuizmo

Salut !

Merci de ta réponse très claire et détaillée, cela confirme ce que j'avais compris.

En revanche il me reste une interrogation.

Dans un objet, quand on déclare une variable membre assez lourde (genre db ..) et qu'on fait du print_r à la chaine pour debugger, cela devient très illisible. (quand on manipule une liste d'objets user par exemple, on se retrouve avec 10 fois l'objet db de printé, on ne retrouve plus rien à l'écran)

Je voudrais supprimer l'emploi d'une variable membre contenant la db (ou classe de log, ou classe de formulaire...) pour justement ne garder que les informations sémantiquement liées à l'objet.

Est ce que cela nuirait significativement aux performances et à la qualité du code si je déclare dans chaque méthode les objets dont j'ai besoin en singleton, au lieu d'utiliser une variable membre de l'objet que je manipule ?

Merci beaucoup et bonne fin d'après midi !
(et bon week end pour les chanceux qui font le pont)

ZeGuizmo

Ok, je vais opter pour la méthode mixte que tu décris, je trouverai une solution pour le debug :)

Merci encore !

ZeGuizmo

Ou ArrayObject, mais c'est un peu plus compliqué.

Oh oui merci pour la méthode toString, je ne savais pas qu'elle existait pour tous les objets !

Ca fonctionne à merveille :)

Je n'ai pas encore regardé ArrayObject, ce n'est pas nécessaire car toString suffit, mais par contre je vais y jeter un coup d'oeil, autant mettre le plus d'outils possible dans ma besace.

ZeGuizmo