Модуль mod_adsmanager_menu - AdsManager

Если я правильно понял вашу задумку, при её реализации, чтобы обновить данные о количестве объявлений в каждой категории, нужно будет сделать N запросов на обновление данных, где N - число категорий. Это при том, что прежде то же N число запросов нужно будет сделать, чтобы подсчитать количество объявлений в них. Не дёшево выйдет, если выполнять подсчёты даже раз в час.

Попробуйте так

Код

<?php
defined('_JEXEC') or die('Restricted access');

function mod_adsmanager_menu_categories($id, $level, &$children, $itemid, $current_list, $expand, $order, $displaynumads) {
	global $cur_template, $option;
	if (@$children[$id]) {
		foreach ($children[$id] as $row) {
			if ($level == 0) {
			?>
				<tr align="left"><td>
			<?php
				$class='menu_ads';
			} else {
				echo '<div style="padding-left: '.($level*4).'px"><span id="arrow"></span>';
				$class='sublevel';
			}
			if ($option)
				$link = sefRelToAbs($option."?page=show_category&amp;catid=".$row->id);
			else
				$link = sefRelToAbs("index.php?option=com_adsmanager&amp;page=show_category&amp;catid=".$row->id);

			if ($children[$id] == $row->id)
				$class='menu_ads_1';

			if ($displaynumads == 1) {
				$num = $row->num_ads;
				$nums = number_format($num,0,'.',' ');
				echo '<a href="'.$link.'" class="'.$class.'">'.$row->name.'<sup style="font-weight:normal;color:#999;">&nbsp;'.$nums.'</sup></a>';
			} else {
				echo '<a href="'.$link.'" class="'.$class.'">'.$row->name.'</a>';
			}
			if (@$current_list[count($current_list) - 1 - $level] == $row->id) {
				mod_adsmanager_menu_categories($row->id, $level + 1, $children, $itemid, $current_list, $expand, $order, $displaynumads);
			}
			if ($level == 0) {
			?>
				</td></tr>
			<?php
			} else {
			?>
				</div>
			<?php
			}
		}
	}
}

function calc_nb_ads($id, &$children, $sqltype) {
	switch($sqltype) {
		case 1:
			$nb_ads = 0;
			if (@$children[$id]) {
				$nb = count($children[$id]);
				for($i=0; $i < $nb; $i++) {
					$children[$id][$i]->num_ads += calc_nb_ads($children[$id][$i]->id, $children, $sqltype);
					$nb_ads += $children[$id][$i]->num_ads;
				}
			}
			return $nb_ads;
			break;
		case 0:
			$nb_ads = 0;
			if (@$children[$id]) {
				$nb = count($children[$id]);
				for($i = 0; $i < $nb; $i++) {
					if (!isset($children[$id][$i]->not_empty))
						$children[$id][$i]->num_ads = 0;
					$children[$id][$i]->num_ads += calc_nb_ads($children[$id][$i]->id,$children,$sqltype);
					if (isset($children[$id][$i]->not_empty))
						$nb_ads += $children[$id][$i]->num_ads;
				}
			}
			return $nb_ads;
			break;
	}
}

/****************************************************/

$itemid = intval($params->get('default_itemid', mosGetParam($_GET, 'Itemid', 0)));
$option = $params->get('option', '');
$catid = intval(mosGetParam($_GET, 'catid', -1));
$expand = intval(mosGetParam($_GET, 'expand', 0));
$order = intval(mosGetParam($_GET, 'order', 0));
$displaynumads = $params->def('displaynumads', 1);
$sqltype = $params->def('sqltype', 1);

$database->setQuery("SELECT c.*, COUNT(a.id) AS num_ads ".
				"FROM #__adsmanager_categories AS c ".
				"LEFT JOIN #__adsmanager_ads AS a ON a.category = c.id AND a.published = 1 ".
				"WHERE c.published = 1 GROUP BY c.id ORDER BY c.parent, c.ordering");
$rows = $database->loadObjectList();
if ($database->getErrorNum()) {
	echo $database->stderr();
	return false;
}

// establish the hierarchy of the menu
$children = array();
$orderlist = array();
// first pass - collect children
foreach($rows as $v) {
	$pt = $v->parent;
	$list = @$children[$pt] ? $children[$pt] : array();
	array_push($list, $v);
	$children[$pt] = $list;
	$orderlist[$v->id] = $v;
}


if ($displaynumads == 1)
	$nb_ads = calc_nb_ads(0, $children, $sqltype);

$current_list[] = $catid;
if ($catid != -1) {
	$current = $catid;
	while((isset($orderlist[$current])) && ($orderlist[$current]->parent != 0)) {
		$current_list[] = $orderlist[$current]->parent;
		$current = $orderlist[$current]->parent;
	}
}

if (file_exists($mosConfig_absolute_path.'/components/com_adsmanager/lang/lang_'.$mosConfig_lang.'.php'))
	include_once($mosConfig_absolute_path.'/components/com_adsmanager/lang/lang_'.$mosConfig_lang.'.php');
else
	include_once($mosConfig_absolute_path.'/components/com_adsmanager/lang/lang_english.php');
?>
<table class="moduletable_menu" width="100%" border="0" cellpadding="0" cellspacing="0">
<?php

			if($option)
				$link_show_all = sefRelToAbs($option."?page=show_all");
			else
				$link_show_all = sefRelToAbs("index.php?option=com_adsmanager&page=show_all");
?>
<?php
if ($displaynumads == 1) {
	$nb_ads1 = number_format($nb_ads, 0, '.', ' ');
	$all = ADSMANAGER_MENU_ALL_ADS. '<sup style="color:#666;font-weight:normal;">'.$nb_ads1.'<sup style="color:green;font-size:8px;">+'.$nb_ads_1.'</sup></sup>';
}
else
	$all = ADSMANAGER_MENU_ALL_ADS;
    ?>
<tr align="left"><td><ul><li><a href="<?php echo $link_show_all; ?>" style="font-size:10px;"><?php echo $all; ?></a></li></ul></td></tr>

<?php
mod_adsmanager_menu_categories(0, 0, $children, $itemid, $current_list, $expand, $order, $displaynumads);
?>
</table>

Спасибо. Запрос чуть менее нагружает мускул, но если одновременно 10 человек делают такой запрос, то сайт на секунд 20 уходит.

Я сначала думал кэшировать хоть как то, но при кэше модуль не расскрывает пункты меню.

SELECT c.id, c.parent, c.name, COUNT(a.id) AS num_ads

SELECT c.*, COUNT(a.id) AS num_ads

Если полей в таблице adsmanager_categories много, можно попробовать делать выборку только нужных для результата. Что-то типа

Код

SELECT c.id, c.parent, c.name, COUNT(a.id) AS num_ads

вместо

Код

SELECT c.*, COUNT(a.id) AS num_ads

Боюсь, это максимум, чего можно добиться без кэширования.

Ваш вариант реализации с записью количества объявлений будет намного более нагруженным этого решения, так как тот же запрос на выборку числа (из ячейки, в которую записывается количество объявлений) всё равно нужно будет выполнять, а к нему в добавок прибавится n*2 либо n+1 запросов, где n - количество категорий, которые будут обновлять это число в обозначенные промежутки времени.

Если не секрет, какое количество объявлений на вашей доске? У себя не припомню, чтобы запросы выполнялись более секунды даже при выборке из таблицы с ~350000 строками.

Хм. Есть возможность выполнить предложенный мною запрос в командной строке MySQL или через phpMyAdmin? Интересует время его выполнения. Что-то мне подсказывает, что есть другой, какой-то более тяжёлый запрос либо для генерации страницы делается очень много запросов.

Count: 25  Time=1.75s (43s)  Lock=0.00s (0s)  Rows=257.0 (6425), 
  SELECT c.*, COUNT(a.id) AS num_ads FROM _adsmanager_categories AS c LEFT JOIN _adsmanager_ads AS a ON a.category = c.id AND a.published = N WHERE c.published = N GROUP BY c.id ORDER BY c.parent, c.ordering