Расширенная статистика для конкурсов.

[kak2z]

В общем решил месяца два назад я провести у себя на форуме конкурс. Конкурс на лучшего писаку)) Назвал конкурс "Конкурс блогеров". Смысл конкурса в следующем - люди пишут свои статьи. За каждую статью читатели голосуют с помощью мода рейтинга http://dragomano.ru/page/topic-rating-bar
Все статьи выкладывают в одном разделе.
Самая большая загвоздка у меня была в анализе голосов. Конкурс уже был в разгаре, а адекватной статистики так и не было. Общая статистика то есть в моде, но мне нужен был анализ статей в одном разделе, а не по всему форуму.
Немного пошаманив сделал я вот такую табличку http://kak2z.ru/rat.php
Как оказалось самому сделать несложно)

[GeorG]

Ещё бы выронить всю это писанину которая выводится, а то кривовато смотрится. Можно тупо в таблицу засунуть.

[kak2z]

Легко) Три файла из двух архивов положить в корень сайта)
Скрипт делает
1. выборку пользователей по группе
2. выборку их тем из определенного раздела
3. выборку балов каждой темы
4. сортировка

[Inter]

Код Выделить Развернуть

$query_member = sprintf ("SELECT * FROM smf_members WHERE id_group='14'");
Функция sprintf зачем?

[Inter]

Код Выделить Развернуть

while ($row_member = mysql_fetch_array($result_member)){ //Перебираем все пользователей участников конкурса (выборка сделана по ID группы)

$query_topic = sprintf ("SELECT * FROM smf_topics WHERE id_board='65'"); //Выборка smf_topics
$result_topic = mysql_query ($query_topic);

а если у вас 10000 разных id_member в форуме №65 будет?

Лучше так:

Код Выделить Развернуть

$q = mysql_query ('SELECT * FROM smf_topics WHERE id_board = 65');
if (mysql_num_rows($q) == 0)
echo 'Пусто!';
else
{
while ($row = mysql_fetch_assoc($q))
{
$data[] = $row;
// ...
}
mysql_free_result($q);
}

[maestrosite.ru]

Код Выделить Развернуть

$query_member = sprintf ("SELECT * FROM smf_members WHERE id_group='14'");
Функция sprintf зачем?

Пнуть коллегу - это всегда приятно =)) Поэтому, поехали:
1.

Код Выделить Развернуть

require_once $boarddir . 'SSI.php';
require_once dirname(__FILE__) . '/SSI.php';
подключаемся два раза, если учесть

Три файла из двух архивов положить в корень сайта

плюс не определена $boarddir

2.

Код Выделить Развернуть

echo 'Проблемы блин'.mysql_error ().'';
include 'down.php';
die();

Код Выделить Развернуть

//echo '<PRE>';
//print_r ($arr1);
//echo '</PRE>';
вызов не один раз - значит это функция

3.

Код Выделить Развернуть

'<a href="http://kak2z.ru/index.phpзлобный хардкод

4. чего вы не используете $smcFunc ?

5.

Код Выделить Развернуть

$query_member = sprintf ("SELECT * FROM smf_members WHERE id_group='14'"); //Открываем таблицу pic_albums по идентификатору id_album
Прочитал. Не понял.

6.

Код Выделить Развернуть

while ($row_topic = mysql_fetch_array($result_topic)){
if($row_topic['id_member_started']==$row_member['id_member']){
...
}
}из каких соображений в sql-запрос не поставили это условие?

[kak2z]

Код Выделить Развернуть

$query_member = sprintf ("SELECT * FROM smf_members WHERE id_group='14'");
Функция sprintf зачем?

не знаю зачем) раньше без нее делал. А потом увидел где то что правильнее форматировать строку запроса) Хоть там было написано про POST и GET запросы.

а если у вас 10000 разных id_member в форуме №65 будет?

спасибо за замечания но не понял смысла кода)) я еще не совсем быстро в код чужой вникаю)
у меня была цель выбрать всех пользователй из группы "Участник конкурса" а потом в той теме выбрать все их топики... а что я не так сделал? Если можно поподробнее.. спасибо.

[maestrosite.ru]

7. два sql-запроса в цикле -  по смыслу, между таблицами отношение один к одному - напрашивается INNER JOIN

[kak2z]

подключаемся два раза, если учесть

а я где то на форуме сразу нашел две строки) думал две сразу надо)) не вникал особо)) Спасибо буду знать)

вызов не один раз - значит это функция

не понял о чем это.. про include?? а зачем функция? в ней же кроме отключения от базы еще и футер записан... если можно поподробнее про все пишите) а то мне сложно понять где я набочинил)

злобный хардкод

а как надо?)

чего вы не используете $smcFunc ?

а я даже не знал что такая штука есть)) только что прочитал) это вместо mysql_query  или не правильно?

Прочитал. Не понял.

ошибся в комментарии))

7. два sql-запроса в цикле -  по смыслу, между таблицами отношение один к одному - напрашивается INNER JOIN

вообще не понял, но почитаю)

[Inter]

kak2z, в цикле не нужно выбирать - большая нагрузка на базу данных

[kak2z]

а.. то есть создаем массив а потом уже его обрабатываем? правильно я понял?

[maestrosite.ru]

не понял о чем это.. про include?? а зачем функция? в ней же кроме отключения от базы еще и футер записан... если можно поподробнее про все пишите) а то мне сложно понять где я набочинил)

Если код повторяется несколько раз, то оформляйте его в виде функции (метода), а в местах повтора вызывайте эту функцию

а как надо?)

так вы же сами используете $boardurl !
Это же + имя файла вместо $_SERVER['PHP_SELF']  - адрес вашего скрипта вы также несколько раз используете.

а я даже не знал что такая штука есть)) только что прочитал) это вместо mysql_query  или не правильно?

да

8.

Код Выделить Развернуть

if($row_topic_ratings['total_votes']>3) $number_blog_trast++; - а здесь в таблице флаг по идее
Если так, то в запросе к п.7 будет SUM('sum_total_votes) и SUM(_флаг_трастовых_)

[kak2z]

Пункты 7 и 8 вообще не понял о чем речь) Завтра с утра на работе почитаю) тогда буду задавать вопросы) а то даже не знаю что спросить) Спасибо что показываете куда идти) 

[kak2z]

kak2z, в цикле не нужно выбирать - большая нагрузка на базу данных

то есть создаем массив а потом уже его обрабатываем? правильно я понял?

[Gover]

А по какой формуле рейтинг считается?, а то я по коду резко так не соображу.
Мне тут недавно тоже пришлось рейтинг игровых серверов подсчитывать, но только там для Joomla модуль писал.

[kak2z]

количество голосов делю на количество статей - узнаю некий индекс качественности статей.
И прибавляю к этому индексу количество статей у которых голосов больше 3х. Добавляю к качественному показателю количественный. Для того что бы сравнить шансы тех кто пишет много и тех кто пишет качественно)

[Gover]

У меня сложнее формула
где,

V – общее количество всех оценок, поданных за сервер,
Vср – средняя оценка для сервера, расчитывается как отношение суммы голосов к их количеству,
Aср – средняя оценка всех серверов на сайте, суммируются общие рейтинги всех серверов и делятся на их количество,
Vmin – минимальное количество оценок (на данный момент 5), до достижения которого рейтинг сервера стремится к Aср, а после достижения которого стремится к Vср.

на каком-то сайте нашел и адаптировал под свои нужды, вроде работает.

[maestrosite.ru]

kak2z, в цикле не нужно выбирать - большая нагрузка на базу данных

а.. то есть создаем массив а потом уже его обрабатываем? правильно я понял?

Здесь, видимо, выбирать - имеется ввиду выполнять SQL-запрос. Чем больше циклов, тем больше раз базу дёрнуть придётся. Inter вам пример привёл, в котором сам запрос выполнен перед циклом, а в цикле осталось получение данных

[kak2z]

Все равно не понимаю)

Код Выделить Развернуть

while ($row_member = mysql_fetch_array($result_member)){ //Перебираем все пользователей участников конкурса (выборка сделана по ID группы)

$query_topic = sprintf ("SELECT * FROM smf_topics WHERE id_board='65'"); //Выборка smf_topics
$result_topic = mysql_query ($query_topic);

тут я создал массив пользователей (или как его назвать?) $row_member и для каждого пользователя из массива я выбираю топики из раздела 65.

у Интера тут вот что

Код Выделить Развернуть

$q = mysql_query ('SELECT * FROM smf_topics WHERE id_board = 65');
if (mysql_num_rows($q) == 0)
echo 'Пусто!';
else
{
while ($row = mysql_fetch_assoc($q))
{
  $data[] = $row;
  // ...
}
mysql_free_result($q);
}
он просто обрабатывает массив с топиками... но у меня же задача выбрать топики созданные пользователями из определенной группы в определенном разделе.

В принципе можно ускорить так... создать заранее массив всех топиков из раздела 65. Создать еще один массив из всех пользователей определенной группы - а затем уже вертеть массивами без дергания базы... это и логично и правильно... Интер это имел ввиду?

[Inter]

я не собираюсь тут переписывать ваш код, kak2z, просто я показал как правильно нужно выбирать данные

[maestrosite.ru]

Все равно не понимаю)
...
для каждого пользователя из массива я выбираю топики из раздела 65.
...
задача выбрать топики созданные пользователями из определенной группы в определенном разделе.

В принципе можно ускорить так... создать заранее массив всех топиков из раздела 65. Создать еще один массив из всех пользователей определенной группы - а затем уже вертеть массивами без дергания базы... это и логично и правильно... Интер это имел ввиду?

За Inter-a говорить не могу, надо у него самого узнать. Думаю, он просто вам "пусть указал" - вынести запрос (статический!) из цикла

Сейчас у вас 1 запрос пользователей + Y запросов тем + Z запросов рейтингов тем (где Y - количество пользователей, Z - количество участвующих пользователей) + цикл по Y пользователям + Y*Z циклов по темам + Y*Z операций сравнения.

Вынося запрос тем из цикла получаем 1 запрос пользователей + 1 запросов тем + 1 запросов рейтингов тем
+ цикл по Y пользователям + 1 цикл по Z темам.

и далее, объединение запросов, условие в запрос и тд

[kak2z]

я не собираюсь тут переписывать ваш код, kak2z, просто я показал как правильно нужно выбирать данные

да мне уже дошло)) просто опыта пока мало)) вот я и буквально каждый код воспринимаю)

только что пересмотрел все мои скрипты.. почти везде внутри циклов есть обращения к базе((