Discussion :

[MehrezLabidi]

Bonjour,
Ce que j'ai dans mon controlleur:

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
 
    public function uploadfileAction() {
        $validerfichier = $this->get('Fichier_Valid');
        $validerfichier->uploadfile();
        return $this->redirect($this->generateUrl('envoifichier'));
    }
 
    public function downloadAction($downloadfile) {
        $validerfichier = $this->get('Fichier_Valid');  //la classe Fichier dans FichierBundle
        $validerfichier->download($downloadfile);  // download manuel.pdf :)
    }
 
    public function lirefichierAction() {
 
        $validerfichier = $this->get('Fichier_Valid');  //la classe Fichier dans FichierBundle
        $fichiers_arrays = $validerfichier->toutlesfichiers(); // recuperer array des fichiers dans uploads :)  
        return $this->render('EspritAppGestionBundle:Default:lirefichier.html.twig', array(
                    'fichiers' => $fichiers_arrays
        ));
    }

je veux que $validerfichier sois affectée au début (en haut du controlleur seulement ) et bien sur utilisé partout .

merci, LM

[lilalex]

Salut, ce n'est pas une variable globale que tu as besoin ici, mais d'un service.

Crée un service qui va garder la variable et si possible gérer tous les traitements qui lui sont associés.
Un service pouvant être appelé depuis un controller mais aussi depuis un autre service, même depuis ton template twig si tu fais ce qu'il faut pour, ça remplit pleinement ton besoin actuel. De manière générale, quand tu te rend compte que ton controller fait autre chose que de l'aiguillage c'est qu'il te manque des services pour faire office de boîtes noires.

[Tsilefy]

Qu'est-ce qui t'empêche de créer une propriété $validerfichier et de l'initialiser dans le constructeur??

[MehrezLabidi]

@ Tsilefy j'ai rajouté:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
//
use EspritApp\FichierBundle\Services\Fichier;
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
 
class DefaultController extends Controller {
 
    private $validerfichier;
    public function __construct(Fichier $validerfichier) {
        $this->validerfichier = $validerfichier;
    }
//

mais ça tourne plus

[Tsilefy]

Il faut transformer ton contrôleur en service (le principal effet est qu'il n'hérite plus de Controller), et tu injectes ensuite dans le service les paramètres que tu veux depuis ton conteneur à injection de dépendance (tu peux aussi injecter tout le conteneur, mais ce n'est pas une bonne pratique).

Mais regarde également ce que lilalex a conseillé :-)

[lilalex]

Tu peux aussi injecter tout le conteneur, mais ce n'est pas une bonne pratique

Alors moi aussi j'avais lu ça et j'y ait cru dur comme fer ... mais ça c'était avant ^^.
Les mecs qui ont conseillé ça n'ont jamais développé de commandes Symfony de un, de deux tu passes ta vie à modifier ton fichiers services.yml si tu fais passer les services un à un.
A moins de vouloir distribuer ton bundle, il n'y a aucun intérêt à se refuser le "service container" dans les services que l'on crée.