控制器必须返回响应（给定1）。 500内部服务器错误 - LogicException

时间：2016-04-22 07:14:29

标签： symfony doctrine-orm dql

我开始使用以下控制器代码

<?php

namespace AppBundle\Controller;

use AppBundle\Entity\events;
//use AppBundle\Entity\eventtype;
use AppBundle\Entity\users;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\Form\Extension\Core\Type\SubmitType;
use Symfony\Component\Form\Extension\Core\Type\ChoiceType;

class DefaultController extends Controller {

    /**
     * @Route("/home", name="homepage")
     */
    public function indexAction(Request $request) {
        $events = new events();

        $form = $this->createFormBuilder($events)
                ->add('eT1', ChoiceType::class, array(
                    'choices' => array(
                        'Poker' => 1,
                        'Chess' => 2,
                        'Cricket' => 3,
                        'Marbles' => 4,
                        'Football' => 5,
                    ),
                    'choices_as_values' => true,
                    'expanded' => true,
                    'multiple' => false,
                    'label' => 'Choose After Breakfast Event',
                ))
                ->add('eT2', ChoiceType::class, array(
                    'choices' => array(
                        'Poker' => 1,
                        'Chess' => 2,
                        'Cricket' => 3,
                        'Marbles' => 4,
                        'Football' => 5,
                    ),
                    'choices_as_values' => true,
                    'expanded' => true,
                    'multiple' => false,
                    'label' => 'Choose After Snacks Event',
                ))
                ->add('save', SubmitType::class, array('label' => 'Submit'))
                ->getForm();

        if ($request->isMethod('POST')) {
            $form->submit($request);

            $formData = $form->getData();

            //check to see if the after breakfast event (eT1) is full (ie.has reached the maxlimit)
            //for now the maxLimit is set to 4
            $ET1 = $formData->getET1();

            $repository = $this->getDoctrine()->getRepository('AppBundle:events');
            $query1 = $repository->createQueryBuilder('p')
                    ->select('count(p)')
                    ->where('p.eT1 = :eT1')
                    ->setParameter('eT1', $ET1)
                    ->getQuery();
            $a = $query1->getSingleScalarResult();

            if ($a >= $maxLimit=4) {
                echo 'choose another event, this one is full';
            }
            else{
               return $ET1; 
            }

            //check to see if the after snacks event (eT2) is full (ie.has reached the maxlimit)
            //for now the maxLimit is set to 4
            $ET2 = $formData->getET2();

            $query2 = $repository->createQueryBuilder('p')
                    ->select('count(p)')
                    ->where('p.eT2 = :eT2')
                    ->setParameter('eT2', $ET2)
                    ->getQuery();
            $b = $query2->getSingleScalarResult();

            if ($b >= $maxLimit=4) {
                echo 'choose another event, this one is full';
            }
            else{
               return $ET2; 
            }

            //get the user_id of the logged in user
            $user = $this->container->get('security.context')->getToken()->getUser();
            $events->setUser($user);
            $x = $events->getUser()->getID();

            if ($form->isValid()) {

                //check if the user has already registered or not
                $y = $repository->findOneBy(array('user' => $x));

                // If the user is registering for the first time (execute the insert query)
                if (!$y) {                    
                    $em = $this->getDoctrine()->getManager();
                    $em->persist($events);
                    $em->flush();
                    return $this->redirectToRoute('homepage');
                } 

                //If the user has registered already and want change his registered events (execute the update query)
                else {                    
                    $formData = $form->getData();
                    $ET1 = $formData->getET1();
                    $ET2 = $formData->getET2();
                    $events->setET1($ET1);
                    $events->setET2($ET2);

                    $em = $this->getDoctrine()->getManager();
                    $em->persist($events);
                    $em->flush();
                    return $this->redirectToRoute('homepage');
                }
            }
        }

        return $this->render('default/index.html.twig', array(
                    'form' => $form->createView(),
        ));
    }

}

但后来我修改了代码，为已经注册并且第一次注册的用户显示不同的页面。 以下代码给出错误 - 控制器必须返回响应（给定1）。 500内部服务器错误 - LogicException 我的猜测是错误在于语句return $ET1;和return $ET2; 以下是我找不到答案的问题： 1.为什么错误 - 控制器必须返回响应（给定1）。 500内部服务器错误 - LogicException 对于新用户和已经注册的用户，我必须重复两次相同的代码。有工作吗？ 3.如何从config.yml文件中为不同的事件（例如poker = 12，chess = 4，cricket = 10等）设置不同的$ maxLimit参数值？

这是修改后的控制器代码

<?php

namespace AppBundle\Controller;

use AppBundle\Entity\events;
//use AppBundle\Entity\eventtype;
use AppBundle\Entity\users;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\Form\Extension\Core\Type\SubmitType;
use Symfony\Component\Form\Extension\Core\Type\ChoiceType;

class DefaultController extends Controller {

    /**
     * @Route("/home", name="homepage")
     */
    public function indexAction(Request $request) {
        $events = new events();
        // check if the user is registering for the first time or else
        $user = $this->container->get('security.context')->getToken()->getUser();
        $events->setUser($user);
        $x = $events->getUser()->getID();

        $repository = $this->getDoctrine()->getRepository('AppBundle:events');
        $y = $repository->findOneBy(array('user' => $x));

        // If the user is registering for the first time (execute the insert query)
        if (!$y) {
            echo 'Welcome to the Birthday party';
            echo 'You can choose from the below options:';

            $form = $this->createFormBuilder($events)
                    ->add('eT1', ChoiceType::class, array(
                        'choices' => array(
                            'Poker' => 1,
                            'Chess' => 2,
                            'Cricket' => 3,
                            'Marbles' => 4,
                            'Football' => 5,
                        ),
                        'choices_as_values' => true,
                        'expanded' => true,
                        'multiple' => false,
                        'label' => 'After Breakfast Event',
                    ))
                    ->add('eT2', ChoiceType::class, array(
                        'choices' => array(
                            'Poker' => 1,
                            'Chess' => 2,
                            'Cricket' => 3,
                            'Marbles' => 4,
                            'Football' => 5,
                        ),
                        'choices_as_values' => true,
                        'expanded' => true,
                        'multiple' => false,
                        'label' => 'After Snacks Event',
                    ))
                    ->add('save', SubmitType::class, array('label' => 'Submit'))
                    ->getForm();

            if ($request->isMethod('POST')) {
                $form->submit($request);

                $formData = $form->getData();

                //check to see if the after breakfast event (eT1) is full (ie.has reached the maxlimit)
                $ET1 = $formData->getET1();

                $repository = $this->getDoctrine()->getRepository('AppBundle:events');
                $query1 = $repository->createQueryBuilder('p')
                        ->select('count(p)')
                        ->where('p.eT1 = :eT1')
                        ->setParameter('eT1', $ET1)
                        ->getQuery();
                $a = $query1->getSingleScalarResult();
//            var_dump($a);
//            exit;
                if ($a >= $maxLimit = 4) {
                    echo 'choose another event, this one is full';
                } else {
                    return $ET1;
                }

                //check to see if the after snacks event (eT2) is full (ie.has reached the maxlimit)
                $ET2 = $formData->getET2();

                $query2 = $repository->createQueryBuilder('p')
                        ->select('count(p)')
                        ->where('p.eT2 = :eT2')
                        ->setParameter('eT2', $ET2)
                        ->getQuery();
                $b = $query2->getSingleScalarResult();
//            var_dump($a);
//            exit;
                if ($b >= $maxLimit = 4) {
                    echo 'choose another event, this one is full';
                } else {
                    return $ET2;
                }

                if ($form->isValid()) {

                    $em = $this->getDoctrine()->getManager();
                    // tells Doctrine you want to (eventually) save the Product (no queries yet)
                    $em->persist($events);

                    // actually executes the queries (i.e. the INSERT query)
                    $em->flush();
                    return $this->redirectToRoute('homepage');
                }
            }
        }
        // If the user has registered already and want change his registered events (execute the update query)
        else {
            echo 'Welcome back';
            echo 'You registered for:';
            echo 'After Breakfast event:.$eventName.';
            echo 'After Snacks event:.$eventName.';
            echo 'You can change your selection by choosing again from below options:';

            $form = $this->createFormBuilder($events)
                    ->add('eT1', ChoiceType::class, array(
                        'choices' => array(
                            'Poker' => 1,
                            'Chess' => 2,
                            'Cricket' => 3,
                            'Marbles' => 4,
                            'Football' => 5,
                        ),
                        'choices_as_values' => true,
                        'expanded' => true,
                        'multiple' => false,
                        'label' => 'After Breakfast Event',
                    ))
                    ->add('eT2', ChoiceType::class, array(
                        'choices' => array(
                            'Poker' => 1,
                            'Chess' => 2,
                            'Cricket' => 3,
                            'Marbles' => 4,
                            'Football' => 5,
                        ),
                        'choices_as_values' => true,
                        'expanded' => true,
                        'multiple' => false,
                        'label' => 'After Snacks Event',
                    ))
                    ->add('save', SubmitType::class, array('label' => 'Submit'))
                    ->getForm();

            if ($request->isMethod('POST')) {
                $form->submit($request);

                $formData = $form->getData();

                //check to see if the after Breakfast event (eT1) is full (ie.has reached the maxlimit)
                $ET1 = $formData->getET1();

                $repository = $this->getDoctrine()->getRepository('AppBundle:events');
                $query1 = $repository->createQueryBuilder('p')
                        ->select('count(p)')
                        ->where('p.eT1 = :eT1')
                        ->setParameter('eT1', $ET1)
                        ->getQuery();
                $a = $query1->getSingleScalarResult();
//            var_dump($a);
//            exit;
                if ($a >= $maxLimit = 4) {
                    echo 'choose another event, this one is full';
                } else {
                    return $ET1;
                }

                //check to see if the after snacks event (eT2) is full (ie.has reached the maxlimit)
                $ET2 = $formData->getET2();

                $query2 = $repository->createQueryBuilder('p')
                        ->select('count(p)')
                        ->where('p.eT2 = :eT2')
                        ->setParameter('eT2', $ET2)
                        ->getQuery();
                $b = $query2->getSingleScalarResult();
//            var_dump($a);
//            exit;
                if ($b >= $maxLimit = 4) {
                    echo 'choose another event, this one is full';
                } else {
                    return $ET2;

                    if ($form->isValid()) {

                        $formData = $form->getData();
                        $ET1 = $formData->getET1();
                        $ET2 = $formData->getET2();
                        $events->setET1($ET1);
                        $events->setET2($ET2);
                        $em = $this->getDoctrine()->getManager();

                        $em->persist($events);
                        $em->flush();
                        return $this->redirectToRoute('homepage');
                    }
                }
            }
        }
        return $this->render('default/index.html.twig', array(
                    'form' => $form->createView(),
        ));

    }

}

以下是用户实体

<?php

namespace AppBundle\Entity;

use FOS\UserBundle\Model\User as BaseUser;
use Doctrine\ORM\Mapping as ORM;
use Symfony\Component\Validator\Constraints as Assert;
use Doctrine\Common\Collections\ArrayCollection;

/**
 * users
 *
 * @ORM\Table(name="users")
 * @ORM\Entity(repositoryClass="AppBundle\Repository\usersRepository")
 */
class users extends BaseUser
{
    /**
     * @ORM\Column(name="id", type="integer")
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="AUTO")
     */
    protected $id;

    /**
     * Get id
     * @return integer 
     */
    public function getId()
    {
        return $this->id;
    }

    /**
     * @ORM\OneToOne(targetEntity="events", mappedBy="user")
     */
    protected $event;


//    public function __construct()
//    {
//        parent::__construct(); 
//         }

    }

以下是事件实体

<?php

namespace AppBundle\Entity;

use Doctrine\ORM\Mapping as ORM;
//use Symfony\Component\Validator\Constraints as Assert;

/**
 * events
 *
 * @ORM\Table(name="events")
 * @ORM\Entity(repositoryClass="AppBundle\Repository\eventsRepository")
 */
class events {

    /**
     * @var int
     *
     * @ORM\Column(name="id", type="integer")
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="AUTO")
     */
    protected $id;

    /**
     * @var int
     *
     * @ORM\Column(name="ET1", type="integer")
     */
    protected $eT1;

    /**
     * @var int
     *
     * @ORM\Column(name="ET2", type="integer")
     */
    protected $eT2;

    /**
     * @ORM\OneToOne(targetEntity="users", inversedBy="event")
     * @ORM\JoinColumn(name="user_id", referencedColumnName="id")
     */
        protected $user;

    /**
     * Get id
     *
     * @return integer 
     */
    public function getId() {
        return $this->id;
    }

    /**
     * Set user
     *
     * @param users $user
     * @return events
     */
    public function setUser($user) {
        $this->user = $user;

        return $this;
    }

    /**
     * Get user
     *
     * @return events
     */
    public function getUser() {
        return $this->user;
    }


    /**
     * Set eT1
     *
     * @param integer $eT1
     * @return events
     */
    public function setET1($eT1) {
        $this->eT1 = $eT1;

        return $this;
    }

    /**
     * Get eT1
     *
     * @return integer 
     */
    public function getET1() {
        return $this->eT1;
    }

    /**
     * Set eT2
     *
     * @param integer $eT2
     * @return events
     */
    public function setET2($eT2) {
        $this->eT2 = $eT2;

        return $this;
    }

    /**
     * Get eT2
     *
     * @return integer 
     */
    public function getET2() {
        return $this->eT2;
    }

//    public function __construct()
//    {
//        parent::__construct();
//    }
}

3 个答案:

答案 0 :(得分：1)

$ET2和formData之类的元素没有响应，但是JsonResponse数组，Symfony希望您返回一个响应，因此请确保将其呈现在视图中，或者可能将其返回为数据通过新的<div id="centertbl"> <table id="tblData" class="TSS"> <tr> <td align="center" colspan="4"><b>Table 1</b></td> </tr> </table> <table id="tblData" class="TSS"> <tr> <td align="center" colspan="4"><b>Table 2</b></td> </tr> </table> 对象？

您还可以添加@Template annotation以便能够返回数据。该注释告诉Symfony应该根据返回的数据呈现模板。

答案 1 :(得分：1)

回答你的第一个问题：Controller必须总是返回一个Response类型的对象，如果它不能返回响应对象，那么它必须通过对返回响应对象的控制器执行内部子请求来转发控制。在symfony中，twig模板用于使用

向用户显示输出

return $this->render('navigation/homepage.html.twig', array('et1' => $et1, 'et2' => $et2));

将使homepage.html.twig模板中的两个变量et1和et2可用。

回答第二个问题：通过$this->getUser()获取当前登录用户，这是security.context令牌link

的捷径。

我在这里猜测，你试图检查userId是否与'events'对象的userId匹配，你认为它是一个返回的用户，如果它不是那么它是一个新用户，在那个前提下

$user = $this->getUser()->getUserId();
$event = $this->getDoctrine()->getRepository('AppBundle:events')->findOneBy(array('user' => $this->getUser()->getUserId() ));
if($event instanceof events){ //events refers to 'events' entity
// code to handle returning user
}

回答第三个问题：避免使用全局变量，通过使用config.yml存储maxValue，您可以为整个项目提供maxValue。您可以在代码中更改内容以不使用golbal变量，而只是为了回答问题

将你的全局变量存储在parameters.yml而不是config.yml中只是为了让事情变得可以理解，做这样的事情link 

parameters:
    cricketMaxLimit: 10

使用$cricMaxLimit = $this->container->getParameter('cricketMaxLimit');

检索它

旁注：像if ($a >= $maxLimit = 4)这样的东西甚至不应该出现在这个世界上，做if ($a >= 4)

-dheeraj：）

答案 2 :(得分：0)

您的控制器是不可读的......您可能希望将业务代码放在服务中，将formType放在专用类中，并将查询放入存储库类中。

我认为你不进入if(!$y )条件

放在最后：

return $this->render('default/index.html.twig'));
相关问题
最新问题