Сообщение от 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(). Как минимум ты будешь в одном стиле писать, про комбу в рукаве я вообще молчу.