在Laravel中使用GOTO比使用GOTO更好

Question

现在我不得不在Laravel 5.6v中使用GOTO来实现一些逻辑。

如何才能使代码的功能与GOTO相同，并且代码更美观？:)

public function job_finished_search(Request $request)
{
    $rFrom = $request->input('datepicker_from');
    $rTo = $request->input('datepicker_to');
    $rName =$request->customer_name;

    if($rFrom == null){
        if($rTo == null){
            if($rName == null){
            Flash::error('Search result not found');
            return redirect(route('pdf.index'));
            }
            $query = Job::onlyTrashed()
                        ->where('is_trash', 0)
                        ->where('customer_name', $rName)
                        ->orderBy('created_at', 'desc');
                        goto A;
        }

        elseif($rTo != null){
            if($rName == null){
                $query = Job::onlyTrashed()
                ->where('is_trash', 0)
                ->where('created_at', '<=', Carbon::parse($rTo)->endOfDay())
                ->orderBy('created_at', 'desc');
                goto A;
            }
        ...
        ...
}

A: $job_ids = $query->pluck('id')->all();  

Answer 1

更干净的方式，

$buildquery=Job::onlyTrashed()->where('is_trash', 0);

if($request->filled('customer_name')){
    $buildquery->where('customer_name', $request->customer_name);
}

if($request->filled('datepicker_to')){
    $buildquery->where('created_at', '<=', Carbon::parse($request->datepicker_to)->endOfDay());
}

$result = $buildquery->orderBy('created_at', 'desc')->get();

$job_ids = $result->pluck('id');

您没有在任何地方使用$rFrom，所以我省略了它。