evgeny777
У тебя есть массив $_POST['komnat'] , вот там и содержатся данные только отмеченных чекбоксов
И не надо создавать левые переменные! Работай непосредственно с имеющимся массивом.

inpost а как в таком случает выгружать все данные (если ничего не отметили)? может примерчик скинешь чтобы принцип понять?

qqq

$query = "... WHERE `komnat` IN (".implode(',',$_POST['komnat']).")";

И не забываем данные привести к целочисленному виду, для защита от иньекций

$_POST['komnat'] = array_map('intval', $_POST['komnat']);

inpost большое спасибо за помощь, но при вашем варианте из базы ничего не выгружается если ничего не отмечено. это как то возможно исправить?

evgeny777
Конечно, а теперь подумай сам, как сделать так, чтобы в условие не попадало IN (), если массива нет

у меня получился вот такой вариант, но думаю, что это не идеал.... и его можно оптимизировать :)

if (empty($_POST['komnat'])) {
$komnatzapros="komnat LIKE '%%'";
 } else {
 $km1=$_POST['komnat'][0];
 $km2=$_POST['komnat'][1];
 $km3=$_POST['komnat'][2];
 $km4=$_POST['komnat'][3];
 $km5=$_POST['komnat'][4];
$komnatzapros="komnat !='' AND (komnat='$km1' OR komnat='$km2' OR komnat='$km3' OR komnat='$km4' OR komnat='$km5')";
 }
  $query = "select * from base where (."$komnatzapros.")"; 

evgeny777
Я что тебе написал? НАФИГА ТЕБЕ $km1 ? Объясни, что мешает тебе пользоваться переменной $_POST['komnat'] ?
И 2, я дал оптимильный код, а ты всё испоганил

да killer8080 должен быть дефолтовы вариант на случай если по этому парамету ничего не проставлено, так как в моей базе данных около 30 различных параметров кстати большое спасибо за совет про инъекции! правда я в этом пока не особо разбираюсь а можно как то сделать защиту от инъекций в том варианте который я написал выше?
inpost вы имеете в виде, сделать проверку на пустоту, если массив пустой то выгружать все занчения через '%%' (как в моем верхнем варианте), а если если что то отмечено то через ваш ариант через IN ?

inpost я оптимизировал и сделал вот так, но почему то когда у меня что то отмечено запрос не работает, работает только когда ничего не отмечено:
if (empty($_POST['komnat'])) {
$komnatzapros="komnat LIKE '%%'";
} else {
$komnatzapros="`komnat` IN (".implode(',',$_POST['komnat']).")";
}
$query = "select * from base where (".$komnatzapros.") ";

Ты можешь объяснить, что значит эта запись:
komnat LIKE '%%'
То есть какой смысл в ней?

evgeny777

$query = "SELECT * FROM `base`".
         (
             !empty($_POST['komnat'] && is_array($_POST['komnat']) ? 
             " WHERE `komnat` IN (".implode(', ', array_map('intval', $_POST['komnat'])).")" : 
             ""
         );

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

killer8080 большое спасибо!
только в твоем коде пропущена скобка: !empty($_POST['komnat'])

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

$query = "SELECT * FROM `base`".
         (
             !empty($_POST['metro']) && is_array($_POST['metro']) ? 
             " WHERE `metro` IN (".implode(', ', $_POST['metro']).")" : 
             ""
         );

т.е. когда ничего не отмечено - выгружается все, а кога отмечены конкретные чекбоксы - то ничего не выгружается

evgeny777
потому что текстовые данные должны быть в кавычках. И для защиты от SQLi тут нужно использовать mysql_real_escape_string().

$query = "SELECT * FROM `base`".
         (
             !empty($_POST['metro']) && is_array($_POST['metro']) ? 
             " WHERE `metro` IN ('".implode("', '", array_map('mysql_real_escape_string', $_POST['metro']))."')" : 
             ""
         );

ураааа! заработало! killer8080 огромное тебе спасибо!

evgeny777
да на здоровье

PS так глядишь, в новый год с трехзначной кармой