Показать сообщение отдельно
  #15 (permalink)  
Старый 21.12.2012, 11:43
Аватар для Gozar
Отправить личное сообщение для Gozar Посмотреть профиль Найти все сообщения от Gozar
 
Регистрация: 07.06.2007
Сообщений: 7,504

Сообщение от Tim
мы получим именно лес из точек и кавычек.

01	$where = array();
02	        $where[] = '`cpu` LIKE "'.$url.'%"';
03	        $where[] = '`id`!='.$id;
04	        if(!empty($item_catids))
05	            $where[] = '`i2c`.`cat_id` IN ('.implode(',',$item_catids).')';
06	        $query = 'SELECT `cpu` FROM `'.$kernel->pub_prefix_get().'_catalog_'.$kernel->pub_module_id_get().'_items` `items`'
07	                    .'JOIN `'.$kernel->pub_prefix_get().'_catalog_'.$kernel->pub_module_id_get().'_item2cat` `i2c` ON `i2c`.`item_id`=`items`.`id`'
08	                    .'WHERE '.implode(' AND ',$where);
09	        $result = $kernel->runSQL($query);
10	        while($row = mysql_fetch_assoc($result))
11	            $cpu_list[] = $row;
Что это за каша? Я такой код года два не видел.

Сообщение от Tim
Неужели такой стиль лучше?
Это вообще не стиль.

Почитай про pdo что ли. Раз уж ты используешь $kernel->, зачем использовать mysql_fetch_assoc, а не $s->fetch(). Как минимум ты будешь в одном стиле писать, про комбу в рукаве я вообще молчу.
__________________
Последний раз редактировалось Gozar, Сегодня в 24:14.
Ответить с цитированием