Symfony2.8错误在不使用doctrine的对象上下文中时使用$ this

时间:2017-06-07 20:18:00

标签: php doctrine symfony-2.8

我正在尝试从我的控制器调用一个函数并将$ todolist返回给我的控制器。但是我得到这个错误,当我把这个功能留在控制器里面时工作正常,但我不希望它在控制器中。

这是我的控制器

namespace AppBundle\Controller;

use Sensio\Bundle\FrameworkExtraBundle\Configuration\route;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;

use AppBundle\Entity\Todo;
use AppBundle\Model\TodoModel;

class TodoController extends Controller
{
    /**
    * @Route("/", name="homepage")
    */
    public function ShowList()
    {
        $todolist = TodoModel::getTodolist();

        $html = $this->container->get('templating')->render(
            'todolist/todolist.html.twig',
            array('todolist' => $todolist)
        );

        return new response($html);
    }
}

这是我的模特

namespace AppBundle\Model;

use AppBundle\Entity\Todo;

class TodoModel 
{

     public function getTodolist()
     {
       $repository = $this->getDoctrine()
           ->getRepository('AppBundle:Todo');
       $todolist = $repository->findAll();
       return $todolist;
    }

}

2 个答案:

答案 0 :(得分:2)

正确的方法是:

public function ShowList()
{
    $entityManager = $this->get('doctrine.orm.entity_manager');
    $repository = $entityManager->getRepository(Todo::class);
    $todolist = $repository->findAll();

    $html = $this->container->get('templating')->render(
        'todolist/todolist.html.twig',
        array('todolist' => $todolist)
    );

    return new Response($html);
}

如果您想使用" TODO提供商"进行某种抽象,那么更好的命名方式是TodoProvider而不是TodoModel

看起来应该是这样的:

namespace AppBundle\Todo;

use Doctrine\ORM\EntityManager;
use AppBundle\Entity\Todo;

class TodoProvider
{
    /**
     * @var EntityManager
     */
    private $entityManager;

    /**
     * TodoProvider constructor.
     * @param EntityManager $entityManager
     */
    public function __construct(EntityManager $entityManager)
    {
        $this->entityManager = $entityManager;
    }

    /**
     * @return Todo[]
     */
    public function getAllTodos()
    {
        return $this->entityManager->getRepository(Todo::class)->findAll();
    }
}

然后将TodoProvider注册为服务(http://symfony.com/doc/current/service_container.html)并使用$this->get('my_todo_provider')

进入控制器

答案 1 :(得分:0)

您需要将学说注入模型,并将模型定义为服务。最好使用存储库。

Using repositories