Как сделать сортировку связанных таблиц в Yii2?

Question

[Сергей Свиридонов]

Доброго времени суток!

Ребят, помогите, сбит с толку.

в общем есть 2 таблицы:
Товары (название, фото и т.д)
Остатки (цены, в каких магазинах есть и т.д)

Эти таблицы связаны, т.е на сайте вывожу продукт и его остатки.
А еще есть возможность выбрать магазин (и продукция на сайте выводится из этого магазина) храню id магазина в сессиях и при наличии сессии добавляю условие where().
Вывод такой:

Product::find()->joinWith('ostatki')->where(['ostatki.id' => $id])->orderBy(['ostatki.price' => SORT_ASC])->all();

Но почему то цены не верно сортируются (они такого вида: 1523.52, 52.5, 52.05 и т.д). И пагинация на сайте не правильно работает. PageSize указал 15, но выводит рандомно 5-6-1-2 и т.д.

Код:

$view = Yii::$app->request->get('view');
		$sort_value = (int) Yii::$app->request->get('sort_value');
		$pharm = Yii::$app->session;
		$pharm->open();
		if (isset($pharm['pid'])) {
			$pharm_id = $pharm['pid'];
			$pharms = Pharmacy::find()
			                  ->where(['pharm_id' => $pharm_id])
			                  ->limit(1)
			                  ->one();
		}
		if (!empty($pharms)) {
			$query = Product::find()
			                ->joinWith('tailing')
			                ->where(['tailings.pharm_id' => $pharm_id]);
		} else {
			$query = Product::find()->joinWith('tailing');
		}
		if (!empty($sort_value)) {
			if ($sort_value === 1) {
				$query->orderBy(['tailings.price' => SORT_ASC]);
			} elseif ($sort_value === 2) {
				$query->orderBy(['tailings.price' => SORT_DESC]);
			} elseif ($sort_value === 3) {
				$query->orderBy(['name_medication' => SORT_ASC]);
			} elseif ($sort_value === 4) {
				$query->orderBy(['name_medication' => SORT_DESC]);
			}
		}
		$pages = new Pagination([
			'totalCount' => $query->count(),
			'pageSize' => 15,
			'forcePageParam' => false,
			'pageSizeParam' => false,
		]);
		$medicines = $query->offset($pages->offset)->limit($pages->limit)->all();
		
		return $this->render('index', [
			'medicines' => $medicines,
			'pages' => $pages
		]);

Comments

[Сергей Свиридонов]

В общем в phpmyadmin такой запрос:

SELECT * FROM `nomenclature` LEFT JOIN `tailings` ON `nomenclature`.`code` = `tailings`.`code` ORDER BY `tailings`.`price` ASC

Тоже говорит что 232 больше чем 17000.. что не так?

Answer 1

[yiiworld]

1) вы сортируете строки, а не числа, поэтому вам кажется что сортировка не работает. Используйте для хранения цен числовые форматы данных в MySQL, например DECIMAL. Но лучше всего использовать Integer(или BigInt) и хранить в них копейки или тысячные доли копеек в зависимости от требуемой точности
2) пагинация тоже работает правильно, просто вы передаете в нее не то что подразумеваете - в totalCount вы передаете объект который подвержен мутациям. Нужно клонировать первоначальный объект от которого зависит пагинация и исполдьзовать его для count
Controller action:

public function actionIndex()
{
    $query = Article::find()->where(['status' => 1]);
    $countQuery = clone $query;
    $pages = new Pagination(['totalCount' => $countQuery->count()]);
    $models = $query->offset($pages->offset)
        ->limit($pages->limit)
        ->all();

    return $this->render('index', [
         'models' => $models,
         'pages' => $pages,
    ]);
}

View:

foreach ($models as $model) {
    // display $model here
}

// display pagination
echo LinkPager::widget([
    'pagination' => $pages,
]);

Comments

[Сергей Свиридонов]

Большое спасибо! Не ожидал столь развернутого ответа. Еще позвольте уточнить: Цены теперь хранить в integer (там можно же указывать 244.4)? Я просто думал что он для ЦЕЛЫХ чисел.
И насколько правильно мое решение в корне? Такое стоит ли делать? Имею ввиду не "гавнокод"?

[Сергей Свиридонов]

И еще, я так понял вы имеете ввиду сделать два поля? Для цены и остатка цены? т.е 204 | 04 копеек?

Если бы подробнее по этому моменту, вопрос закроется. Спасибо вам большое!

[yiiworld]

1) В Integer хранятся Целые, поэтому при его использовании вы храните не рубли, а например копейки или тысячные доли копеек в зависимости от точности которая вам требуется (например 24440), а в коде приложения уже выдаете значения в нужном формате (244руб 40коп, или 244.4)
2) Любой код с точки зрения более опытного программиста это говнокод, поэтому нет предела совершенства
- если в сессии вдруг не будет типа сортировки, то и сортировки не будет
- не надо делать ->limit(1) если делаете ->one(), но нужно если вы хотите получить данный в формате массива объектов записей например: ->limit(1)->all();
3) это

$pharms = Pharmacy::find()
                        ->where(['pharm_id' => $pharm_id])
                        ->limit(1)
                        ->one();

лучше заменить на это

$pharms = Pharmacy::findOne(['pharm_id' => $pharm_id]);

4) (int) используйте для приведения к Целому только если вы уверенны что входные данные представленны именно числом в десятичном измерении т.к. при некоторых обстоятельствах может произойти приведение к восьмеричной или шестнадцатиричной системе исчисления (лучше явно указывать это в функции intval )
5) конструкция из IF ELSEIF ->ORDERBY лучше бы выглядела если бы вы указывали ->ORDERBY однократно без использования IF ELSEIF. Поместите все настройки в массив $oPrams=[1=>['attr_name'=>'tailings.price', 'order'=>SORT_ASC], 2=>[...]...] и делайте ->ORDERBY([$oPrams[$sort_value]['attr_name']=>$oPrams[$sort_value]['order'])
6) не нужно явно открывать сессию $pharm->open();, она по умолчанию открывается автоматически

[Сергей Свиридонов]

Сделал так:

$countQuery = clone $query;
		$pages = new Pagination([
			'totalCount' => $countQuery->count(),
			'pageSize' => 15,
			'forcePageParam' => false,
			'pageSizeParam' => false,
		]);
		$medicines = $query->offset($pages->offset)->limit($pages->limit)->all();

По прежнему выводит рандомно (кол-во).

[yiiworld]

Сергей Свиридонов, данных слишком мало чтобы ответить на ваш вопрос. Вероятно при переходе от страницы к странице меняется количество результативных данных. Сделайте вывод значения count на страницу и посмотрите меняется ли она при переходах по страницам

[Сергей Свиридонов]

yiiworld, написал вот так, ваши рекомендации учту обязательно - спасибо.
А пока, дабы добиться работоспособности, написал это:

public function actionIndex()
	{
		$view = Yii::$app->request->get('view');
		$sort_value = (int) Yii::$app->request->get('sort_value');
		$pharm = Yii::$app->session;
		
		if (isset($pharm['pid'])) {
			$pharm_id = $pharm['pid'];
			$pharms = Pharmacy::findOne(['pharm_id' => $pharm_id]);
		}
		
		if (!empty($pharms)) {
			$query = Product::find()->joinWith('tailing')->where(['tailings.pharm_id' => $pharm_id]);
		}
		
		if (!empty($sort_value)) {
			if ($sort_value === 1) {
				$query = Product::find()->joinWith('tailing')->orderBy(['tailings.price' => SORT_ASC]);
			} elseif ($sort_value === 2) {
				$query = Product::find()->joinWith('tailing')->orderBy(['tailings.price' => SORT_DESC]);
			} elseif ($sort_value === 3) {
				$query = Product::find()->joinWith('tailing')->orderBy(['name_medication' => SORT_ASC]);
			} elseif ($sort_value === 4) {
				$query = Product::find()->joinWith('tailing')->orderBy(['name_medication' => SORT_DESC]);
			}
		}
		
		if (empty($pharms) && empty($sort_value)) {
			$query = Product::find()->with('tailing');
		}
		
		$countQuery = clone $query;
		
		$pages = new Pagination([
			'totalCount' => $countQuery->count(),
			'pageSize' => 20,
			'forcePageParam' => false,
			'pageSizeParam' => false,
		]);
		
		$medicines = $query->offset($pages->offset)->limit($pages->limit)->all();
		
		
		return $this->render('index', [
			'medicines' => $medicines,
			'pages' => $pages
		]);
	}

1, 2, 3 страницы и т.д отображают 20 продуктов. Однако дальше опять идет разброс.

Быть может дело в том, что я для сортировки использую GET? Т.е в адресной строке параметры вида: /index?sort_value=3&page=2
Но когда там: /index?page=2

Проблем нет.

[Сергей Свиридонов]

И еще, второй вариант: /index?page=2
Работает и без клона $query. Но я видел ранее что в доках указан клон. Так и сделаю. Проблема кажется в GET параметрах.. Не подскажете куда копать?

[yiiworld]

Сергей Свиридонов, проблема не в get, а скорее в том, что когда меняется sort_value(или отсутствует), то меняется результирующее количество записей

[Максим Тимофеев]

Сергей Свиридонов, А зачем Вы себя так мучаете? Есть же activeDataProvider в котором все из коробки есть, в том числе и сортировка.

Answer 2

[Сергей Свиридонов]

А насчет цен, я так понял вы имеете ввиду следующее:
если пришла цена (их мы парсим) 1000.50 то я обработав строку заношу в БД 100050. Но при выводе на сайт, последние 2 символа отсеиваю т.е 1000.50.
Тогда как быть с изначально целыми числами? Объясните дубу((

P.S

Спасибо вам! Почему меняется понял, вот только не знаю что теперь делать.

Я так понимаю что дело joinWith() и просто with(). Просто orderBy (как мне кажется) по определению не может влиять на кол-во выдачи ибо он их просто сортирует. Ошибаюсь? Пожалуйста, подскажите.

Нашел еще это:
https://github.com/yiisoft/yii2/issues/8583
https://reformatcode.com/code/php/yii2-leftjoin-no...

И такой запрос выдает 14 записей, а не 50
Product::find()
->joinWith('tailing')
->orderBy(['tailings.price' => SORT_DESC])
->limit(50)
->all();

Comments

[yiiworld]

да, все верно. Храните количество копеек: 12 рублей == 12руб 00коп == 1200 копеек ))

[yiiworld]

если цены мельче копеек(например единица товара в опте), то используйте более мелкие единицы, например млКопейки, тогда 12 рублей === 1200000 млКопеек

[yiiworld]

ну вы в код свой загляните и увидите почему оно меняется (просмотрите все IFы с sort_value)

[Сергей Свиридонов]

yiiworld,

P.S

Спасибо вам! Почему меняется понял, вот только не знаю что теперь делать.

Я так понимаю что дело joinWith() и просто with(). Просто orderBy (как мне кажется) по определению не может влиять на кол-во выдачи ибо он их просто сортирует. Ошибаюсь? Пожалуйста, подскажите.

Нашел еще это:
https://github.com/yiisoft/yii2/issues/8583
https://reformatcode.com/code/php/yii2-leftjoin-no...

И такой запрос выдает 14 записей, а не 50
Product::find()
->joinWith('tailing')
->orderBy(['tailings.price' => SORT_DESC])
->limit(50)
->all();