Какой способ лучше использовать вместо GOTO в Laravel? - PullRequest
0 голосов
/ 18 мая 2018

Теперь я должен использовать GOTO для некоторой логики в Laravel 5.6v.

Как я могу сделать код, который работает так же, как 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();  

1 Ответ

0 голосов
/ 18 мая 2018

Намного более чистый способ,

$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, где бы то ни было, поэтому я его опустил.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...