Как по имени пользователя получить данные из базы и подсчитать их?

Question

[Иван]

Мне необходимо из базы данных по имени пользователя получить количество положительных и отрицательных отзывов, где положительные имеют статус ok, а отрицательные trush

Вот так обращаюсь к базе:

$reviews = $db->super_query("SELECT * FROM ".PREFIX."_users_balance WHERE name='".$row['autor']."'",true);

Как далее получить результат для конкретного пользователя $row['autor'] из поля status и сложить результат?

Т.е. в таблице PREFIX."_users_balance могут быть сотни записей про одного юзера $row['autor'], как со статусом ok, так и trush, а получить в итоге нужно числа.

Кроме того, информации о пользователе может и не быть в таблице или статус может иметь иное значение.

Помогите сделать верный запрос и вывести результат.

Спасибо!

Answer 1

[Gennady S]

Ну, вывести результат придется самостоятельно. А вариантов запросов множество. Через JOIN, через вложенные запросы, через группировку по значению, но в любом случае с агрегацией. Далее примеры решения, без знания структуры Ваших данных и без оптимизаций. В первом случае Вы получите по 2 (и более) строки на каждого пользователя (например, user_id=1, status='ok', name= 'John galt', status_count=2 и user_id=1, status='trash', name= 'John galt', status_count=67):

SELECT *, COUNT(status) as status_count FROM table_name WHERE name = :name GROUP BY user_id, status;

В этом случае получите 1 строку на каждого пользователя, но запрос, возможно, тяжелее (user_id=1, name= 'John galt', status_ok_count=2, status_trash_count=95)

SELECT COUNT(ok.status) as status_ok_count, COUNT(trash.status) as status_trash_count, table_name.user_id, table_name.name
FROM table_name WHERE name=?
   INNER JOIN table_name as ok ON ok.user_id = table_name.user_id
   INNER JOIN table_name as trash ON trash.user_id = table_name.user_id
WHERE table_name.name = :name AND ok.status = 'ok' AND trash.status='trash'
GROUP BY table_name.user_id;

PS Есть смысл делать выборку не по имени, а по идентификатору, если надо получить данные на пользователя (а не найти именно по имени), запросы по первичному ключу всегда существенно быстрее.

Comments

[Иван]

Спасибо!
Видимо придется переписать скрипт, добавить id пользователя, да наверное и статус сделать не ok, trush, а 0 и 1, так наверное будет проще.

[Gennady S]

Иван, безусловно, все данные лучше связывать идентификаторами, а флаги и статусы хранить константами. Но и ничего особо критичного в "ok", "trash" нет, можно оставить, если это повысит удобство, например.

[Иван]

Gennady S, заменил поиск с имени на id, в вашем первом примере не смог разобраться, нашел такой вариант

$reviews = $db->super_query("SELECT count(status = 'ok') as ok_count, count(status = 'trush') as trush_count FROM ".PREFIX."_users_balance WHERE user_id='$user_id'",true);

Выборка по user_id, не знаю верный запрос или нет, как считаете?

Пробовал вывести результат с помощью while, ничего не получилось.

[Gennady S]

Иван, нет, это тарабарщина получается. SELECT указывает только поля выборки, все выражения в разделе SELECT применяются тут же. Результаты могут группироваться (GROUP BY) по полям. Так, чтобы вывести COUNT, SUM, AVG и другие результаты обобщения (COUNT за кадром сводит в одну строку результат выборки нескольких строк), движку нужно знать, относительно чего эти строки группировать. Все условия указываются в выражении WHERE, там движок смотрит, по каким критериям отбирать. В выше приведенном примере (я сам его не тестировал, т.к. не на чем, у меня нет Ваших данных, могут быть огрехи):

SELECT *, COUNT(status) as status_count FROM table_name WHERE name = :name GROUP BY user_id, status;

— мы группируем выборку по пользователям и по статусу, а значит, движок выберет все строки с условием, указанным в WHERE, скажем, их 10, и разобьет на строки по указанным условиям в GROUP BY, обобщив результаты групп в COUNT (функция считает число строк в группе).

Почитайте что-то на тему агрегирования в SQL. Да и вообще, основы SQL. Потратите час-два — пригодится на всю жизнь. Вслепую тыкаться не вариант.

[Иван]

Gennady S, буду читать.
Ваш пример я не понял, вот что я делаю

SELECT *, COUNT(status) as status_count FROM ".PREFIX."_users_balance WHERE name = :name GROUP BY user_id, status;

На что заменять значения после WHERE?
name = :name
user_id
Или так и нужно оставлять, у меня ошибка получается )))

Видимо нужно все таки менять ok и trush на цифры и выводить с помощью sum результаты.

sum возвращает сумму значений заданного поля

[Gennady S]

Иван, :value_name — это расчет на подстановку значения по именованному параметру, вот на примере PDO: https://www.php.net/manual/ru/pdo.prepare.php — там есть рабочие примеры. Это корректный путь установки значения в запрос, вместо 'WHERE name = ' . $name, Вам необязательно это использовать. Но это правильный путь. user_id приведено для примера. Почитайте что-то по агрегированию и группировке, перечитайте и мое сообщение выше — "группируем выборку по пользователям и", если user_id нет, значит нужно по имени группировать. Разбирайтесь с SQL. Поймите, не освоив хотя бы азы, нельзя работать в слепую. Ставите под угрозу приложение и результат работы. А освоить SQL не сложно — всего-навсего перебираете строки в таблицах.