Jezeli id nie istnieje to error Opcje

[Rafał 40x]

Korzystam z takiego zapytania:

[PHP] pobierz, plaintext 
<?php
$result = mysql_query (" Select * From `users` Where `active` = 1 and `id` = '".$_GET['id']."' ");
?>
[PHP] pobierz, plaintext 

Link mam taki: users.php?id_user=1

Np. w bazie mam 200 userow... Jak zrobic jezeli ktos wpisze np. users.php?id_user=243 albo users.php?id_user=1422... czyli jak wpisze id usera ktory nie istnieje to wyskoczy blad?

Dostalem chyba jakiejs niewiedzy poniewaz nie moge wykombinowac jak to zrobic (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Prosze o pomoc.

[Cysiaczek]

[PHP] pobierz, plaintext 
<?php
$result = mysql_query (" Select * From `users` Where `active` = 1 and `id` = '".$_GET['id']."' ") or die ("Błąd - nie ma takiego użytkownika . Tyle");
?>
[PHP] pobierz, plaintext 

Pozdrawiam,

[Rafał 40x]

Nie dziala to :/ Nic nie wyswietla.

A po drugie to ja mam cos takiego (przepraszam ze dopiero teraz to pisze):

[PHP] pobierz, plaintext 
<?php
if ( isset ( $_GET['id'] ) && is_numeric ( $_GET['id'] ) ) {
// wszystko jest ok
} else {
// nie jest ok
}
?>
[PHP] pobierz, plaintext 

i chcialbym do warunku if dodaj to jezeli id istnieje to jest ok a jezeli nie to nie jest ok

Rozumiesz teraz (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Pozdrawiam.

[skowron-line]

to sobie najpierw policz rekordy w bazie i porownaj z liczba z $_GET-a i jak sie nie bedzie zgadac to wywal blad

[Rafał 40x]

cos w tym stylu?

[PHP] pobierz, plaintext 
<?php
$num = mysql_num_rows ($result);
if ( $num != $_GET['id'] ) {
// user nie istnieje
}
?>
[PHP] pobierz, plaintext 

[Cysiaczek]

To, że sprawdzasz, czy jest ustawiona $id, to nie znaczy, ze jest ona w bazie. musiasz wykonać zapytanie do bazy, alby się o tym przekonać. Jednak, skoro chcesz po prostu sprawdzić, czy uzytkownik o danym id jest zarejestrowany, to sprawdź za pomoca funcji mysql_num_rows(), czy sa jakiekolwiek wyniki zapytania, które podajesz.

[PHP] pobierz, plaintext 
<?php
if (mysql_num_rows($result)!==1){
print "Błąd - nie ma użytkownika";
}
?>
[PHP] pobierz, plaintext 

[Rafał 40x]

Nie działa (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

Mam tak:

[PHP] pobierz, plaintext 
<?php
$res = mysql_query (" Select * From `users` Where `active` = '1' ");
 
if ( isset ( $_GET['id'] ) && is_numeric ( $_GET['id'] ) && mysql_num_rows ( $res )!== 1 ) {
$result = mysql_query (" Select * From `users` Where `active` = '1' and `id` = '".$_GET['id']."' ");
 
$data = mysql_fetch_array ( $result );
 
// tutaj dane usera
 
} else {
 
	$content .= 'Taki user nie istnieje.';
 
}
 
?>
[PHP] pobierz, plaintext 

[My4tic]

@Rafał 40x - Twoje rozwiązanie jest bardzo niebezpieczne. Pobierasz parametr przez URL i wstawiasz go bezpośrednio do zapytania. Powiedzmy, ze bede userem o id = 1, wystarczy ze wpisze w url id = 20 i dostane dane dostępne dla usera o id = 20. Najlepiej pobierz sobie id usera podczas jego logowania, ustaw to w sesji i w zapytaniu uzywaj danych z SESSION a nie GET. Chyba ze akurat w Twoim przypadku nie jest to istotne.

[Rafał 40x]

@Rafał 40x - Twoje rozwiązanie jest bardzo niebezpieczne. Pobierasz parametr przez URL i wstawiasz go bezpośrednio do zapytania. Powiedzmy, ze bede userem o id = 1, wystarczy ze wpisze w url id = 20 i dostane dane dostępne dla usera o id = 20. Najlepiej pobierz sobie id usera podczas jego logowania, ustaw to w sesji i w zapytaniu uzywaj danych z SESSION a nie GET. Chyba ze akurat w Twoim przypadku nie jest to istotne.

Nie mam rejestracji i logowania userow (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

A moge zrobic tak:

[PHP] pobierz, plaintext 
<?php
$id = $_GET['id'];
 
// i w zapytaniu dac
 
$id
?>
[PHP] pobierz, plaintext 

I czy ktos wie jak rozwiazac moj problem?

[phpion]

[PHP] pobierz, plaintext 
<?php
$res = mysql_query (" Select * From `users` Where `active` = '1' "); // po co to zapytanie?? nie wywalam bo moze ma byc...
 
if (isset ($_GET['id']) && is_numeric ($_GET['id']))
{
	$q = "SELECT * FROM users WHERE active='1' AND id='".$_GET['id']."'";
	$q = mysql_query($q);
 
	if (mysql_num_rows($q) > 0)
	{
		// user jest w bazie
		$data = mysql_fetch_array ($q);
		// tutaj dane usera
	}
	else
		$content .= 'Taki user nie istnieje.';
}
?>
[PHP] pobierz, plaintext 

Powinno dzialac.

[Rafał 40x]

Ok. Zacznijmy od nowa.

- korzystam ze Smarty;
- żaden z powyższych skryptów nie zadziałał;

Otóż dlaczego? Zaraz wyjaśnię (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Wypróbowałem jeszcze raz mój kod, który napisałem tak od ręki (można nawet powiedzieć, że nie wiedziałem co piszę. Oto on:

[PHP] pobierz, plaintext 
<?php
$_GET['id'] = ( int ) $id;
 
if ( mysql_num_rows ( $result ) != $id ) {
	$content .= "Taka user nie istnieje.";
} else {
	// dane usera
$smarty -> assign ("content", $content);
 
}
?>
[PHP] pobierz, plaintext 

Pewnie domyślacie się już w czym sęk. Assign`owałem wartość zmiennej $content jeżeli użytkownik istnieje. Jeżeli nie istniał tylko definiowałem zmienną, i skrypt jej nie parsował (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) To był błąd z mojej strony!

Teraz tylko zastanawiam się, który z w.w skryptów wybrać :] A może swój? (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Pozdrawiam i dziękuję.

[phpion]

Moj kod powinien dzialac, a w twoim jest byk jak nic:

[PHP] pobierz, plaintext 
<?php
if ( mysql_num_rows ( $result ) != $id )
?>
[PHP] pobierz, plaintext 

juz pomijam fakt ze nie widze tu zapytanie SQL ale sprawdzasz, czy ILOSC rekordow wybranych z bazy (przy pomyslnym wykonaniu zapytania powinna wynosic w tym wypadku 1) jest rowna przekazywanemu $id. Zadziala to tylko dla usera o id=1.
Jesli nie potrafisz wykorzystac mojego kodu, a bierzesz sie za smarty - to sorry...

[Rafał 40x]

Ok. Zwacam honor. Mój błąd. Wykorzystałem Twój kod i działa cacy (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[phpion]

No.
(IMG:http://forum.php.pl/style_emoticons/default/smile.gif) pozdrawiam (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)