7

从 MVC 模式和 Symfony2 的角度来看，我可以稍微精简一下我的控制器代码，去掉一些持久化逻辑吗？例如，给定一个像这样的标准新动作：

public function newAction(\Symfony\Component\HttpFoundation\Request $request)
{

    // Create a new entity and a new form type for managing
    $entity = $this->createNewEntity();
    $form   = $this->createForm($this->createNewFormType(), $entity);

    // If it's GET just return the view
    if('GET' == $request->getMethod())
        return array('form' => $form->createView());

    // It's POST request so bind the form
    $form->bindRequest($request);

    // ... and if it's valid just persist the entity
    if($form->isValid()) :

        $em = $this->getEntityManager(); // Should be carried by controller?
        $em->persist($entity);           // Should be carried by controller?
        $em->flush();                    // Should be carried by controller?

        // Redirect the user to the default page
        return $this->redirect($this->getOnNewSuccessRedirectUrl($entity));

    endif;

    // Return the view plus errors
    return array(
        'errors' => $this->getValidator()->validate($entity),
        'form'   => $form->createView()
    );

}

将该逻辑移至存储库是否正确？一个例子（警告：可能不起作用）：

class BaseRepository extends \Doctrine\ORM\EntityRepository
{

    /**
     * Persist the entity (either new or existent)
     *
     * @param object $entity
     */
    public function save($entity)
    {
        $em = $this->_em;
        $em->persist($entity);
        $em->flush();
    }

    /**
     * Delete the entity.
     *
     * @param object $entity
     */
    public function delete($entity)
    {
        $em = $this->_em;
        $em->remove($entity);
        $em->flush();
    }

}

控制器代码将是：

if($form->isValid()) :

    $this->getRepository()->save($entity);

    // Redirect the user to the default page
    return $this->redirect($this->getOnNewSuccessRedirectUrl($entity));

endif;
4

2 回答 2

11

我认为将持久化并从控制器中删除逻辑是一个好主意，但存储库不是它的正确位置。

来自 Symfony2 文档（http://symfony.com/doc/current/book/doctrine.html#fetching-objects-from-the-database）：

您可以将存储库视为一个 PHP 类，它的唯一工作是帮助您获取某个类的实体。

您应该只使用您的存储库类从数据库中获取数据。

我会将您的持久/删除逻辑移动到服务中（请参阅http://symfony.com/doc/current/book/service_container.html），因此您的控制器代码将类似于

if($form->isValid()) :

    $this->get('service.entity')->save($entity);

    // Redirect the user to the default page
    return $this->redirect($this->getOnNewSuccessRedirectUrl($entity));

endif;
于 2012-05-11T00:08:56.777 回答
4

克里斯是绝对正确的。

但顺便说一句：

为什么要以这种方式再次验证您的实体：

'错误' => $this->getValidator()->validate($entity)?

您将请求绑定到表单，因此错误将在 viewhelpers 'form_errors(...)' 中创建的视图中处理。

于 2012-05-11T01:06:40.197 回答