Как выбрать более правильное решение по коду?

Question

[Сергей Ерин]

Доброго времени суток! В модели поиска Search.php есть метод для получения массива с именами пользователей для дальнейшего вывода их списком dropdown в шаблоне

public function getDropDownData(): array
   {
        $data = [];

        $query = self::find()->select('id')->distinct()->all();

        foreach ($query as $user) {
            $data[$user->id] = $user->getName();
        }

        return $data;
    }

Нужно помимо имени пользователя вывести название компании рядом в скобках из связанной таблицы. Я доработал метод и сделал так

foreach ($query as $user) {
            $data[$user->id] = $user->getName() . ' (' . $user->company->getName() . ')';
        }

Но мне кажется такой способ не очень красив, так как вдруг понадобится этот список в другом месте, но без имени компании, да и принцип открытости-закрытости нарушается как мне кажется. Подскажите как можно решить эту проблему?

Answer 1

[Sergey]

Сделайте два метода.
1. Имена пользователей
2. Имена пользователей + компании (sql запрос: join с таблицей company)
В цикле только формирование значения из полученных данных

Comments

[Vitsliputsli]

Сергей Ерин, а также, можно поразмыслить, почему метод в модели называется getDropDownData (уже какая-то смесь model и view, а вы ее еще хотели усугубить). Почему модель называется Search, если мы в ней ничего не ищем, а берем все скопом, и вообще она занимается пользователями. Что за подозрительный distinct в запросе, неужели id не уникальны? Почему массив с пользователями называется query? Зачем вообще этот foreach, вроде в yii был специальный метод для этих целей.

Answer 2

[Vitaliy K]

Имхо, сделать две модели: Companies.php и Users.php.

$search = Users::find()->with('Companies')->where([условие])->all();

Дальше разбирать полученный массив.
Рекомендую использовать ArrayHelper