过滤数据时使用雄辩而没有冗余

Question

我正在尝试通过Eloquent过滤数据，该方法只能回答AJAX请求，然后返回视图以及过滤后的数据。 但是我有一个问题：

首先，让我们看一下我的代码：

public function AJAX_get_reports(Request $request)
{
    if($request->ajax())
    {
        $message = $request->get('message');
        $severity = $request->get('severity');

        if(strlen($message) > 0 && strlen($severity) > 0 )
        {
            $data = ServerReport::where('severity', '=', $severity)->where('message', '=', $message)->get();
        } elseif (strlen($message) > 0) {
            $data = ServerReport::where('message', '=', $message)->get();
        } elseif (strlen($severity) > 0) {
            $data = ServerReport::where('severity', '=', $severity)->get();
        } else {
            $data = ServerReport::all();
        }
        return Response::json(View::make('dashboard.reports.index', compact('data'))->render());

    }
}

这是我唯一能够做到的方法，它可以工作，但是我觉得这不是最好的方法，尤其是如果您有更多要过滤的字段，那么代码将在验证中占很大比例，这是最好的做到这一点的方法？

例如，在进行验证时构建查询，然后在最后运行它？

if(strlen($message) > 0)
{
    // add WHERE to query
}
if(strlen($severity) > 0)
{
    // add WHERE to query
}

// Execute query and get the results

Answer 1

我认为，您的代码可以像这样简化：

public function AJAX_get_reports(Request $request) {
    if($request->ajax()) {
        $message = $request->get('message');
        $severity = $request->get('severity');
        $report = ServerReport::select('*'); //Initiate the variable for query container, change * to the columns that needs to be returned or just leave it

        if (strlen($message) > 0) {
            $report->where('message', '=', $message); //Add message filter
        }

        if (strlen($severity) > 0) {
            $report->where('severity', '=', $severity); //Add severity filter
        }

        //Add another filters if needed

        $data = $report->get(); //Get the data from either the filtered data or every single data without any filters

        return Response::json(View::make('dashboard.reports.index', compact('data'))->render());
    }
}

我在公司的某些项目中使用了这种方法，并且一切正常。 试一试。

Answer 2

考虑到您在原始代码中生成的内容对我来说似乎是完全有效的，因此我不确定是否有最佳方法来做到这一点。 例如，如果您有定期使用的WHERE子句，则可以使用作用域来集中旨在达到特定结果的构建条件，并在必要时进行堆叠。 这样，您将来可以灵活地在代码中更改一个位置，并在必须修改条件的情况下影响使用更新范围的所有其他位置。 但这可能与更大的代码可维护性问题有关，比我认为您可能在此特定实例中要寻找的更多。

就个人而言，我已经大量使用了您的假设方法。 当我具有可过滤资源的API端点并且过滤器可以是可变的时，它对我来说非常有用。 例如，通过名字，姓氏和/或电子邮件的任意组合来搜索用户。

这就是我的样子：

public function getIndex(Request $request, Response $response)
{
    //Start with a copy of the model
    //I prefer to instantiate the class from the container,
    //instead of having it auto-injected using the method signature
    //just personal preference
    $users = app('App\User');

    //Filter by first name
    if ($request->has('first_name')) {
        $users->where('first_name', $request->input('first_name'));
    }

    //Filter by last name
    if ($request->has('last_name')) {
        $users->where('last_name', $request->input('last_name'));
    }

    //Filter by email
    if ($request->has('email')) {
        $users->where('email', $request->input('email'));
    }

    //If no conditions were set, get() *should* function like all()
    $data = $users->get();

    //Do what you need with the data from here on out
    //Typically I paginate the results for something like this
}

我出于某些原因而认为这是一个很好的约定。 首先，我发现代码具有很高的可读性，因为每个块都倾向于很好地自我记录（ 如果 请求中 包含 电子邮件 ...）。 其次，我可以轻松，轻松地删除将来需要删除的任何块，而无需取消任何事务。

此方法的一个重要警告是，每个过滤器的行为均独立于其他过滤器-任意选择0个或多个过滤器。 如果您发现自己处于可以按组组合进行过滤的情况，这将变得有些棘手。 即使在那儿，您通常也可以将组合分成自己的块，并大致达到相同的目的。

再次重申，我认为问题第一部分的内容完全正确，但建议问题第二部分的代码是更干净，更可维护的选择。