Оптимизация кода

[FedY@]

Оптимизация кода

Подсажите пожалуйсто, может кому - то (професионалу) покажеться глупым, но для этого и есть этот форум что нужно спрашивать! Вот код:

if($rooms!='Выберите'){
        if($add==true){
                $sql.=" AND rooms='".$rooms."'";
                }else{
        $sql="SELECT * FROM flat_sail WHERE rooms='".$rooms."'";
        $add=true;
}
}
if($type!='Выберите'){
        if($add==true){
                $sql.=" AND type='".$type."'";
                }else{
        $sql="SELECT * FROM flat_sail WHERE type='".$type."'";
        $add=true;
}
}
if($etag!=''){
        if($add==true){
                $sql.=" AND etag='".$etag."'";
                }else{
        $sql="SELECT * FROM flat_sail WHERE etag='".$etag."'";
        $add=true;
}
}

Что нужно зделать чтобы этот код был немного меньше, я пробывал через функцию - не получилось!

 

[Сенсей]

$HTTP_POST_VARS --- >> $_POST

гы

Кстати, SQL иньекция тебе обеспечена

 

[FedY@]

Я не такой ответ ждал!

 

[Макс]

if ($_SERVER['REQUEST_METHOD'] == 'POST') {
   $sql = 'SELECT * FROM flat_sail';
   $params = array();
   if ($name) $params[] = "name = '".$name."'";
   if ($type != 'Выберите') $params = "type = '".$type."'";
   if ($etag) $params[] = "etag = '".$etag."'";
   ........
   if (sizeof($params)) 
      $sql .= " WHERE ".implode(" AND ", $params);
}

если у тебя отключен magic_quotes_gpc то будет дыра
И код здесь нечистый (много нотайсов может быть)
Если вместо значений "Выберите" будешь передавать пустое значение, то можно еще упростить :

if ($_SERVER['REQUEST_METHOD'] == 'POST') {
   $fields = array('rooms','name','type','etag');
   $sql = 'SELECT * FROM flat_sail';
   $params = array();
   foreach ($fields as $f ) {
        if ($_POST[$f]) $params[] = " $f ='".$_POST[$f]."'";
   }
   if (sizeof($params)) 
      $sql .= " WHERE ".implode(" AND ", $params);
}

 

[neko]

Location: Ukraine, Boyarka

у них там что, радиационный фон повышен чтоли....

 

[FedY@]

обьясни что такое sizeof(); ?

-~{}~ 06.06.04 16:55:

Автор оригинала: neko
Location: Ukraine, Boyarka

у них там что, радиационный фон повышен чтоли....

ТЫ на УКРАИНУ, и на её города не гони, ты понял?

 

[lucas]

что такое sizeof();

[m]sizeof[/m], лучше использовать [m]count[/m]

-~{}~ 06.06.04 18:58:

ТЫ на УКРАИНУ, и на её города не гони

Вообще-то, гонят на одного жителя...

 

[FedY@]

Автор оригинала: lucas
Вообще-то, гонят на одного жителя...

вообще он написал у них ... или я слеповат, кроме радиации?

 

[lucas]

у них

Это он тебя на "вы" назвал.

 

[FedY@]

можно вопрос не по теме, чтобы не создавать топик,
как зделать запрос, чтобы из базы данных выводилось например записи за посл. 10 дней, есть столбец date DATE; ( 2004-06-06).

 

[IBSN]

Автор оригинала: lucas
Это он тебя на "вы" назвал.

 

[.des.]

sizeof, лучше использовать count

чем лучше?

 

[IBSN]

Автор оригинала: FedY@
можно вопрос не по теме, чтобы не создавать топик,
как зделать запрос, чтобы из базы данных выводилось например записи за посл. 10 дней, есть столбец date DATE; ( 2004-06-06).

DATE_SUB(date ,INTERVAL expr interval)

 

[FedY@]

Автор оригинала: IBSN
DATE_SUB(date ,INTERVAL expr interval)

Примерчик можно ?

 

[IBSN]

может мне еще и станцевать???
можно за 10$

 

[neko]

- Слушай, девочка, приниси весь мешок, я тебе их сразу наколю.
- Ты, что, думаешь на помойке орехи мешками валяются?!

 

[FedY@]

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

-~{}~ 06.06.04 18:41:

SELECT * FROM flat_sail WHERE DATE_SUB(DATE, INTERVAL 2 DAY)

какая тут ошибка ? выводит все... не смотря на интервал!

 

[Crazy]

FedY@, флудишь по тредам?

 

[FedY@]

нет, просто с этой темы перешел на ту... потому что тут ничего не отвечали!

 

[lucas]

.des.

sizeof, лучше использовать count
чем лучше?

It is usually a bad idea to use aliases, as they may be bound to obsolescence or renaming, which will lead to unportable script.

(c) manual