Жуткие тормоза (100 категорий , 10 000 товаров)

[SooR]

в модели категории, но не суть.

почитай уже топик

if (!empty($data['filter_category_id'])) {
				if (!empty($data['filter_sub_category'])) {
					$implode_data = array();
					
					$implode_data[] = "p2c.category_id = '" . (int)$data['filter_category_id'] . "'";
					
					$this->load->model('catalog/category');
					
					$categories = $this->model_catalog_category->getCategoriesByParentId($data['filter_category_id']);
										
					foreach ($categories as $category_id) {
						$implode_data[] = "p2c.category_id = '" . (int)$category_id . "'";
					}
								
					$sql .= " AND (" . implode(' OR ', $implode_data) . ")";			
				} else {
					$sql .= " AND p2c.category_id = '" . (int)$data['filter_category_id'] . "'";
				}
			}

Это что тогда?

В моделе категории тоже есть, но я сократил кол-во запросов до 36 в категории убрав этот блок

if (!empty($data['filter_sub_category'])) {
					$implode_data = array();
					
					$implode_data[] = "p2c.category_id = '" . (int)$data['filter_category_id'] . "'";
					
					$this->load->model('catalog/category');
					
					$categories = $this->model_catalog_category->getCategoriesByParentId($data['filter_category_id']);
										
					foreach ($categories as $category_id) {
						$implode_data[] = "p2c.category_id = '" . (int)$category_id . "'";
					}
								
					$sql .= " AND (" . implode(' OR ', $implode_data) . ")";			
				} else {

Да и вот тоже..

[freelancer]

$categories = $this->model_catalog_category->getCategoriesByParentId

я устал спорить

[SooR]

freelancer, *** стыд, ну а где вызывается вы видите? И почему и зачем она вызывается вы понимаете?

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

А если галки будут ставиться автоматом, то не придется _искать_ дочерние категории, они уже будут известны в таблице product_to_category.

[SooR]

В некоторых местах так даже удобнее, но это в основном применимо в админке...

В паблике зачастую требуется получить полное дерево категорий, например для меню типа superfish... и тогда лучше рекурсивно трепать массив, а не базу.

Это для админки вообще-то и делалось. Смотрите по какому принципу проставляются категории..

[Yesvik]

Так, секундочку, дерево для чего? Для вытяжки товаров из дочерних категорий, так?

А зачем это делать?

Я приводил пример для чего это надо - superfish.

Если этого мало - добавлю: генератор сайтмэп, выгрузка в Яндекс.Маркет...

[Yesvik]

$implode_data[] = "p2c.category_id = '" . (int)$data['filter_category_id'] . "'";

$this->load->model('catalog/category');

$categories = $this->model_catalog_category ->getCategoriesByParentId($data['filter_category_id']);

foreach ($categories as $category_id) {
	$implode_data[] = "p2c.category_id = '" . (int)$category_id . "'";
}

$sql .= " AND (" . implode(' OR ', $implode_data) . ")";

Если дотошно всё оптимизировать - то цикл в данном месте явно лишний...

$this->load->model('catalog/category');

$categories = $this->model_catalog_category ->getCategoriesByParentId($data['filter_category_id']);

$categories[] = (int)$data['filter_category_id'];

$sql .= " AND (p2c.category_id = '" . implode("' OR p2c.category_id = '", $categories) . "')";