Question

我想通过消除太多的if语句来创建更具可读性的代码，但仍然可以完成工作。我尝试创建一个私有方法并提取日期范围查询并返回构建器实例，但是每当执行该操作时，它都不会返回正确的构建器查询结果，因此我最终粉碎了此方法的所有内容。

其他参数将很快添加，因此if语句将很快消失。：（

任何有关如何改进的提示将不胜感激。谢谢！

/**
 * @param array $params
 *
 * @param $orderBy
 * @param $sortBy
 *
 * @return Collection
 *
 * Sample:
 * `/orders?release_date_start=2018-01-01&release_date_end=2018-02-20&firm_id=3` OR
 * `/orders?claimId=3&status=completed`
 *
 * Problem: Too many if statements
 *
 */
public function findOrdersBy(array $params, $orderBy = 'id', $sortBy = 'asc'): Collection
{
    $release_date_start = array_get($params, 'release_date_start');
    $release_date_end = array_get($params, 'release_date_end');
    $claimId = array_get($params, 'claimId');
    $firm_id = array_get($params, 'firm_id');
    $status = array_get($params, 'status');

    $orders = $this->model->newQuery();

    if (!is_null($release_date_start) && !is_null($release_date_end)) {
        $orders->whereBetween('releaseDate', [$release_date_start, $release_date_end]);
    } else {
        if (!is_null($release_date_start)) {
            $orders->where('releaseDate', '>=', $release_date_start);
        } else {
            if (!is_null($release_date_end)) {
                $orders->where('releaseDate', '<=', $release_date_end);
            }
        }
    }

    if (!is_null($claimId)) {
        $orders->where(compact('claimId'));
    }

    if (!is_null($firm_id)) {
        $orders->orWhere(compact('firm_id'));
    }

    if (!is_null($status)) {
        $orders->where(compact('status'));
    }

    return $orders->orderBy($orderBy, $sortBy)->get();
}

Answer 1

如果您对使用收集方法感兴趣，则可以使用when()收集方法来省略您的if-else语句。因此，根据您的陈述，它将类似于：

$orders->when(!is_null($release_date_start) && !is_null($release_date_end), function($q) {
    $q->whereBetween('releaseDate', [$release_date_start, $release_date_end]);
}, function($q) {
    $q->when(!is_null($release_date_start), function($q) {
        $q->where('releaseDate', '>=', $release_date_start);
    }, function($q) {
        $q->when(!is_null($release_date_end), function($q) {
            $q->where('releaseDate', '<=', $release_date_end);
        })
    })
})
->when(!is_null($claimId), function($q) {
    $q->where(compact('claimId'));
})
->when(!is_null($firm_id), function($q) {
    $q->orWhere(compact('firm_id'));
})
->when(!is_null($status), function($q) {
    $q->where(compact('status'));
})

有关更多信息，请参见文档中的conditional-clauses。希望这会有所帮助。

Answer 2

一个可用的选项是这样的php中的三元操作：

$claimId ? $orders->where(compact('claimId')) : ;
$firm_id ? $orders->orWhere(compact('firm_id')) : ;
$status ?  $orders->where(compact('status')) : ;

这将比语句代码更干净。

您可以在laravel中使用的

另一个选项是Conditional Clauses

Answer 3

感谢您的建议，但我想出了另一种解决方案：

/**
 * @param array $params
 *
 * @param $orderBy
 * @param $sortBy
 *
 * @return Collection
 */
public function findOrdersBy(array $params, $orderBy = 'id', $sortBy = 'asc'): Collection
{
    $release_date_start = array_get($params, 'release_date_start');
    $release_date_end = array_get($params, 'release_date_end');

    $orders = $this->model->newQuery();

    if (!is_null($release_date_start) && !is_null($release_date_end)) {
        $orders->whereBetween('releaseDate', [$release_date_start, $release_date_end]);
    } else {
        if (!is_null($release_date_start)) {
            $orders->where('releaseDate', '>=', $release_date_start);
        } else {
            if (!is_null($release_date_end)) {
                $orders->where('releaseDate', '<=', $release_date_end);
            }
        }
    }

    $fields = collect($params)->except($this->filtersArray())->all();
    $orders = $this->includeQuery($orders, $fields);

    return $orders->orderBy($orderBy, $sortBy)->get();
}

/**
 * @param Builder $orderBuilder
 * @param array $params
 *
 * @return Builder
 */
private function includeQuery(Builder $orderBuilder, ... $params) : Builder
{
    $orders = [];
    foreach ($params as $param) {
        $orders = $orderBuilder->where($param);
    }

    return $orders;
}

/**
 * @return array
 */
private function filtersArray() : array
{
    return [
        'release_date_start',
        'release_date_end',
        'order_by',
        'sort_by',
        'includes'
    ];
}

private method includeQuery(Builder $orderBuilder, ... $params)上的主要因素，它以$params作为variable length argument。我们只是迭代要作为查询参数/orders?code=123&something=test传递的变量，并在查询构建器中将它们作为where()子句传递。

某些参数可能不是对象的属性，因此我们仅需过滤与对象属性匹配的查询参数。因此，我创建了一个filtersArray()，它将返回要排除的参数并防止错误。

嗯，实际上，在撰写本文时，我应该有一个相反的词only()，否则它将有无限的东西要排除。 :)我猜那将是另一个重构。：P

如何摆脱Laravel中过多的if语句？

3 个答案: