Как лучше составить запрос к БД?

Question

[Александр]

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

$comd_sql = $sql->run("select * from ".MY_PREFIX."_chatArhive where news_id = ? and quote_id = ?  order by id ASC",array($_REQUEST['newsID'],$_REQUEST['msgID']));

Как доработать, чтобы добавить сюда данные по этому же условию (таблицы идентичны) из таблицы ".MY_PREFIX."_comments?
За ранее спасибо

Comments

[Александр]

Далал через union, но не работало, оказывается просто синтаксическая ошибка в запросе была. Всем спасибо, сам сделал

[ZIK1337]

Александр, разве нельзя просто через запятую в FROM указать несколько таблиц?

[Александр]

ZIK1337, гуглил, вроде можно, но у меня почему то ошибку возвращает запрос... Может я что-то не так делал конечно, но по сути, что тут не так может быть?

[ZIK1337]

Александр, покажите ошибку

[FanatPHP]

ZIK1337, можно.
А ты знаешь что получится в результате?

[Александр]

ZIK1337, вот запрос

$comd_sql = $sql->run("select * from ".MY_PREFIX."_chatArhive,".MY_PREFIX."_comments where news_id = ? and quote_id = ?  order by id ASC",array($_REQUEST['newsID'],$_REQUEST['msgID']));

Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[23000]: Integrity constraint violation: 1052 Column 'news_id' in where clause is ambiguous' in /var/www/user/data/www/site.pro/systemDir/classes/DbMysql.php:39 Stack trace: #0 /var/www/user/data/www/site.pro/systemDir/classes/DbMysql.php(39): PDOStatement->execute(Array) #1 /var/www/user/data/www/site.pro/systemDir/classes/ajax.php(2016): DbMysql::run('select * from u...', Array) #2 /var/www/user/data/www/site.pro/systemDir/classes/ajax.php(49): selectHistoryMsg(Object(Ajax), Object(Error_handler)) #3 {main} thrown in /var/www/user/data/www/site.pro/systemDir/classes/DbMysql.php on line 39

Вот ошибка:

[ZIK1337]

FanatPHP, особенностей php не знаю, а по запросу декартово произведение по идее получится?

[FanatPHP]

ZIK1337, смотри-ка, успел нагуглить :)

[ZIK1337]

FanatPHP, ага, за минуту?

[FanatPHP]

Александр, ошибка исправляется легко, но сам запрос через запятую ничего хорошего тебе не принесет

[FanatPHP]

ZIK1337, не, ну если бы заранее знал, то советовать же не стал бы?
или типа из-за каких-то "особенностей РНР" такой запрос вернет что-то другое? :)

[Александр]

FanatPHP, ну разобраться в чем ошибка интересно было бы. Для себя понять... А вариант с union меня вполне устраивает, вот только проблемы при union с ZEROFILL

Приходится велосипед делать такой

$comd_sql = $sql->run("(select *,CONVERT(`ho`,CHAR) as `ho`,CONVERT(`mi`,CHAR) as `mi`,CONVERT(`da`,CHAR) as `da` from ".MY_PREFIX."_chatArhive where news_id = ? and quote_id = ?) union (select *,CONVERT(`ho`,CHAR) as `ho`,CONVERT(`mi`,CHAR) as `mi`,CONVERT(`da`,CHAR) as `da` from ".MY_PREFIX."_comments where news_id = ? and quote_id = ?)  order by id ASC",array($_REQUEST['newsID'],$_REQUEST['msgID'],$_REQUEST['newsID'],$_REQUEST['msgID']));

[ZIK1337]

FanatPHP, забыл просто, только недавно начал изучать, и почти не пользовался еще субд

[ZIK1337]

Александр, вроде из-за того, что одинаковые имена у столбцов - надо указывать название таблицы еще перед ними

".MY_PREFIX."_chatArhive.news_id = ? and ".MY_PREFIX."_chatArhive.quote_id = ?

типо такого, но как выяснили - выведет декартово произведение строк таблиц, надо будет добавить еще условие на равенство значений уникального столбца этих 2 таблиц

[Александр]

ZIK1337, понятно. Спасибо, но думаю что union вполне нормально справляется )))

[FanatPHP]

Александр, а просто два запроса сделать - не вариант?

Или - если уж архивная таблица так активно используется - то совсем от нее отказаться?

[Александр]

FanatPHP, два запроса? И потом как-то собирать данные в массив, чтобы передать методу для обработки данных дальше? Мне кажется это куда сложнее чем просто использовать union.
Таблица архивная, но ее могут просматривать по запросу, сохранить нужно.
А вся канетель собственно из-за того, что сообщения в архив попадают спустя двое суток. А при просмотре архива, кем либо, он может ответить на комментарий. Но автор его не увидит (хоть и получит уведомление), потому как сообщение новое и не в архиве еще... Вот и приходится костыль такой делать

[FanatPHP]

Сделай одну таблицу, гений.
По дате отправки всегда можно определить, сколько суток прошло

[Александр]

FanatPHP, архивная таблица это на данный момент ~ 500 000 записей. Таблица с комментариями сейчас имеет около 150 000 записей. Архив используется очень редко. И естественно выборка работает куда быстрее таким образом

[FanatPHP]

Чушь.
Выборка по индексу из таблицы в 650000 записей ничем не отличается от выборки из 150000

Хочу видеть explain обоих запросов и конкретные цифры "куда быстрее"

[Александр]

FanatPHP, расчет на будущее. рано или поздно с этим придется столкнуться, если не чистить

[FanatPHP]

Ох уж эти оптимизаторы...
Все им мерещится проект размером с фейсбух, не меньше.
И прямо сейчас надо уродовать код и бд в угоду влажным эротическим мечтам о несбыточном

[Александр]

FanatPHP, Ну а ты чего так заводишься? у каждого свои идеи, свой подход, своя реализация и свое видение. Смысл с пеной у рта кому-то что-то доказывать? Как совет, я безусловно принял и возможно в будущем, возьму это на заметку. Но сейчас, переделывать много в структуре кода нужно будет. Ну а из-за одного union и раздельной таблицы говорить про уродство кода, думаю, преждевременно!

[FanatPHP]

Это ты не видел как я с пеной у рта доказываю )))
У меня нет иллюзий, что ты сразу перестанет решать несуществующие, воображаемые проблемы.
Здесь просто сожаление, что есть такие новичковые заблуждения, которые ничем не пробить. В частности вот такое решение проблем про запас

[Александр]

FanatPHP, ну ведь еще все зависит от сервера... Ну и конечно программиста, как он запрос напишет )))... А вообще, сколько реально записей может без проблем обрабатывать БД, если разницы между 100К и 500К нет совсем?

[FanatPHP]

Самое-то смешное, что "требуется очень редко" - это такое же "точное" предсказание, как и то что "выборка куда быстрее".

И вместо того чтобы сделать нормальную структуру бд и оптимизировать запросы, ты сейчас получил кривой код, который делает ДВА медленных запроса :)))

[FanatPHP]

сколько реально записей может без проблем обрабатывать БД

Зависит от железа.
Если индексы влезают в память, то десятки миллионов без проблем.

Ты главное не думай что юнион - это один запрос.
Для БД это два разных запроса, которые она выполняет отдельно и потом просто склеивает результат.

Answer 1

[Андрей Ежгуров]

$comd_sql = $sql->run(
  '(SELECT * FROM ' . MY_PREFIX . '_chatArhive WHERE news_id = ? AND quote_id = ?)
UNION
(SELECT * FROM ' . MY_PREFIX . '_comments WHERE news_id = ? AND quote_id = ?)
ORDER BY id ASC',
  [$_REQUEST['newsID'], $_REQUEST['msgID'], $_REQUEST['newsID'], $_REQUEST['msgID']]);

Comments

[Александр]

Да, спасибо. Так и сделал!