[PHP]Złe odczytywanie "ifów" Opcje

[JustHuman4]

Witam. Mam mały problem z ifami. Otóż tworzę rejestrację i mam takie oto pola:
Nick
E-mail
Hasło
Powtórz hasło
Mam ifa sprawdzającego czy wszystkie pola zostały uzupełnione oraz ifa sprawdzającego czy podane hasła są takie same. Tak to wygląda:

[PHP] pobierz, plaintext 
//po wciśnięciu przycisku "rejestruj"
if(isset($_POST['submit']))
{
        $nick = $_POST['nick'];
        $mail = $_POST['mail'];
        $pass = $_POST['pass'];
 
 
	if(empty($nick) || empty($mail) || empty($pass)) echo 'Nie wszystkie pola zostały uzupełnione.';
	if($_POST['pass'] != $_POST['pass2'])
{
 die('Podane hasła nie są takie same.');
}
else
{
//dalsza część kodu
[PHP] pobierz, plaintext 

Ładnie sprawdza czy hasła są takie same czy też nie, jednak problem z tym, że nie trzeba wcale uzupełniać wszystkich pól. Czyli np po uzupełnieniu pól nick i e-mail wyświetla się komunikat
"Nie wszystkie pola zostały uzupełnione." (czyli dobrze) ale od razu po tym wyświetla się komunikat "Gratulacje! Utworzyłeś konto ........"
O co tu chodzi?

Z góry dziękuję za każdą pomoc
Pozdrawiam


Ten post edytował JustHuman4 27.12.2011, 11:43:18

[dami95]

[PHP] pobierz, plaintext 
 die('Podane hasła nie są takie same.');
[PHP] pobierz, plaintext 

Ten kod zakańcza dalszą interpretację skryptu.

Ja zawszę sobie sprawdzam wszystkie elementy formularza i zależnie czy są błędy czy nie ma zmieniam zmienną $errors odpowiedio na wartość 1 lub 0.
W twoim przypadku zrobił bym tak:

[PHP] pobierz, plaintext 
if(isset($_POST['submit'])) {
	$nick = $_POST['nick'];
	$mail = $_POST['mail'];
	$pass = $_POST['pass'];
 
	$errors = 0;
 
	if(empty($nick) || empty($mail) || empty($pass)) {
		echo 'Nie wszystkie pola zostały uzupełnione.';
		$errors = 1;
	} else {
		if($_POST['pass'] != $_POST['pass2']) {
			echo 'Podane hasła nie są takie same.';
			$errors = 1;
		}
	}
}
 
if ($errors) {
	//dodajemy do bazy czy coś tam
}
[PHP] pobierz, plaintext 

Ten post edytował dami95 27.12.2011, 19:37:02

[JustHuman4]

[PHP] pobierz, plaintext 
 die('Podane hasła nie są takie same.');
[PHP] pobierz, plaintext 

Ten kod zakańcza dalszą interpretację skryptu.

Ja zawszę sobie sprawdzam wszystkie elementy formularza i zależnie czy są błędy czy nie ma zmieniam zmienną $errors odpowiedio na wartość 1 lub 0.
W twoim przypadku zrobił bym tak:

[PHP] pobierz, plaintext 
if(isset($_POST['submit'])) {
	$nick = $_POST['nick'];
	$mail = $_POST['mail'];
	$pass = $_POST['pass'];
 
	$errors = 0;
 
	if(empty($nick) || empty($mail) || empty($pass)) {
		echo 'Nie wszystkie pola zostały uzupełnione.';
		$errors = 1;
	} else {
		if($_POST['pass'] != $_POST['pass2']) {
			echo 'Podane hasła nie są takie same.';
			$errors = 1;
		}
	}
}
 
if ($errors) {
	//dodajemy do bazy czy coś tam
}
[PHP] pobierz, plaintext 

Wykrywa błąd w linijce

[PHP] pobierz, plaintext 
$errors = 1;
[PHP] pobierz, plaintext 

za

[PHP] pobierz, plaintext 
echo 'Nie wszystkie pola zostały uzupełnione.';
[PHP] pobierz, plaintext 

Pozdrawiam i dzięki za odpowiedź