Tentative d'injection de dépendances via un service

Version imprimable

Bonjour,

J'ai actuellement un Bundle Symfony2 qui a été écrit sans injection de dépendances, j'ai dans mon contrôleur les méthodes suivantes :

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
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
65
66
67
68
69
70
 
private function calulcateMaterial(Product $product, $tab = null, $percentage_field = null, $simulationMaterials = false)
    {
        $em = $this->getDoctrine()->getEntityManager();
 
        // Initialize array to 11 values set to 0
        if ($tab != null) {
            $m = $tab;
        } else {
            $m = array_fill(0, 11, 0);
        }
 
        foreach($product->getAssemblies() as $assembly) {
            foreach($assembly->getParts() as $part) {
 
                if ($part->getParent() != null) {
                    $quantity = $part->getParent()->getQuantity();
 
                    $base = $part->getMaterial();
 
                    if ($base != null) {
 
                        if ($part->getParam2() != null) {
                            $val = $part->getParam1() * $part->getParam2();
                        } else {
                            $val = $part->getParam1();
                        }
 
                        // Simulation for parts
                        if ($percentage_field != null && !$simulationMaterials) {
                            $percentage = $part->getParent()->$percentage_field();
                            if ($percentage != null) {
                                $val = $val * $percentage;
                            }
                        }
 
                        // Simulation for materials
                        if ($simulationMaterials) {
                            if ($part->getMaterial() != null) {
                                $simulation = $em->getRepository('ArtoAcvBundle:SimulationMaterial')->findOneBy(array(
                                    'product'  => $product->getId(),
                                    'material' => $part->getMaterial()->getId()
                                ));
 
                                if ($simulation != null && $simulation->$percentage_field() != null) {
                                    $val = $val * $simulation->$percentage_field();
                                }
                            }
                        }
 
                        $mul = $assembly->getQuantity() * $quantity * $val;
 
                        $m[0]  += $base->getRmd() * $mul;
                        $m[1]  += $base->getEd() * $mul;
                        $m[2]  += $base->getWd() * $mul;
                        $m[3]  += $base->getGwp() * $mul;
                        $m[4]  += $base->getOdp() * $mul;
                        $m[5]  += $base->getAt() * $mul;
                        $m[6]  += $base->getPocp() * $mul;
                        $m[7]  += $base->getAa() * $mul;
                        $m[8]  += $base->getWt() * $mul;
                        $m[9]  += $base->getWe() * $mul;
                        $m[10] += $base->getHwp() * $mul;
                    }
                }
            }
        }
 
        return $m;
    }

Code:

1
2
3
4
5
6
 
private function calculateAll(Product $product)
    {
        $tab = $this->calulcateMaterial($product);
        return $tab;
    }

La méthode calculateAll est appelée dans une de mes actions, seulement toutes ces méthodes n'ayant rien à faire dans mon contrôleur, j'ai voulu les externaliser.

J'ai donc créé un service Calculator qui va faire les calculs.

Mon services.xml ressemble à ça :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
<?xml version="1.0" encoding="UTF-8"?>
 
<container xmlns="http://symfony.com/schema/dic/services"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
    xsi:schemaLocation="http://symfony.com/schema/dic/services http://symfony.com/schema/dic/services/services-1.0.xsd">
 
    <services>
        <service id='acv.calculator'
                 class='Arto\AcvBundle\Calculator\Calculator'>
                 <argument type="service" id="doctrine.orm.entity_manager"/>
                 <call method="calculateMaterial">
                      <argument type="service" id="acv.product"></argument>
                 </call>
        </service>
        <service id='acv.product'
                 class='Arto\AcvBundle\Entity\Product'
                 public='false'>
        </service>
    </services>
</container>

J'ai donc écrit dans calculateAll(Product $product) le code suivant :

Code:

1
2
3
 
$calculator = $this->getCalculator();
$tab = $calculator->calculateMaterial($product);

Seulement dans cette méthode je récupère toujours un $product NULL.

A priori, je ne l'ai pas initialisé dans ma balise <argument></argument> dans le services.xml.

Mais je ne vois pas comment je peux passer ma variable $product de ma méthode calculateAll() vers le services.xml dynamiquement.

Si quelqu'un comprend mon souci et veux bien m'aider/me dire ce que je fais mal.

Merci

Antoine

slt,

tu as un getter ?

Code:

$calculator = $this->getCalculator();

Si tu veux récupérer un service il faut faire :

Code:

1
2
 
        $service = $this->get('mon_service');

Code:

1
2
 
        $service = $this->container->get('mon_service')

une config simple :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
<?xml version="1.0" ?>
 
<container xmlns="http://symfony.com/schema/dic/services"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
    xsi:schemaLocation="http://symfony.com/schema/dic/services http://symfony.com/schema/dic/services/services-1.0.xsd">
 
    <parameters>
        <parameter key="mon_service.class">App\FrontendBundle\Service\MonService</parameter>
    </parameters>
 
    <services>
        <service id="mon_service" class="%mon_service.class%">
        </service>
    </services>
 
</container>

Maintenant si dans ta class MonService tu as une methode calculate, dans ton controller tu n'as plus qu'a faire :

Code:

1
2
3
4
 
        $data = array();
        $service = $this->container->get('mon_service');
        $result = $service->calculate($data);

Ensuite on parlera de passer la Request a ton service.

12/06/2014, 20h18
grokh

Bonsoir et merci mais tout ça a priori je l'ai déja,

J'ai en effet fait mon :

Code:

$calculator = $this->getCalculator();

ou getCalculator me donne un accès à mon service.

Je fais bien mon

Code:

$calculator->calculate($product)

et en effet ça fait appel à la bonne méthode de la classe Calculator.

$product est une entité Doctrine, après je pourrais très bien passer un $productId et reconstituer le $product dans mon Calculator.

Mon souci est que je récupère toujours un $product NULL au lieu de récupérer le $product que je passe à ma méthode
12/06/2014, 23h21
goabonga

Poses ton code sur un git ou autre que l'on puisse regarder, par ce que j'ai du mal a te suivre.

Enfait les arguments de ton services que tu configures sont souvent les services que ton service va utiliser.
Un service (de scope normal mais tu verras les scopes plus tard c'est plus avancé) est une seule instance de cette classe la qui est accessible par le container. Il est créé au moment de l'initialisation du container (de mémoire).

Voici a quoi devrait ressembler ton service calculator, de mon point de vue:

la config (déolé j'écrit en yml mais ca reste identique)
Code:

1 2 3 4 5 6 7 parameters: calculator.class : Path\To\Service\CalculatorService services: calculator: class: "%calculator.class%" arguments: [ "@doctrine.orm.entity_manager"]
Ton service:
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 class CalculatorService { private $em; public function __construct(EntityManagerInterface $em) { $this->em = $em; } public function calculate(Product $product) // Initialize array to 11 values set to 0 if ($tab != null) { $m = $tab; } else { $m = array_fill(0, 11, 0); } foreach($product->getAssemblies() as $assembly) { foreach($assembly->getParts() as $part) { /* [...] */ if ($part->getMaterial() != null) { $simulation = $this->em->getRepository('ArtoAcvBundle:SimulationMaterial')->findOneBy(array( 'product' => $product->getId(), 'material' => $part->getMaterial()->getId() )); /* [...] */ return $m; } }
Après je vois pas trop l'interet de deux boucles imbriquées foreach alors que tu peux aller chercher les parts directement avec une requete. En plus le nombre de requete que tu génères est anormal.
Ducoup le plus propre ce serait:
Code:

1 2 3 4 5 6 7 public function calculate($product_id) // Initialize array to 11 values set to 0 // [...] $parts = $this->em->getRepository('ArtoAcvBundle:Part')->getPartByProductJoinSimulation(); l foreach($parts as $part) { /* [...] */
Cette requete $this->em->getRepository('ArtoAcvBundle:Part')->getPartByProductJoinSimulation(); est une requete personnalisée écrite dans ton repository, qui va chercher tous les parts des assembly reliés au produit. Essaye de faire une jointure vers simulation pour pas généré tes requetes au milieu d'une boucle $simulation = $this->em->getRepository('ArtoAcvBundle:SimulationMaterial')->findOneBy(