Discussion :

[samche]

Bonjour,

J'ai développé une petite application de gestion des interventions en php5/mysql et je rencontre un problème avec la montée en charge de cette application.

Une intervention peut avoir une ou plusieurs actions.
Une action peut avoir un ou plusieurs technicien.
Un technicien appartient à un site d'une société.
Un site appartient à une société.

Concrètement, j'ai une classe pour chacun de ces 5 objets et dans ma base une table pour chaque objet. J'initialise un objet en lui passant un identifiant (la clé primaire), le constructeur fait un select avec l'id en paramètre et initialise les attributs de mon objet avec les résultats de la requête.

C'est super pratique car si je veux le nom de la société du site a du technicien b de l'action c de l'intervention d, ça donne:
$intervention[d]->action[c]->technicien[b]->site[a]->societe->nom

Mais voilà, lorsque j'affiche 30 interventions, chacun de mes objets génère une requete à l'initialisation. Du coup, j'arrive très vite à atteindre entre 1000 et 2000 requêtes pour l'affichage d'une page (30 interventions, 2 ou 3 actions chacune, 2 ou 3 techniciens chacune, ...) !!!

Existe-t-il des méthodes de développement (voire un framework) où, lorsqu'un objet est déjà initialisé, php rappelle l'objet instancié sans refaire la requête ?

Je n'ai qu'une trentaine de techniciens. Si le même technicien intervient sur plusieurs actions différentes, je fais ma requête pour chacune des actions où il intervient (c'est mal). Ce que j'aimerais, c'est faire la requête qu'une seule fois et ensuite, réutiliser l'objet ainsi obtenu autant de fois que nécessaire (c'est bien).

D'après mes recherches, je crois que la technique que j'utilise est du lazy loading. A mon avis, je me plante quelque part pour optimiser tout ça.

Je pensai sinon charger l'ensemble des objets au démarrage de la page mais comment savoir lesquelles j'aurai besoin ? Il est impensable de charger l'ensemble des actions par exemple (plus de 16.000 aujourd'hui après seulement 6 mois d'utilisation).

Merci pour vos réponses.

[kaymak]

lazy loading exactement.

l'idée est de ne pas appeler des objets périphérique à l'instanciation, mais JIT, just in time, bref au moment souhaité.

Pour cela, utilise des méthodes de get, ou les accesseurs magiques de PHP.

Schématisons la classe intervention.

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
class Intervention{
 
     protected $p_id_technicien;
     protected $p_technicien;
 
     public function __construct( $id=null ){
          // faire quelque chose pour lire les propriétés de l'objet 
          // genre id, nom, id_technicien ect
          // Cependant on ne charge pas maintenant l'objet technicien
     }
 
     public function getTechnicien(){
          if( $this->p_technicien == NULL )
               $this->p_technicien = Technicien::lireParId( $this-> p_id_technicien );
          return $this->p_technicien;
     }
}

Et Voilàà, l'objet technicien est lazy loader. Simple non ?

Pour refaire exactement comme cela :
$intervention[d]->action[c]->technicien[b]->site[a]->societe->nom

utilises __get sur le même principe.

Ce que j'aimerais, c'est faire la requête qu'une seule fois et ensuite, réutiliser l'objet ainsi obtenu autant de fois que nécessaire (c'est bien).

Ca c'est encore autre chose, c'est un système de cache.
Je laisse la main à d'autres.
Faudra te tourner vers apc, eaccelerator ect pour utiliser de la mémoire partagée.

a plus

[samche]

ok pour le lazy loading.

Mon soucis vient de là:

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
24
25
26
27
28
29
class Intervention
{
    var $id;         // Identifiant de l'intervention
    var $libDemande; // Libelle de la demande
    var $actions;    // Tableau d'actions
 
    function Intervention( $id = null )
    {
        global $db;
        if ( isset( $id ) )
        {
            $this->id = intval( $id );
 
            $table = new DB_Table( T_INTERVENTIONS );
            $champs = @mysql_fetch_object( $table->load( $this->id ) );
 
            $this->libdemande = $champs->demande;
 
            $this->actions = array( );
            $table = new DB_Table( T_ACTIONS );
            $reqActions = $table->loadSpec( "id_intervention", $this->id, "dateAction" );
            while ( $champs = @mysql_fetch_object( $reqActions ) )
            {
                $act = new Action( $champs->id );
                array_push( $this->actions, $act );
            }
        }
    }
}

A l'initialisation, je lance ma requête sur les actions pour les avoir à dispo.

Tu me diras, c'est anodin, et je pourrais faire une fonction getActions( ) que j'appelle au moment où j'ai réellement besoin des actions.

Oui mais voilà, ça m'oblige maintenant à revoir l'ensemble de mon code. C'est une boulette que je ne ferai plus

Je n'avais pas pensé au système de cache. Je vais de ce pas me renseigner sur les pistes que tu viens de me donner. Merci

[kaymak]

A l'initialisation, je lance ma requête sur les actions pour les avoir à dispo.

Tu me diras, c'est anodin, et je pourrais faire une fonction getActions( ) que j'appelle au moment où j'ai réellement besoin des actions.

Oui mais voilà, ça m'oblige maintenant à revoir l'ensemble de mon code. C'est une boulette que je ne ferai plus

Hey des erreurs on en fait tous, même les forgerons..
Et à cela je ne peux rien y faire, à part te signifier que 5 - 6 classes avec peut être 20 - 25 accesseurs, ça ne me parait pas être le bout du monde.
Mais cela ne concerne que moi.

Pour le système de cache http://www.php.net/apc

bye

[zevince]

je crois que propel gere le lazy loading :
http://propel.phpdb.org/trac/