Variable statique dans Respository : bonne pratique ?

Bonjour à tous,

Dans un respository, j'ai une fonctionn countCabinets() qui me revoit simplement un nombre. Cette fonction sera appelé à de nombreux endroits dans une même page : dans le controller, dans certains services et même dans le menu.

Quelle est la meilleure pratique selon-vous dans ce cas, afin de ne pas répéter la requête dans une même page ? Créer dans le respository une variable statique stockant dans un tableau tous les résultats countCabinets() afin de le rappeler ensuite est-il une bonne pratique ?

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

protected static $count_cabinets;   public function countCabinets() { if (isset(self::$count_cabinets)) { return self::$count_cabinets; } $rsm = new ResultSetMappingBuilder($this->_em); $rsm->addScalarResult('count_cabinets', 'count_cabinets', 'integer'); $query = $this->_em->createNativeQuery(' SELECT COUNT(c.id) AS count_cabinets FROM cabinet c WHERE c.date_suppression IS NULL ', $rsm); return self::$count_cabinets = $query->getSingleScalarResult(); }

---

J'ai un doute sur la meilleure façon d'aborder ce problème ...

Merci d'avance :)

Non, c'est une mauvaise pratique.
Si tu veux optimiser ce genre de traitement, utilise un système de cache de requêtes

Salut et merci de ta réponse, je m'en doutais un peu :)

Cependant si je sais utiliser le système de cache [ en envoyant un ID et, éventuellement, une durée ], ça me parait compliqué à mettre en place, imaginons que countCabinet() ait eu un argument quelconque repris dans le Where, ça aurait multiplier le nombre d'ID à enregistrer en cache par les possibilités de ce dit-argument. De plus, le countCabinet() original est une requête avec plusieurs jointures ... il faudrait donc vider ce cache (donc, l'ID correspondant) dès qu'il y a une intervention sur une des tables jointes ... vraiment compliqué à mettre en place :-S

Si tu as quelques conseils, n'hésites pas :)

J'ai du mal à comprendre ta logique.

Soit ta requete est identique et ton système(qui est un cache mal foutu) fonctionnera... soit elle sont différentes et ca n'ira pas. (tu retournera le resultat de la première requetes, pas les suivantes.

Le mieux dans ce cas la est, selon moi d'utiliser qq chose du genre :

Code:

---

1 2 3 4 5 6 7

$qb = $this->createQueryBuilder('l'); $qb->andWhere('l.urlName IS NOT NULL') $query = $qb->getQuery(); $query->useResultCache(true); $query->setResultCacheLifetime(3600); return $query->getResult();

---

Si tu as peur que ton cache soit invalide, soit tu défini une valeur faible(ton cache ne sera pas invalide longtemps) soit tu défini juste des bons index en SQL et ca ira assez vite, soit tu regarde si il existe un moyen d'invalider un cache doctrine.

bon courage.
La solution choisie m'interesse si tu peux la donner.

pierre

Salut et merci de ta réponse :)

Effectivement passer par un système de cache tel que APC me parait plus cohérent, mais le soucis c'est que la requête peut changer en fonction de l'utilisateur connecté. Par exemple, imaginions que countCabinet() ait un ou plusieurs arguments qui se retrouvent dans le Where, ces arguments ne doivent pas se marcher dessus dans la construction de l'ID.

Voici un code que je viens de "bricoler" :

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 28 29 30 31 32

public function countCabinets() { $rsm = new ResultSetMappingBuilder($this->_em); $rsm->addScalarResult('count_cabinets', 'count_cabinets', 'integer'); $query = $this->_em->createNativeQuery(' SELECT COUNT(c.id) AS count_cabinets FROM cabinet c WHERE c.date_suppression IS NULL ', $rsm); $id = $query->getSQL(); /*   * On charge le moteur "APC Cache"   */ $cacheDriver = new \Doctrine\Common\Cache\ApcCache(); /*   * Si le résultat de la requête est déjà en cache, on la renvoit   */ if ($cacheDriver->contains($id)) { return $cacheDriver->fetch($id); } else { /*   * Sinon, on enregistre la requête en cache   */ $result = $query->getSingleScalarResult(); $cacheDriver->save($id, $result); return $result; } }

---

Ici, il n'y a pas d'argument - pour faire simple - mais imaginons que la requête ait un argument qui fait changer le Where (typiquement un ID provenant d'une autre table) ... c'est pour ça que j'ai utilisé la requête entière comme ID ( avec $query->getSQL() ), mais je doute, la encore, du coté bonne pratique ... :-S

Je précise que je suis dans ce cas d'un back-office, donc un peu plus complexe qu'un blog avec des articles ... il s'agit de mettre en cache par exemple les X :
- Nombre d'articles (X)
- Nombre d'articles dans catégorie A (X)
- Nombre d'articles dans catégorie B (X)
- Nombre de catégorie (X)

Ok, quelle taille fait ta base d'article.

Parce que je pense vraiment que tu t’embêtes pour rien...
Pour chaque requêtes SQL, tu auras une association entre ta requete SQL et le resultat mis dans un tableau associatif.
Donc select count(*) from article where categorie = A sera bien différent de select count(*) from article where categorie = B

Si tu rapelle 10 fois la requete 1, tu auras 10 fois le même resultat et juste une seule vraie requete en BDD.
Dés que tu fera une requete qui n'est pas encore connue ou expirée... tu auras le même comportement appliqué sur cette nouvelle requete, en parralele.

Pierre