0

我的代码有什么问题？我想Moteur根据 url 更新或插入一个对象。
提前谢谢。

/**
* @Route("/moteur/{moteurid}", name="moteur", requirements={"moteurid" = "\d+"}, defaults={"moteurid" = null})
* @Template()
*
* Cette page permet d'enregistrer de nouveaux moteurs (et de les éditer).
*/
public function moteurAction($moteurid)
{
    $args=array();
    $avertissement = null;
    if (!$this->get('security.context')->isGranted('ROLE_ADMIN'))
    {
        $avertissement = "Vous n'avez pas le droit d'accéder à cet espace.";
        return $this->redirect($this->generateUrl('index', array('avertissement' => $avertissement)));
    }

    $args['menu']['admin'] = 'selected';
    $obj = null;
    if ($moteurid == null)
    {
        $obj = new Moteur();
    }
    else
    {
        $obj = $this->getDoctrine()->getRepository('CreasixtineAFBundle:Moteur')->find($moteurid);
    }
    $form = $this->createForm(new FormMoteur(), $obj);
    $args['form'] = $form->createView();

    if ($this->getRequest()->getMethod() == 'POST')
    {
        $form->bindRequest($this->getRequest());

        if ($form->isValid())
        {
            $obj = $form->getData(); // Type Moteur()
            $pn = $obj->getPnid();

            $em = $this->getDoctrine()->getEntityManager();
            if ($moteurid == null)
            {
                $em->persist($obj);
                $avertissement = "Moteur créé !";
            }
            else 
            {
                // Rien, le moteur sera mis à jour avec flush()
                $avertissement = "Moteur mis à jour !";
            }
            foreach ($pn as $my_pn){$em->persist($my_pn);}
            $em->flush();

            return $this->redirect($this->generateUrl('admin', array('avertissement' => $avertissement)));
        }
        else
        {
            throw new Exception("Le formulaire n'est pas valide.");
        }
    }

    $contenu = $this->rendu($args, "formulaire_moteur.html.twig");
    return $contenu;
}
4

1 回答 1

1

首先，您不需要这一行，因为 PHP5 本身通过引用传递对象：

$obj = $form->getData(); // Type Moteur()

然后，您在 Moteur 和 Pn 之间的关系有点混乱。你用 getPnid() 得到一个 Pn 但你得到一个你想要坚持的对象？

无论如何，这些 Pn 对象应该在 Moteur 之前持久化，所以这是我要写的：

if ($form->isValid())
{
    $em = $this->getDoctrine()->getEntityManager();

    $pn = $obj->getPnid();

    //Persist these related objects BEFORE Moteur
    foreach ($pn as $my_pn)
    {
        $em->persist($my_pn);
    }

    if ($moteurid == null)
    {
        $em->persist($obj);
        $avertissement = "Moteur créé !";
    }
    else 
    {
        // Rien, le moteur sera mis à jour avec flush()
        $avertissement = "Moteur mis à jour !";
    }
    $em->flush();

    return $this->redirect($this->generateUrl('admin', array('avertissement' => $avertissement)));
}
else
{
    throw new Exception("Le formulaire n'est pas valide.");
}
于 2012-04-18T09:22:13.950 回答