4

在 Laravel 控制器中，我有几个方法都从获取数据库记录开始，然后在检查是否找到数据后继续渲染视图，或者在没有数据的情况下转到 404 页面。

这是一个例子：

<?php

function get_show_user($id)
{
    $user = static::get_user($user_id);
    if (!$user) {
        return Response::error('404', static::$some_common_error404_message);
    }
    return View::make('users.show_readonly_user_data')->with('user', $user);
}

function get_edit_user($id)
{
    $user = static::get_user($user_id);
    if (!$user) {
        return Response::error('404', static::$some_common_error404_message);
    }

    return View::make('users.display_edit_user_form')->with('user', $user);
}

我if (!$user)在这些方法中重复整个声明，即使它们都做同样的事情。

我宁愿做这样的事情：

function get_show_user($id)
{
    $user = Users::find($id);
    static::go404_if_null($user);
    return View::make('users.show_readonly_user_data')->with('user', $user);
}

function get_edit_user($id)
{
    $user = Users::find($id);
    static::go404_if_null($user);
    return View::make('users.display_edit_user_form')->with('user', $user);
}

实现这种 DRY 功能的最佳方式是什么？

显然，一个简单return Response::error('404')的方法在常见的存在检查器方法中不起作用，因为它只会从该方法返回。

似乎 anEvent::fire('404')也不理想，因为它不会终止触发它的方法。

也许这里需要使用异常，但我不确定这一点，或者应该如何在 Laravel 中完成。我应该在哪里捕获控制器的 404 异常？

4

1 回答 1

1

我认为解决此问题的最佳方法是控制器上的前置过滤器。

public static $require_user = array(
    'edit_user',
    'show_user',
);

public function before()
{
    $route = Request::route();
    if ( in_array( $route->controller_action, static::$require_user ) )
    {
        $this->user = User::find( $route->parameters[0] );
        if ( is_null($this->user) ) return Response::error('404');
    }
}

在构造控制器之后但在调用方法之前调用过滤器之前。如果 before 过滤器返回 null 以外的任何内容，则不会调用该方法并因此停止执行。

在这里，我们获取当前正在执行的路由，因此我们可以检查将针对我们的数组 $require_user 调用哪个方法。这让我们可以使用不需要用户 ID 的方法，例如登录。

然后我们检索用户实例，从传递给方法的内容中获取 id。您可能应该在此处添加更多错误处理。

最后，我们检查返回的用户是否为空，即未找到。如果是这种情况，我们会返回 404 响应，停止执行该方法。

我希望这有帮助！

于 2012-10-22T19:05:27.270 回答