Discussion :

[Doubrovski]

Bonjour à tous,

J'ai besoin de conseils sur des questions de méthodologie concernant les formulaires et relations entre entités. Je débute sur symfony2 évidemment

J'ai un formulaire (QCM) avec un Type et un Handler (qui sont des services). Le formulaire est construit dans le Type, puis manipulé par le Handler. Pour l'instant je récupère, valid, persist et flush dans le handler via une méthode process(). Voici le controller pour comprendre un peu mieux :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
public function startAction(Request $request)
{
    $formHandler = $this->get('test_handler');
 
    if ($formHandler->process()) {
        return $this->redirect($this->generateUrl('architest_front_test_result'));
    }
 
    return $this->render('ArchitestFrontBundle:Test:start.html.twig', array('form' => $formHandler->getForm()->createView()));
}

Je voudrais qu'à la récupération des réponses du form soit instancié un objet. Les valeurs des attributs de cet objet seraient calculées en fonction des réponses du formulaire. (Le but est que les réponses du QCM attribuent des points à chacun des attributs de cet objet).

L'entité du formulaire est Test, et je veux la lier à l'entité Profil qui contiendrait les attributs.

J'ai donc établi une relation OneToOne. Test est l'entité propriétaire, elle contient donc maintenant un attribut $profil.


Très bien... Mais l'objet Profil n'est pas instancié.

Je veux donc qu'une instance de Profil soit créée et alimentée dès que le formulaire est renvoyé par l'utilisateur.

La solution que j'ai testé est de faire une méthode __construct() dans l'entité Test qui instancie l'objet Profil. C'est peut-être une hérésie.

Code :

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

1
2
3
4
5
6
7
 
//Entité Test sur laquelle est construit le formulaire
public function __construct()
{
    //Est-ce une hérésie ?
    $this->profil = new Profil();
}

Le bouquin et les tutos que je consulte donnent des exemples ou les deux objets sont instanciés dans le controller.
$test = new Test();
$profil = new Profil();
Puis $test->setProfil($profil) pour lier les objets.
Puis pour accéder aux méthodes de Profil : $test->getProfil()->setAttr1(3);

C'est pas mal, même si moi j'ai déplacé la logique dans le Handler, je pourrais instancier Profil à cet endroit, mais pour éviter de vérifier toutes les réponses du form, j'aimerais que l'instanciation et l'alimentation de l'objet Profil soit automatique en fonction des réponses.
Si j'instancie Profil dans la méthode __construct() de Test, je peux ensuite appeler les méthodes de Profil dans les setters de Test pour modifier les attributs de Profil ? (à moins que ce soit encore une hérésie

Le but est d'arriver à ça :
Question 3 du formulaire ---> Si réponse 'b' --> attribut2++ et attribut3-- de l'objet Profil.


Merci, n'importe quel conseil m'aidera à mieux structurer mon code.

[dukoid]

HUGH ! Que la paix soit avec toi !


pour faire ça je schématise grossièrement :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
1:  $form->isValid() ?
pour déterminer si ton formulaire est valide ou pas
 
2: si valid,   $data = $form->getData();
tu récupères les données reçus du formulaire
 
3: tu analyses les données ($data) et implémente ta logique :
Si réponse 'b' --> attribut2++ et attribut3-- de l'objet Profil.
instanciation, persist .....

est ce que cela réponds à ta question?

[Doubrovski]

Merci !
Dans mon cas c'est le Handler du formulaire qui s'occupe de $this->form->isValid();
Je peux effectivement ensuite faire $this->form->getData();

Donc tu me conseilles d'instancier l'objet Profil à cet endroit ? Puis de faire ma logique.
Oui, c'est logique ensuite je persist et je flush l'ensemble.
Je compte à terme séparer le travail du formulaire (testHandler.php), et le travail des entités (entityManager.php) donc je déplacerai tout ça dans entityManager.

Alors c'est pas possible d'instancier un objet dans le construct de l'entité propriétaire ?
De toute façon en y réfléchissant ça ne m'apporterait pas vraiment d'avantage.

[dukoid]

la bonne pratique est de mettre ton traitement dans un service qui lui va faire le traitement et les divers appels aux entitymanager ou repository.
car l'action d'un contrôleur doit contenir un minimum de logique.(voir aucune)

quant à ta dernière question , c'est de l'objet c'est possible. après à savoir si c'est mieux ou pas, je n'ai pas trop compris ton sujet pour te répondre.

[Doubrovski]

Ok, merci !
Je pense avoir respecté cette logique pour l'instant vu que mon controller est quasi vide (cf code posté).
Par contre mon service form_handler (dans lequel j'ai injecté le form, la request et doctrine.orm.entity_manager) s'occupe à la fois de valider le formulaire et de manipuler les entités liées au formulaire. Selon mon tuto de video2brain, c'est un truc à ne pas faire. Un formHandler devrait uniquement s'occuper du formulaire (pas d'injection de doctrine.orm.entity_manager dans ce service). Il conseille de créer un entityManager pour gérer les manipulations d'entités ou les appels au repository.
Bon après ça me paraît un peu complexe comme méthode, surtout vu ce que je fais !

Merci pour ton aide !

ps: C'est bon ça marche, reste à récupérer les données sur ma page d’arrivée