[PHP][MYSQL]Rejestracja... Opcje

[-Guards321-]

Witam, mam skrypt rejestracji, ale nie wiem jak zrobić w przypadku, gdy dany login istnieje już w bazie żeby to sprawdziło i jeżeli istnieje wywaliło komunikat o tym. Z góry dzięki.

Kod

<?php

$login = ($HTTP_POST_VARS['login']);
$pass = ($HTTP_POST_VARS['haslo']);
$pass2 = ($HTTP_POST_VARS['haslo2']);
$email = ($HTTP_POST_VARS['email']);


if($pass != $pass2)
{
$blad = TRUE;

echo("Hasla nie zgadzaja sie !");
}

if(empty ($_POST['email']))
{
$blad = TRUE;

echo("Nie podales e-maila ! ");
}

if(empty ($_POST['haslo']))
{
$blad = TRUE;

echo("Nie podales hasla ! ");
}


if(empty ($_POST['login']))
{
$blad = TRUE;

echo("Nie podales nazwy uzytkownika ! ");
}

elseif($blad != TRUE)

{



$connection = mysql_connect ("localhost", "root", "krasnal") or die("Nie moge sie polaczyc z baza danych");

mysql_select_db("logowanie", $connection) or die("Nie moge wybrać bazy danych");

mysql_query("insert into users VALUES ('', '$login', '$pass', '$email')", $connection) or die("Nie moge wpisac rekordow do tabeli");

print "SIE UDALO SIE";
}

?>

[wookieb]

[PHP] pobierz, plaintext 
<?php
 
$login = ($HTTP_POST_VARS['login']);
$pass = ($HTTP_POST_VARS['haslo']);
$pass2 = ($HTTP_POST_VARS['haslo2']);
$email = ($HTTP_POST_VARS['email']);
 
 
if($pass != $pass2)
{
$blad = TRUE;
 
echo("Hasla nie zgadzaja sie !");
}
 
if(empty ($_POST['email']))
{
$blad = TRUE;
 
echo("Nie podales e-maila ! ");
}
 
if(empty ($_POST['haslo']))
{
$blad = TRUE;
 
echo("Nie podales hasla ! ");
}
 
 
if(empty ($_POST['login']))
{
$blad = TRUE;
 
echo("Nie podales nazwy uzytkownika ! ");
}
 
elseif($blad != TRUE)
 
{
 
 
 
$connection = mysql_connect ("localhost", "root", "krasnal") or die("Nie moge sie polaczyc z baza danych");
 
mysql_select_db("logowanie", $connection) or die("Nie moge wybrać bazy danych");
$wynik=mysql_query('SELECT login FROM users WHERE login="'.$login.'"');
if(mysql_num_rows($wynik))
{
echo 'Taki login juz istnieje';
}
else
{
mysql_query("insert into users VALUES ('', '$login', '$pass', '$email')", $connection) or die("Nie moge wpisac rekordow do tabeli");
 
print "SIE UDALO SIE";
}
}
 
?>
[PHP] pobierz, plaintext 

[Heweliusz]

Zamiast

[PHP] pobierz, plaintext 
<?php
$login = ($HTTP_POST_VARS['login']);
$pass = ($HTTP_POST_VARS['haslo']);
$pass2 = ($HTTP_POST_VARS['haslo2']);
$email = ($HTTP_POST_VARS['email']);
?>
[PHP] pobierz, plaintext 

powinieneś raczej używać:

[PHP] pobierz, plaintext 
<?php
$login = strip_tags($_POST['login']);
$pass = strip_tags($_POST['haslo']);
$pass2 = strip_tags($_POST['haslo2']);
$email = strip_tags($_POST['email']);
?>
[PHP] pobierz, plaintext 

Trzeba przecież filtrować, to co się odstaje z zewnątrz, a używanie $HTTP_POST_VARS to przeżytek.

[-Guards321-]

Dzieki bardzo, działa bardzo dobrze

[-Guards321-]

Masz rację Heweliusz, dzięki za podpowiedź.