Problem z kombinacją zmiennej. Opcje

[w0jt3k]

Czy taka kombinacja zmiennej jest poprawna i zostanie zinterpretowana przez serwer poprawnie? Głównie chodzi o ochronę przed SQLInjection.

[PHP] pobierz, plaintext 
$nazwa_uz = addslashes((string)trim($_POST['nazwa_uz']));
[PHP] pobierz, plaintext 

Czy może powinno to wyglądać tak?

[PHP] pobierz, plaintext 
if(!get_magic_quotes_gpc()){
	$nazwa_uz = addslashes($nazwa_uz);
}
$nazwa_uz = (string)$_POST['nazwa_uz'];
$nazwa_uz = trim($_POST['nazwa_uz']);
[PHP] pobierz, plaintext 

Albo jeszcze inaczej wymyśliłem:

[PHP] pobierz, plaintext 
if(!get_magic_quotes_gpc()){
	$nazwa_uz = addslashes($_POST['nazwa_uz']);
	$nazwa_uz = addslashes($nazwa_uz);
	$nazwa_uz = (string)$_POST['nazwa_uz'];
	$nazwa_uz = (string)$nazwa_uz;
	$nazwa_uz = trim($_POST['nazwa_uz']);
	$nazwa_uz = trim($nazwa_uz);
}
else
{
	$nazwa_uz = (string)$_POST['nazwa_uz'];
	$nazwa_uz = trim($_POST['nazwa_uz']);
	$nazwa_uz = trim($nazwa_uz);
}
[PHP] pobierz, plaintext 

Czy to jest poprawne?

Ten post edytował w0jt3k 30.09.2013, 20:43:45

[nospor]

Zrezygnowałem z PDO, bo mnie dosłownie wkur... to, ze nie ma zaimplementowanych wielu funkcji z Mysqli.

Sam sobie robisz krzywde, tylko dlatego ze przywykles do dwoch czy trzech funkcji, ktorych odpowiednikow nie potrafisz znalezc....

Niby kolega wyżej mówił, że addslashes nie potrzeba, przy int, no, ale... na wszelki wypadek:

A do szkoly/pracy tez chodzisz z pontonem na wszelki wypadek powodzi? (IMG:style_emoticons/default/wink.gif)

Int to int, wystarczy ze zrobisz:
$zm = (int)$zm;

i juz.

Z twojego kodu wynika, ze haslo i email to inty? Ktos z nas czegos tu nie rozumie (IMG:style_emoticons/default/wink.gif)
Z twojego kodu wynika rowniez, ze dane zaczynasz zabezpieczac juz dawno po tym jak wykonales zapytanie, ktore korzysta z tych danych.... Nie sadzisz ze to odrobine za pozno?

I jeszcze jedno pytanie. Jak zabezpieczyć pusty VALUE id_uz

A co ty tu chcesz zabezpieczac? Przeciez to ty wkladasz te wartosc. Jak bedziesz sam w kodzie wstawial wartosc 2, to tez bedziesz chcial ja zabezpieczac? Znowu na wszelki wypadek? (IMG:style_emoticons/default/wink.gif)

Poza tym dobrą praktyką jest wstawianie NULL a nie ''

No i czemu po real_escape_string robisz znowu addslashes?? Przeciez podwojnie zaczynasz slashowac wszystko..... Naprawde, im wiecej, nieznaczy lepiej... W tym wypadku znaczy gorzej.