[Вопрос]

[syn]

Коллеги, ужны ваши советы, подсказки по коду.
Как бы так хитро извернуться для уменьшения количества запросов.

Это код вывода аватарок на главной у ластпостеров.

$qtext = "SELECT userid, avatarid, avatarrevision  FROM `" . TABLE_PREFIX . "user` WHERE username = \"" . $lastpostinfo[lastposter] . "\"";
$qresult = $vbulletin->db->query_read($qtext);
$rowarray = $vbulletin->db->fetch_array($qresult);
$userid = $rowarray['userid'];
$avatarid = $rowarray['avatarid'];
$avatarrevision = $rowarray['avatarrevision'];

if (!($userid))
{
	// Dummy id, will result to have anonymous avatar
	$userid = 0;
}

if ($avatarid)
{
	$avatar = $vbulletin->db->query_first("SELECT avatarpath FROM `" . TABLE_PREFIX . "avatar` WHERE avatarid = " . $avatarid . "");
	$avatarpath = $avatar['avatarpath']; 
	$avatarforumhome = "<img src=\"$avatarpath\" style=\"width:25px; height:25px\" alt=\"$lastpostinfo[lastposter]\" />"; 
}
else
{
	// Check if it's custom avatar
	if ($avatardata = $vbulletin->db->query_first("SELECT dateline, userid FROM `" . TABLE_PREFIX . "customavatar` WHERE userid = " . $userid . ""))
	{ 
		$useravatarurl = $vbulletin->options[avatarurl] . "/avatar" . $avatardata['userid'] . "_" . $avatarrevision . ".gif";
		$avatarforumhome = "<img src=\"$useravatarurl\" style=\"width:25px; height:25px\" alt=\"$lastpostinfo[lastposter]\" />";
	}
	else
	{
		// No avatar for this user
		$avatarforumhome = "<img src=\"tps/img/unknown.gif\" style=\"width:25px; height:25px\" alt=\"$lastpostinfo[lastposter]\" />";
	} 
}

[kerk]

перепиши запрос таким образом, что бы вывод результата был в цикле

PHP код:

$qresult = $vbulletin->db->query_read("здесь_запрос_всех_данных");

while($rowarray = $vbulletin->db->fetch_array($qresult))
{
       // здесь вывод результата запроса
} 


[kerk]

да, еще
если планируется использование на странице форумхом или форумдисплей, то там достаточно хуков, что бы не писать свои запросы, а использовать готовые вобловские + выборка нужных полей

[syn]

ага, на форумхоум, но я полазил там вокруг да около, но чет пока так и не сообразил, каким воблозапросом это дело выдернуть.

[kerk]

индексная страница, еще не означает ковыряние в файле index.php
там нужно рыть в файлах функций, которые создают форумбиты
functions_forumlist.php
functions_forumdisplay.php

[syn]

kerk, да я в индекс и не пялился, разумеется =)
вот как раз по фанкшинам и зырил.

syn добавил 20.08.2011 в 17:13
ну и в любом случае, все равно на хоум пойдет +1 запрос [как минимум].
совсем избавиться от лишнего запроса ж ведь все равно не получится, даже при использовании воблозапроса.

хотя, безусловно корректнее, использовать родной механизм.


syn добавил 20.08.2011 в 17:14
пойду дальше втыкать в воблофанкшины =)

[kerk]

хм
щас глянул файлы
в одной из функций нет хуков вообще, значит хрен чего сотворишь
значит нужно редактировать файлы + добавлять поля в таблицу forum
тогда можно вообще обойтись без доп запросов
т.е. при добавлении поста, вносить данные о юзере в эти поля, кроме тех, что форум сам пихает, а это время последнего поста, кол-во тем, кол-во ответов

[syn]

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

[netwind]

если уж экономить запросы, то можно так написать:

select
 ifnull(avatar.avatarpath,
  concat('customavatars/avatar',user.userid, '_', avatarrevision, '.gif')
 ) filename
from user
 left join avatar on (user.avatarid=avatar.avatarid)
 left join customavatar on (user.userid=customavatar.userid)
where user.userid=$userid

тут сделано предположение, что аватары в файловой системе в папке customavatars, а не в базе.
для хаков широко применения такое может не подойти

netwind добавил 20.08.2011 в 17:32

Сообщение от syn

чисто гипотетически можно подвязаться к иду ласпостера

поздравляю. ты придумал JOIN теперь почитай про него в документации.

[kerk]

Сообщение от syn

просто добавлять к нему выборку из аватар -> аватарид_аватарревижн

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