[PHP]Skrypt rejestracji problem Opcje

[porywacz]

Witam, napisałem tak na szybko skrypt rejestracji na stronę. Jednak nie wiem czemu instrukcja wykonuje się nie tak jak chcę ;/
Wszystkie pola wypełniam zgodnie z kryteriami z preg_match a i tak wyskakuje mi komunikat z else'a ;///
Tutaj formularz w html:

[HTML] pobierz, plaintext 
Wypełnij poniższy formularz, aby się zarejestrować.<br/>
Pamiętaj, wypełniając formularz rejestracyjny wyrażasz zgodę na przetwarzanie twoich danych osobistych.
<table style="text-align: right; margin-left: 20px;">
<form method="post" action="zarejestruj.php">
<tr><td></td><td>* Pola obowiązkowe</td></tr>
<tr>
<td>Nazwa użytkownika*:</td> 	<td><input type="text" name="login"/></td>
</tr>
<tr>
<td>Hasło*: </td>				<td><input type="password" name="password"/></td>
</tr>
<tr>
<td>Powtórz hasło*: </td>		<td><input type="password" name="password_rep"></td>
</tr>
<tr>
<td>E-mail*: </td>				<td><input type="text" name="email"></td>
</tr>
<tr>
<td>Powtórz e-mail*: </td>				<td><input type="text" name="email_rep"></td>
</tr>
<tr>
<td></td>						<td><input type="submit" value="Zarejestruj się"></td>
</tr>
</form>
</table>
[HTML] pobierz, plaintext 

A tutaj skrypt rejestracji:

[PHP] pobierz, plaintext 
<?php
$login = $_POST['login'];
$password = $_POST['password'];
	$password = md5($password);
$password_rep = $_POST['password_rep'];
	$password_rep = md5($password_rep);
$email = $_POST['email'];
$email_rep = $_POST['email_rep'];
 
if(	!empty($login) 
	&& 
	!empty($password)
	&&
	!empty($password_rep)
	&& 
	!empty($email) 
	&& 
	!empty($email_rep)
	) 		{
	if(
		preg_match('@^[a-z0-9]{6,12}$@',$login)
		&&
		preg_match('!^[a-z0-9]{6,14}$!',$password)
		&&
		preg_match('!^[a-z0-9]{3,20}@[a-z0-9]{2,8}.[a-z]{2,5}$!',$email)) {
 
	$db = mysql_connect('localhost','root','') or die('Nie udało się połączyć z serwerem :(');
	$db = mysql_select_db('uzytkownicy');
 
	$query = mysql_query("INSERT INTO uzytkownicy (login, password, email) VALUES('$login','$password','$email')") or die('Nie udało się umieścić values w tabeli');
 
	} else {
		echo('Nie udało mi się Ciebie zarejestrować. <br/> Sprawdź czy poprawnie wypełniłeś wszystkie pola.');
	}
	} else {
		echo('Wypełnij wszystkie pola !');
	}
?>
[PHP] pobierz, plaintext 

Z góry dziękuję za pomoc (IMG:style_emoticons/default/smile.gif)

Ten post edytował porywacz 9.04.2012, 16:57:25

[porywacz]

Wiem, że nie jest tak dobrze, dlatego napisałem wszystko od nowa tak jak napisał markonix ^^
Tutaj skrypt:

[PHP] pobierz, plaintext 
<?php
$login = $_POST['login'];
$password = $_POST['password'];
$password_rep = $_POST['password_rep'];
$email = $_POST['email'];
$email_rep = $_POST['email_rep'];
 
			$db = mysql_connect('localhost','root','') or die('Nie udało się połączyć z serwerem :(');
			$db = mysql_select_db('uzytkownicy');
 
$sprawdz_login = mysql_fetch_array(mysql_query("select count(*) from uzytkownicy where login = '$login' limit 1"));
$sprawdz_email = mysql_fetch_array(mysql_query("select count(*) from uzytkownicy where email = '$email' limit 1"));
 
if(	!empty($login) 
	&& 
	!empty($password)
	&&
	!empty($password_rep)
	&& 
	!empty($email) 
	&& 
	!empty($email_rep)
	) 		{
		if($sprawdz_login[0] >= 1) {
		$errors .= 'Ta nazwa użytkownika jest już zajęta<br/>';}
		if($sprawdz_email[0] >= 1) {
		$errors .= 'Taki email jest już zajęty<br/>';}
		if(strlen($password) < 6) {
		$errors .= 'Twoje hasło jest za krótkie<br/>';}
		if(strlen($login) < 6){
		$errors .= 'Twój login jest za krótki<br/>';}
		if(strlen($login) > 14) {
		$errors .= 'Twój login jest za długi<br/>';}
		if(!preg_match('/[a-z]/',$login)){
		$errors .= 'Twój login musi zawierać litery<br/>';}
		if($password !== $password_rep) {
		$errors .= 'Hasła się nie zgadzają<br/>';}
		if($email !== $email_rep) {
		$errors .= 'E-maile się nie zgadzają<br/>';}
 
		if (count($errors)) {
			echo('Rejestracja nie powiodła się. Wystąpiły następujące błędy: <br/>'.$errors);
		} else {
			$password = md5($password);
			$password_rep = md5($password_rep);
			$query = mysql_query("INSERT INTO uzytkownicy (login, haslo, email) VALUES('$login','$password','$email')") or die('Nie udało się umieścić values w tabeli');	
		} 
} else {
	echo('Rejestracja nie powiodła się. Wystąpiły następujące błędy: <br/>Wypełnij wszystkie pola !');
} ?>
[PHP] pobierz, plaintext 

Może nie jest idealnie ale ważne, że działa. Akurat na tym chciałem się skupić, ponieważ jest to mój pierwszy skrypt rejestracji (IMG:style_emoticons/default/biggrin.gif)