Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> rejestracja problem
peter13135
post
Post #1





Grupa: Zarejestrowani
Postów: 1 447
Pomógł: 191
Dołączył: 26.03.2008

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


jak pewnie niektórzy wiedzą, ostatnio zadaje różne głupie pytania związane z php, ponieważ robie skrypt php ;] no i wykryłem błąd w swoim skrypcie który dla mnie jest dziwny, i niewiem jak go naprawić

wygląda on w skróconej formie tak

dodam że funkcja zabezpiecz, to takie coś co mi filtruje przez htmlspecialchars oraz mysql_real_escape_string (czy coś takiego)


  1. <?php
  2. [tu jest formularz]
  3. $zarejestruj=$_POST['zarejestruj'];
  4. $login = zabezpiecz($_POST['login']);
  5. $haslo = zabezpiecz($_POST['haslo']);
  6. $email = zabezpiecz($_POST['email']);
  7.  
  8.  
  9. if($email)
  10. {
  11. if(checkEmail($email)==false) {$reg_error="email nieprawidłowy";}
  12. }
  13.  
  14.  
  15. if(isset($login) && isset($haslo) && isset($email) && isset($zarejestruj) && empty($reg_error) )
  16.  
  17. //wysyłanie
  18.  
  19. else if(($zarejestruj)&&(!$login || !$haslo || !$email ||!$dataurodzenia ) ) 
  20. wywalanie błędu
  21. ?>




problem w tym że żeby sie zarejestrować nietrzeba wpisywać maila, ocb?
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
pyro
post
Post #2





Grupa: Zarejestrowani
Postów: 2 148
Pomógł: 230
Dołączył: 26.03.2008

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


  1. <?php
  2. $email = zabezpiecz($_POST['email']);
  3. ?>


  1. <?php
  2. isset($email)
  3. ?>


sprawdzasz czy zmienna jest przpisana za pomoca isset, i jest, ma wartość pustą ale jest, jak chcesz sprawdzic czy nie jest '' użyj !empty();, poza tym robisz wszystko w złej kolejności. przypisujesz krotkie zmienne, potem sprawdzasz checkmailem maila, a potem czy jest taka zmienna co jest bez sensu.
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: 3.10.2025 - 19:18