Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> Problem z kombinacją zmiennej.
w0jt3k
post
Post #1





Grupa: Zarejestrowani
Postów: 27
Pomógł: 0
Dołączył: 29.09.2013

Ostrzeżenie: (0%)
-----


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

  1. $nazwa_uz = addslashes((string)trim($_POST['nazwa_uz']));


Czy może powinno to wyglądać tak?
  1. $nazwa_uz = addslashes($nazwa_uz);
  2. }
  3. $nazwa_uz = (string)$_POST['nazwa_uz'];
  4. $nazwa_uz = trim($_POST['nazwa_uz']);


Albo jeszcze inaczej wymyśliłem:
  1. $nazwa_uz = addslashes($_POST['nazwa_uz']);
  2. $nazwa_uz = addslashes($nazwa_uz);
  3. $nazwa_uz = (string)$_POST['nazwa_uz'];
  4. $nazwa_uz = (string)$nazwa_uz;
  5. $nazwa_uz = trim($_POST['nazwa_uz']);
  6. $nazwa_uz = trim($nazwa_uz);
  7. }
  8. else
  9. {
  10. $nazwa_uz = (string)$_POST['nazwa_uz'];
  11. $nazwa_uz = trim($_POST['nazwa_uz']);
  12. $nazwa_uz = trim($nazwa_uz);
  13. }


Czy to jest poprawne?

Ten post edytował w0jt3k 30.09.2013, 20:43:45
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
em1X
post
Post #2





Grupa: Zarejestrowani
Postów: 984
Pomógł: 41
Dołączył: 16.03.2002
Skąd: Płock

Ostrzeżenie: (0%)
-----


Nie trzeba zakładać 50 kondomów, jeden wystarczy.
  1. $id=(int)$_GET['id'];

zostanie zawsze na liczbę zamieniony, co by w tym nie było. Jaśniej chyba nie można? Nie trzeba (do liczb) więcej "zabezpieczeń" jak to nazwałeś.
Do zmiennych zawierających tekst użyj tylko htmlspecialchars, więcej też Ci nie potrzeba.

Powinieneś PDO używać, nie potrafisz i twierdzisz, że Cię denerwuje. Jakich funkcji Ci brakuje?
Go to the top of the page
+Quote Post

Posty w temacie


Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 27.12.2025 - 08:13