if w mysql_query Opcje

[akurczyk]

Mam taką linijkę kodu:

[PHP] pobierz, plaintext 
<?php
$wykonaj_zapytanie = mysql_query("SELECT * FROM porady WHERE 1=1 ".if(isset($_GET['jezyk'])) {." AND jezyk='".addslashes($_GET['jezyk'])."'}.if(isset($_GET['fraza'])) {." AND MATCH(tytul, tresc) AGAINST('".addslashes($_GET['fraza'])."')&#092;".if(isset($_GET['strona'] && is_numeric($_GET['strona']))." LIMIT \"($_GET['strona']-1)*50.",50.".} else {." LIMIT 0,50".});
?>
[PHP] pobierz, plaintext 

Jak zrobić aby ify w tym kodzie zaczęły działać
Chodzi mi oto że gdyby użytkownik nie podał języka, frazy lub numeru strony (w adresie) wyniki miały by się wyświetlić.
Proszę o pomoc!!!

Ten post edytował akurczyk 28.08.2008, 21:45:23

[bim2]

[PHP] pobierz, plaintext 
<?php
$wykonaj_zapytanie = mysql_query("SELECT * FROM porady WHERE 1=1 ".(isset($_GET['jezyk'])?" AND jezyk='".addslashes($_GET['jezyk']):"")."".(isset($_GET['fraza'])?" AND MATCH(tytul, tresc) AGAINST('".addslashes($_GET['fraza'])."')":"").((isset($_GET['strona']) && is_numeric($_GET['strona']))?" LIMIT \"($_GET['strona']-1)*50}":" LIMIT 0,50".));
?>
[PHP] pobierz, plaintext 

pomieszałeś, ale powinno chodzić

[likemandrake]

Hmmm, wg mnie trochę zła praktyka programistyczna, to jest jak wrzucenie wszystkich śmieci do jednego worka.

Na mój gust to powinno wyglądać mniej więcej tak:

[PHP] pobierz, plaintext 
<?php
$iOffset = 0;
$iLimit = 50;
$aWhereStmt = array();
$sQuery = 'SELECT * FROM porady';
if(!empty($_GET['jezyk'])) {
	$aWhereStmt[] = "jezyk = '" . mysql_real_escape_string($_GET['jezyk']) . "'";
}
if(!empty($_GET['fraza'])) {
	$aWhereStmt[] = "MATCH(tytul, tresc) AGAINST '" . mysql_real_escape_string($_GET['fraza']) . "'";
}
if(sizeof($aWhereStmt) > 0) {
	$sQuery .= ' WHERE ' . join(' AND ', $aWhereStmt);
}
if(ctype_digit($_GET['strona']) && $_GET['strona'] > 0) {
	$iOffset = ($_GET['strona'] - 1) * $iLimit;
}
$sQuery .= " LIMIT $iOffset, $iLimit;";
?>
[PHP] pobierz, plaintext 

Kod pisałem na czysto, bez sprawdzania, więc nie gwarantuję, że działa poprawnie.

Po za tym do obsługi bazy danych proponuję użyć PDO, dla PHP > 5, oprócz tego powinna się pojawić weryfikacja, czy przypadkiem nie podano numeru strony spoza zakresu dostępnych do wyświetlenia danych (choć nie jest to najważniejsze).

Pozdrawiam

Ten post edytował likemandrake 28.08.2008, 22:15:02