-1

在我的控制器的操作中，我经常通过其 id 从数据库中的表中选择一行，因此我将其分离为一个函数。但是每次我这样做时，我都会检查查询是否返回了某些内容，因此我也想将其与函数分开。这是我的代码：

此功能位于控制器中，由我的控制器扩展，其中包含以下操作：

protected function findById($id, $class)
{
    $result = $this->getEM()->getRepository('EMMyFriendsBundle:'.$class)->find($id);   
    if($result == null) {
        return false;
    } else {
        return $result;
    }
}

这是我的控制器：

class FriendController extends Controller
{
    private $em;
    private $friend;

    private function init($id)
    {
        $this->em = $this->getEM();
        $this->friend = $this->findById($id, 'Friend');

        if(!$this->friend) {
            return $this->render('EMMyFriendsBundle:Friend:error.html.twig', array(
               'item' => 'friend'));
        }   
    }

    /*
     * Displays all the information about a concrete friend
     */
    public function displayAction($id)
    {
        $this->init($id); 

        if($this->friend->getCategory() != null) {
            $category = $this->friend->getCategory()->getName();
        } else {
            $category = null;
        }

        return $this->render('EMMyFriendsBundle:Friend:friend.html.twig', array(
               'friend' => $this->friend, 'category' => $category));
    }

    // ...
}

当我选择一个不存在的 ID（如 38743874）时，它会转到 init 函数中的 if 部分，但不会呈现模板 error.html.twig :( 但如果我选择这部分

if(!$this->friend) {
                return $this->render('EMMyFriendsBundle:Friend:error.html.twig', array(
                   'item' => 'friend'));
            }

退出init()函数并displayAction($id)在调用后将其放入$this->init($id)，它可以工作。但我不想在该控制器的每一个动作中都写这个。任何想法如何将其分开以避免代码重复？

4

1 回答 1

2

您不会返回 init 函数的响应。这就是为什么它会向您显示错误消息。您必须在以下位置执行类似的操作displayAction()：

$response = $this->init();
if ($response) {
    return $response;
}
于 2012-09-14T07:38:52.560 回答