Witam. Tworzę formularz rejestracji i mam problem z sprawdzeniem poprawności adresu email. by był on w formacie np. nazwa@gmail.com.
stosuję taki fragment do tego:

[PHP] pobierz, plaintext 
 function valid_email($email)
{
if (preg_match("/^([a-z0-9+_-]+)(.[a-z0-9+_-]+)
*@([a-z0-9-]+.)+[a-z]{2,6}$/ix", $email)) 
return true;
  else 
    return false;
}
 
if (!valid_email($email)) 
echo "poprawny";
else echo "Nie poprawy adres e-mail"; 
[PHP] pobierz, plaintext 

ale mimo wpisania niepoprawnego adresu(np. nazwa) rekord dodaje się do bazy

oto cały kod kod, poprawnie jest napisany?:

[PHP] pobierz, plaintext 
<?php
 
 
$login = $_POST['login'];
$haslo = $_POST['haslo'];
$haslo2 = $_POST['haslo2'];
$email =  $_POST['email'];
 
 
 
 
 $login = trim($login);
 $haslo = trim($haslo);
 $haslo2 = trim($haslo2);
 $email = trim($email);
 
 // sprawdzenie poprawnosci loginu
 
 if(!$login || !$haslo || !$haslo2 || !$email)
 {
 if(empty($login))
 {
 echo 'Pole login nie zostało wypełnione<br/>';
 }
 
  $db=  mysql_connect('localhost', 'db994313', 'pizgafurtke');
        mysql_query("SET NAMES 'utf8' COLLATE 'utf8_polish_ci'");                   
        mysql_select_db('db994313');
 
    $zapytanie = "select * from uzytkownicy where login='$login'";
    $wynik_login = mysql_query($zapytanie);
    $wynik_login_rezultat = mysql_num_rows($wynik_login);
    if($wynik_login_rezultat > 0)
        {
        echo 'Login jest już używany<br/>';
        }
 
if(strlen($login)<5)
    {
    echo 'Login powinien zawierać przynajmniej 5 znaków<br/>';
    }
 
        if(strlen($login) > 16)
        {   
        echo 'Login powinien zawierać maksynmalnie 16 znaków';    
        }
 
// sprawdzenie hasla
if(empty($haslo))
{
echo 'Pole haslo nie zostalo wypelnione<br/>';
}
if(empty($haslo2))
{
echo 'Pole z potwierdzeniem hasła nie zostało wypełnione<br/>';
}
if(strlen($haslo) < 4)
{
echo 'Hasło powinno zawierać przynajmniej 5 znaków<br/>';
}
if(strlen($haslo2) <5)
{
echo 'Hasło powinno zawierać przynajmniej 5 znaków<br/>';
}
if($haslo !=$haslo2)
{
echo 'Podane hasła różnią się do siebie<br/>';
}
 
// sprawdzenie adresu email
if(empty($email))
{
echo 'Pole email nie zostało wypełnione';
}
 
$zapytanie_email = "select * from uzytkownicy where email='$email'";
$wynik_email = mysql_query($zapytanie_email);
$wynik_email_rezultat = mysql_num_rows($wynik_email);
if($wynik_email_rezultat > 0)
{
echo 'Podany email jest już zarejestrowany<br/>';
}
 
 function valid_email($email)
{
if (preg_match("/^([a-z0-9+_-]+)(.[a-z0-9+_-]+)
*@([a-z0-9-]+.)+[a-z]{2,6}$/ix", $email)) 
return true;
  else 
    return false;
}
 
if (!valid_email($email)) 
echo "poprawny";
else echo "Nie poprawy adres e-mail"; 
 
}
else{
$zapytanie_dodaj = "insert into uzytkownicy values('".$login."', sha1('".$haslo."'), '".$email."')";
$wyslij_zapytanie = mysql_query($zapytanie_dodaj);
if($wyslij_zapytanie){
echo 'Rejestraja ukończona pomyślnie';
}
 
 
 
}
?>
[PHP] pobierz, plaintext 

Od tego są filtry: filter_var.

No dobra sprawdzasz te warunki extra, ale co z tego skoro skrypt sie dalej wykonuje, po kazdym warunku, np puste pole. Daj exit(); w kazdym warunku po echo zeby skrypt sie przerwal i nie wyk. zapytania

Blame

dzięki za pomoc, teraz działa.

MateuszScirka
Faktycznie, wtedy skrypt się zawsze wykonywał. Dodałem teraz na końcu każdego warunku exit; , ale mam jedno pytanie. bo przedtem jak np. cały formularz był pusty to po przyciśnięciu buttona zarejestruj to pokazywało np tak:

Pole login nie zostało wypełnione
Login powinien mieć więcej niż 5 znaków
hasła różnią się od siebie
itd. w zależności od tego które pola były źle wypełnione lub puste, tak wyświetlane były odpowiednie komunikaty.

a teraz po dodaniu exit; jeżeli zostawię pusty formularz i dam zarejestruj to pojawia się tylko komunikat o braku wpisania loginu a reszty w ogóle nie wyświetla. Idze jakoś ten skrypt udoskonalić by pokazywało wszystko co jest do poprawki ?

pozdrawiam

tak

Przed calym skryptem daj $i=0; a pod kazdym echo dodaj $i++; i wywal exity, potem przed zapytaniem daj warunek

[PHP] pobierz, plaintext 
if($i!=0) exit();
[PHP] pobierz, plaintext 

i ma smigac i wyswietlac bledy

MateuszScirka
Działa tak jak chciałem. Dziękuję.

teraz mam taką sytuację, że formularz jest w pliku register.php i w formularzu jest że ma się wykonać skrypt register_cnf.php w którym właśnie jest cały kod rejestrujący. Gdy coś było źle wypełnione w formularzu lub puste i gdy daję wstecz w przeglądarce by poprawić to formularz jest pusty, da się zrobić tak aby dane które były dobrze wpisane zostały tak jak były, nie kasowały się?

lub czy da się zrobić tak aby formularz i skrypt rejestrujący był w jednym pliku *php ?

pozdrawiam

Wszystko się da
Ja z reguły gdy nie korzystam z obiektówki robię pseudo-podział w jednym pliku dla formularzy na zasadzie.

[PHP] pobierz, plaintext 
<?php
$pola = array (
    'pole' => 'default'
);
$field_errors = array();
if( isset( $_POST ) AND !empty( $_POST ) ) {
    if( array_key_exists('save', $_POST) ) {
      //w zależności jaki typ pola dodaję czasem if( array_key_exists( 'pole', $_POST ) )
      //ale zawsze na bank jest to co poniżej, no chyba, że oczekuję 0, to wtedy inaczej ;)
      if( !empty( 'pole' ) ) {
        //tutaj jakaś dodatkowa walidacja itp
       $field['pole'] = $_POST['pole'] //ale $_POST['pole'] już po obróbce jest tu!
      } else {
        $field_errors['pole'] = 'Komunikat błędu';
      }
    }
    if( count( $field_errors ) == 0 ) {
      //tutaj zapis do bazy, ewentualne przekierowania itp bo znaczy, że wszystko OK
    } else {
      //wywalam na ekran, że są błedy formularza i jest on do poprawki
    }
}
?>
<html>
<head>
//sekcja head
</head>
<body>
<form method="post" action="">
    <input name="pole" value="<?php echo $field['pole'] ?>" />
    <input type="submit" name="save" value="zapisz" />
<?php
if( $field_errors['pole'] )
//wywalam komunikat pod polem go tyczącym :)
?>
</form>
</body>
</html>
[PHP] pobierz, plaintext 

To tak w telegraficznym skrócie! Resztę przemyśl...
EDIT: Formularz tego typu to fajny sposób jeśli chcesz sobie zrobić wstęp do obiektówki. Bo można to co podałem jeszcze nieco przerobić i sobie piszesz własną klasę do tworzenia i walidacji formularzy.