[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]

Po co linia:

[PHP] pobierz, plaintext 
$password_rep = md5($password_rep);
[PHP] pobierz, plaintext 

?

Errory łączysz jako stringi ale propozycja z tablicą jest bardziej elastyczna.
Potem błędy możesz wyświetlić w dowolnej postaci (najpierw po enterach ale potem na pewno ktoś by wolał je wymienić od myślników albo jakieś divy).
Zamiast dawać wszędzie <br /> wystarczy implode.

Tym system sprawdzaj też czy pola są puste w takim schemacie.

[PHP] pobierz, plaintext 
if (CZY PUSTY)
$errors[] = 'Jest pusty';
else if (CZY ZA KRÓTKI)
$errors[] = 'Za krótki';
[PHP] pobierz, plaintext 

Dzięki temu nie wyświetlą się 2 błędy, które się pokrywają.

Żeby sprawdzić czy formularz został wysłany wystarczy isset na dowolnym polu z tego formularza ponieważ on zawsze daje true gdy formularz zostanie wysłany nawet gdy go nie wypełnimy. Ładniej można dać input z polem ukrytym.

[HTML] pobierz, plaintext 
<input type="hidden" name="action" value="add_user" />
[HTML] pobierz, plaintext 

i rozpocząć skrypt od sprawdzenia czy $_POST['action'] == 'add_user.

Ten post edytował markonix 9.04.2012, 20:56:36