[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

[markonix]

Napisz to od nowa bo jeżeli to nie robisz tylko dla treningu to userbility tego kuleje.
Czemu?
Walisz całą weryfikacje do jednego warunku - czy ktoś wpiszę zły e-mail albo za krótkie hasło to będzie mu się wyświetlał ten sam błąd.
Jak lepiej?
Pomijając rzucanie wyjątków itp. najprostszy sposób to tablica $errors i taki system:

[PHP] pobierz, plaintext 
if ($strlen_haslo < 6)
$errors[] = 'Zbyt krótkie hasło';
if (KOLEJNY WARUNEK)
$errors[] = 'KOLEJNY BŁĄD';
 
if (count($errors))
// wyświetl błędy
else
// dokonaj rejestracji
[PHP] pobierz, plaintext 

Potem count i foreach, którym wyświetlisz wszystkie błędy (np. w akapitach z klasą error).

Do walidacji e-mail stosuj filter_var, a nie wyrażeń regularnych.

Ten post edytował markonix 9.04.2012, 18:09:18