Ну кстати работает нормально. :goodpost: Но в принципе у меня селектов может быть любое количество, так что буду думать над генератором строки запроса.

Что интересно, получилось так вот:
[PHP]
if(isset($_GET['groups'])){
$groups = array_merge(array_diff($_GET['groups'], array('0'=>'')),array());
}
$sql = '';
$brackets = '';
$groupMaxKey = count($groups) - 1;
for($i=0;$i<=$groupMaxKey;$i++){
$sql .= "SELECT itemId FROM item_group WHERE groupId='".$groups[$i]."' ";
if($i != $groupMaxKey){
$sql .= "AND itemId IN (";
$brackets .= ")";
} else {
$sql .= $brackets;
}
}
[/PHP]
Самое обидно будет если сча придет крутой прогер и скажет что в мускл5.6 есть INTERSECT :)

Можно так. Данные:
[IMG]http://funkyimg.com/u2/1621/695/46846627-04-2012_20-31-10.jpg[/IMG]
Запрос:

Результат:
[IMG]http://funkyimg.com/u2/1621/715/74361927-04-2012_20-35-08.jpg[/IMG]

xdrew, какой вариант быстрее? Я так полагаю с группировкой есть риск что пробежит по всей таблице(я не имею ввиду возврат результатов клиенту), а

тут его точно нет.

На больших выборках должно тормозить (хотя мне кажется, что в этом случае таблица не настолько большая). Ради интереса проверю.

xdrew, вот спасибо за пример запроса (просто я любитель, и использую sql по мере потребностей, так что узнал что-то новое), но к сожалению он тут не подойдет.
Дело в том что количество групп в которых может находится элемент может быть любым, а фильтрация может так же проходить с указанием любого числа групп.

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

Из вопросов у меня остался один:
1. Насколько говнокодно выглядит формирование строки sql запроса, и делают ли так вообще нормальные программеры?



Пробегает в любом случае по всей таблице мой вариант. Ну если появится рабочий вариант запроса, можно посмотреть по числу транзакций что быстрее

Конечно, только лучше сделать это более читаемым использовав implode сколько-то раз.
пример(отвелеченно):
[PHP]
$ins = array();
$fields = array('id', 'v1', 'v2');
foreach ($vals as $val) {
$insdata = array();
foreach ($fields as $f) $insdata[] = $val[$f];
$ins[] = '('.implode(',',$insdata).')';
}
$ins = 'INSERT INTO '.$tbl.' ('.implode(',',$fields).') VALUES .'.implode(',',$ins);
[/PHP]

По крайней мере лучше используйте не ".=" а array_push и implode для построения запроса/подзапросов, разобраться будет легче откуда что берется.




А я вовсе не про Ваш вариант, я про вариант с выборкой с конечным условием WHERE/HAVING для финального отсева нужных данных для двух приведенных примеров(Вашего с 2 подзапросами и xdrew с группировкой) и внутренние операции mysql-сервера. Часто в этом случае при группировке наблюдается fullscan вместо предварительного создания временной таблички, без видимых причин - как оптимизатор решит. А с фильтрацией по условию IN в подзапросах по индексу - такого нет никогда. Вот про что я, поэтому на реальные результаты и хотелось бы посмотреть.

В вашем случаем implode хорош тем, что нет проверки надо ли ставить запятую после последнего элемента. :) А по поводу и без такие приемы хороши в JS, т.к. в нем += намного медленнее чем push.

если говорить о js, то конкатенация через +(+=) медленнее только в IE который на каждую конкатенируемую строчку создает отдельный объект. В остальных монопенисуально.
P.S. впрочем у самого проверить это руки не дошли, возможно я не прав.

А мы используем для совсем сложных запросов не implode а самописный построитель, удобно когда join'ится порядка 16 табличек, кучи полей, кучи фильтров которые могут быть а могу и не быть.
что-то типа
[PHP]
$this->db->select('a.*')->select('b.*, c.field')->from('tbl_1 as a');
$this->db->from('tbl_2 as b')->leftJoinEqual('tbl_3 as c', 'c.ext_id', 'a.id');
if ($filters) {
$this->db->whereEqual('c.field_2',$filters['f']);
$this->db->whereIn('a.field', array(1,2,3));
}
$this->db->groupBy('b.field');

//debug echo $this->db->sql;
$this->db->Marshal('Entity_test');// $this->db->getArray(); $this->db->getGroupArray('c.ext_id'); $this->db->getXML() и т.п.
[/PHP]

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