IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

Langage PHP Discussion :

POO agrégation et bonne pratique


Sujet :

Langage PHP

  1. #1
    Membre averti
    Homme Profil pro
    Lead Dev Web
    Inscrit en
    avril 2005
    Messages
    282
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 37
    Localisation : France, Ille et Vilaine (Bretagne)

    Informations professionnelles :
    Activité : Lead Dev Web
    Secteur : Communication - Médias

    Informations forums :
    Inscription : avril 2005
    Messages : 282
    Points : 390
    Points
    390
    Par défaut POO agrégation et bonne pratique
    Salut à tous, je me pose une question concernant les bonnes pratiques / qualités de code / performance. J'ai une classe principale qui va instancier (automatiquement) des classes modules en passant à leur constructeur sa propre classe $this....
    Une fois instancier ces objets sont gardés dans un array de ma classe maitre.

    Je trouve ça plutôt pratique car j'aimerais utilisé certaines méthodes (add_post_meta, add_post_meta, save_post_meta, etc....) de ma classe maitre (Social) dans mes classes modules (Social_Facebook par exemples) .

    Ci-dessous le code (raccourcis) des classes en question.

    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
    31
    32
    33
    34
    35
    36
    37
    38
    39
    40
    41
    42
    43
    44
    45
    46
    47
    48
    49
    50
    51
    52
    53
    54
    55
    56
    57
    58
    59
    60
    61
    62
    63
    64
    <?php
    class Social
    {
    	public static $pluginPath 	= '/Social/';
    	public static $pluginUrl  	= '/Social/';
     
    	private $_modules = array(
    		'Facebook' 	=> true,
    		'Twitter'	=> true,
    		'Googleplus'=> false,
    	);
     
    	private $_modulesObject = array();
     
    	public function __construct()
    	{	
    		self::$pluginPath 	= WPMU_PLUGIN_DIR . self::$pluginPath;
    		self::$pluginUrl	= WPMU_PLUGIN_URL . self::$pluginUrl;
     
    		$this->_loadModules();
    	}
     
    	private function _loadModules()
    	{
    		foreach($this->_modules as $moduleKey => $state) {
    			if (!$state) continue;
     
    			include(self::$pluginPath . $moduleKey . '.php');
    			$var 	= "_s{$moduleKey}";
    			$class 	= "Social_{$moduleKey}";
    			$this->_oModules[$moduleKey] = $var;
     
    			$this->$var 	= new $class($this);
    		}
    	}
     
    	public function add_post_meta($moduleKey, $metaKey, $metaValue)
    	{
    		$this->_postMetaKey[$moduleKey][$metaKey] = $metaValue;
    	}
     
    	public function delete_post_meta($moduleKey, $metaKey)
    	{
    		unset($this->_postMetaKey[$moduleKey][$metaKey]);
    	}
     
    	public function save_post_meta()
    	{
     
    		 update_post_meta($post_id, $meta_key, $meta_value, $prev_value);
    	}
     
     
    <?php
     
    class Social_Facebook
    {
    	public function __construct(&$parent)
    	{
     
    		$this->_parent = $parent;
    	}
    }
    ?>

    Je suis contrarié dans mon idée car l'idéal serait surement d'avoir une classe Social_Data par exemple qui me serve à gérer tout ajout/supression etc..

    De plus je passe à mes classes modules une référence à la classe maitre et cette dernière contient déjà les instance de mes objets modules....
    En faite je me demande si ce n'est pas un peu bourrin comme méthode...

    Vous avez déjà expérimenté ? Des avis techniques ?

  2. #2
    Expert éminent sénior
    Avatar de rawsrc
    Homme Profil pro
    Dev indep
    Inscrit en
    mars 2004
    Messages
    6 142
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 45
    Localisation : France, Bouches du Rhône (Provence Alpes Côte d'Azur)

    Informations professionnelles :
    Activité : Dev indep

    Informations forums :
    Inscription : mars 2004
    Messages : 6 142
    Points : 16 475
    Points
    16 475
    Billets dans le blog
    12
    Par défaut
    Bonjour,

    Tu devrais privilégier l'approche lazy loading : ne charger les modules que lorsque tu en as besoin et pas avant. Il se peut très bien que dans un traitement tu n'ais pas besoin du tout d'un module, il serait dommage d'y consacrer des ressources pour le charger inutilement.

    Passer explicitement par référence des instance est totalement inutile les instances passent déjà par défaut par référence.

    A vouloir tout connecter, c'est le meilleur moyen pour t'y perdre. Sans compter toutes les redondances... Et puis avoir accès à tout partout dans ton code va t'amener à enfreindre un paquet de règles et de bonnes pratiques de la POO. Tu vas te retrouver avec des (trains) de chaînes à n'en plus finir : $this->_parent->modules()->social()->twitter()->database()->getInstance()->create();Bref ingérable, crois-moi.

    Reprends ton design et penses abstraction, encapsulation, réutilisabilité...

  3. #3
    Modérateur
    Avatar de grunk
    Homme Profil pro
    Lead dév - Architecte
    Inscrit en
    août 2003
    Messages
    6 466
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 37
    Localisation : France, Côte d'Or (Bourgogne)

    Informations professionnelles :
    Activité : Lead dév - Architecte
    Secteur : Industrie

    Informations forums :
    Inscription : août 2003
    Messages : 6 466
    Points : 18 965
    Points
    18 965
    Par défaut
    Pourquoi ne pas simplement dérivé d'une classe sociale tes classes spécialisée avec éventuellement si nécessaire avoir une factory d'objet social ?

    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
    class SocialFactory 
    {
       public static function create ($socialType,$login,$password){
          switch ($socialType){
             case 'facebook':
                $social = new SocialFacebook($login,$password);
                break;
             case 'twitter':
                $social = new SocialTwitter ($login,$password);
                break;
             default:
                throw new Exception ('Type social inconnu');
          }
          return $social;
       }
    }
     
    class SocialFacebook extends Social 
    {
    	public function __construct($login,$password)
    	{
    	//...
    	}
    }
     
    $facebook = SocialFactory::create('facebook','foo','bar');
    Pry Framework php5 | N'oubliez pas de consulter les FAQ Java et les cours et tutoriels Java

  4. #4
    Membre averti
    Homme Profil pro
    Lead Dev Web
    Inscrit en
    avril 2005
    Messages
    282
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 37
    Localisation : France, Ille et Vilaine (Bretagne)

    Informations professionnelles :
    Activité : Lead Dev Web
    Secteur : Communication - Médias

    Informations forums :
    Inscription : avril 2005
    Messages : 282
    Points : 390
    Points
    390
    Par défaut
    Citation Envoyé par rawsrc Voir le message
    Bonjour,
    Tu devrais privilégier l'approche lazy loading : ne charger les modules que lorsque tu en as besoin et pas avant. Il se peut très bien que dans un traitement tu n'ais pas besoin du tout d'un module, il serait dommage d'y consacrer des ressources pour le charger inutilement.
    ....
    A vouloir tout connecter, c'est le meilleur moyen pour t'y perdre. Sans compter toutes les redondances...
    En faite il n'y a pas tout le code là, mais les modules ne sont chargés uniquement si ils sont activés et sur les partis de l'application où j'en ai besoin.
    Comme tu le précise c'est la redondance qui ne me plaisait pas dans ce que j'ai fait, un module qui contient une ref a l'objet parent qui contient lui meme l'objet module.... pas très clean, mais quelques avis extérieurs sont toujours bons

    Pourquoi ne pas simplement dérivé d'une classe sociale tes classes spécialisée avec éventuellement si nécessaire avoir une factory d'objet social ?
    J'ai besoin d'une classe maitre qui va tout chargé aux moment requis ( c'est un plugin wordpress ), par contre je vais faie dérivé mes modules d'une classe Social_Module et gérer les méthodes communes pour sauvegarder / gérer les données requises dans une classe Social_Data...

Discussions similaires

  1. Bonnes pratiques pour la POO en Javascript
    Par piemur2000 dans le forum Général JavaScript
    Réponses: 5
    Dernier message: 05/10/2013, 16h33
  2. [Vxi3] Agrégation : bonnes pratiques pour cumuler des valeurs ?
    Par fprovot dans le forum Designer
    Réponses: 0
    Dernier message: 25/07/2013, 20h34
  3. Bonnes pratiques de protections individuelles
    Par Community Management dans le forum Sécurité
    Réponses: 22
    Dernier message: 05/04/2013, 12h47
  4. Bonne pratique MVC & POO
    Par discmat dans le forum Langage
    Réponses: 3
    Dernier message: 14/03/2012, 18h38
  5. [POO] Bonnes pratiques href="javascript:fonction()"
    Par LhIaScZkTer dans le forum Général JavaScript
    Réponses: 20
    Dernier message: 04/04/2009, 19h26

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo