[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

[!*!]

Trudno żeby działało, skoro generujesz md5 a sprawdzasz czy ma 14 znaków. Poza tym źle się do tego zabrałeś. Nie sprawdzasz czy formularz został wysłany, skoro używasz md5 to sprawdzanie hasła jest zbędne, no i nie sądzę że używanie pregów do sprawdzania długości ciągu jest dobre. Nie sprawdzasz też czy pola istnieją oraz jakiego są typu. I nie przepuści Ci wszytkich adresów email, chociaż są poprawne.

Ten post edytował !*! 9.04.2012, 17:26:21