Как правильно делать MYSQL join и сортировку?

Question

[mxarts]

Здравствуйте,
есть таблица hy_content в нем id,title,seo
есть таблица hy_views в нем id,read,seo

В "read" при каждом просмотре новости добавляется +1.
Связь между новостью и просмотрами этой новости осуществляет поле "seo".

Сейчас мне нужна грамотное реализация top10 по просмотрам новостей за 24 часа. Нужно объединить таблицы и подсчитать самые просматриваемые за 24 часа. С JOIN -ом думаю, но есть ли альтернатива или вообще как правильно это сделать.

Я реализовал запрос к БД но не знаю на сколько грамотно это сделал, посмотрите пожалуйста и есть ли вариант более правильно это сделать:

$sql = "select hy_content.seo,hy_content.id,hy_content.date,hy_content.sham_photos,hy_content.title from hy_content LEFT JOIN gl_views
ON hy_content.seo=gl_views.seo where hide='2' AND hy_content.date > NOW() - INTERVAL '1' DAY ORDER BY gl_views.read DESC LIMIT 10";

Answer 1

[Rsa97]

Связь таблиц один-к-одному? Тогда грамотно будет держать всё в одной таблице. А так запрос нормально составлен.

Comments

[mxarts]

я счетчик просмотров сделал как отдельную таблицу hy_views чтоб mysql_query_cache работал..

[Rsa97]

Тогда всё логично, только можно и NOW() - INTERVAL 1 DAY заменить на нужную дату и время в программе, тогда и этот запрос будет кэшироваться.

[Melkij]

> только можно и NOW() - INTERVAL 1 DAY заменить на нужную дату и время в программе
Не "можно", а необходимо. NOW() в принципе не попадёт в query cache никогда. Что ещё никогда не попадёт в кэш, описано в мануале: dev.mysql.com/doc/refman/5.1/en/query-cache-operat...

Запрос - вам здесь точно left join нужен?

[mxarts]

@Melkij

есть таблица hy_content в нем id,title,seo
есть таблица hy_views в нем id,read,seo

мне нужно грамотное реализация top10 по просмотрам новостей за 24 часа..нужно объединить таблицы и подсчитать самые просматриваемые за 24 часа.

про NOW() уже понял..

выше написал код запроса - ?..есть ли альтернатива более хорошая чем я реализовал

[kirillzorin]

Left join точно не нужен. Нужен просто join (который inner).

[mxarts]

@kirillzorin пожалуйста напишите грамотный пример inner join -а исходя из моего запроса к mysql

[kirillzorin]

@mxarts, да просто уберите слово LEFT.

Answer 2

[Сергей]

Вы бы запрос отформатировали поаккуратнее - читать неудобно

SELECT hy_content.seo,
       hy_content.id,
       hy_content.date,
       hy_content.sham_photos,
       hy_content.title
FROM hy_content
LEFT JOIN gl_views ON (hy_content.seo=gl_views.seo)
WHERE hide=2
      AND hy_content.date > (NOW() - INTERVAL 1 DAY)
ORDER BY gl_views.read DESC
LIMIT 10

Запрос вполне нормальный, если вам нужно вывести именно новости, опубликованные сегодня, которые больше всего посмотрели (посмотрели за всё время, но т.к. всё время это и есть сегодня, то ок).
Если же нужно именно "самые просматриваемые за сегодня новости, неважно когда опубликованные", то нужно структуру таблиц менять. Т.е. если новость вышла год назад, а на неё вдруг именно сегодня зашла толпа народу.

Comments

[kirillzorin]

Не нужен тут left join, простого хватит.