Нужно ли разделять локальный scope на два в зависимости от количества значений в аргументе?

Question

[Андрей Скрипник]

Хотел бы узнать мнения, как корректнее/правильнее писать:

public function scopeByPartnerId($query, $partnerId)
{
    if (is_countable($partnerId)) {
        $query->whereIn('partner_id', $partnerId);
    } else {
        $query->where('partner_id', $partnerId);
    }
}

или

public function scopeByPartnerId($query, $partnerId)
{
    $query->where('partner_id', $partnerId);
}

public function scopeByPartnerIds(Builder $query, array $partnerIds)
{
    $query->whereIn('partner_id', $partnerIds);
}

Answer 1

[Алексей Уколов]

Никакого "правильнее" в данном вопросе нет и быть не может. Может быть "понятнее для других" и "поддерживеемее". По второму разницы нет, по первому я предпочитаю такой вариант:

public function scopeByPartnerId($query, int|array $partnerId)
{
    if (is_countable($partnerId)) {
        $query->whereIn('partner_id', $partnerId);
    } else {
        $query->where('partner_id', $partnerId);
    }
}

И, разумеется, в самом классе описан doc-block для этого метода, в котором тоже корректные типы аргументов указаны.

Answer 2

[pLavrenov]

Лично мое мнение - не нужно разделять. Это же удобно.

Answer 3

[Сергей delphinpro]

Не надо ничего разделять. Нужно оставить один метод множественного выбора.

Answer 4

[jazzus]

Partner отношение? Тогда правильнее использовать методы отношений
$partner->relations
whereHas, with итд

Comments

[Андрей Скрипник]

нет это обычное ограничение выборки по колонке partner_id

[jazzus]

Андрей Скрипник, в отношениях тоже обычное ограничение выборки по колонке partner_id. Если нет модели Partner и belongsTo отношения тогда надо создать