[MySQL][PHP]problem z walidacją adresu email Opcje

[imysz]

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 

[MateuszS]

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

Ten post edytował MateuszScirka 18.02.2010, 18:36:31