[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

[Shili]

If sprawdzający wypełnialność pól ma tylko i wyłącznie taki zasięg.

[PHP] pobierz, plaintext 
if(empty($nick) || empty($mail) || empty($pass)) echo 'Nie wszystkie pola zostały uzupełnione.';
[PHP] pobierz, plaintext 

Podpowiedź: cały (CAŁY) kod pod nim się wykona. Nie ma bloku else, w którym wykonują się instrukcje jeśli pola zostaną wypełnione - instruke pod ifem wykonają się więc zawsze.

[--kaem--]

Bo drugi if wykonuje się niezależnie od wyniku pierwszego ifa. W pierwszym ifie jeśli jest jakieś pole puste wyświetla odpowiedni komunikat, ale potem wykonuję się drugi warunek, w którym jeśli hasła są takie same wykona się to co jest w else.

[JustHuman4]

Problem jakby rozwiązany, zamieniłem po prostu kolejność ifów, Sprawdza i to i to, jest ładnie, tylko że kiedy nie podam takich samych haseł, usuwa mi się zawartość strony. Chodzi o to o czym powiedział @Shili ? Usunęło mi się kilka divów które znajdowały się na dole kodu strony. Jak temu zapobiec?

Pozdrawiam

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

[kadlub]

zacznij to analizować od środka

[PHP] pobierz, plaintext 
if(isset($_POST['submit'])) {
	$nick = $_POST['nick'];
	$mail = $_POST['mail'];
	$pass = $_POST['pass'];
 
 
 
	if(!empty($nick) || !empty($mail) || !empty($pass)) {
 
 
		if($_POST['pass'] == $_POST['pass2']) {
			tutaj kod jak wszystkie warunki są spełnione
		}
else {
echo'hasła się nie zgadzają';
	}
}
else{
echo'nie wypełniłeś wszystkich pól';
 
}
}
 
[PHP] pobierz, plaintext 

Ten post edytował kadlub 28.12.2011, 22:44:32

[JustHuman4]

To co napisał kadlub jest w ogóle dziwne. Pomoże ktoś jeszcze z tym co napisał dami95?? Dlaczego wykrywa tam błąd?

Pozdrawiam

[Turson]

Nie jest dziwne, tylko najprostszym sposobem. I poprawnie

[JustHuman4]

Nie jest dziwne, tylko najprostszym sposobem. I poprawnie

Mam w takim razie taki kod:

[PHP] pobierz, plaintext 
if(isset($_POST['submit']))
{
        $nick = $_POST['nick'];
        $mail = $_POST['mail'];
        $pass = $_POST['pass'];
 
 
	if($_POST['pass'] != $_POST['pass2'])
{
 die('Podane hasła nie są takie same.');
}
	if(empty($nick) || empty($mail) || empty($pass)) echo 'Nie wszystkie pola zostały uzupełnione.';
else
{
		$nick = trim(strip_tags(mysql_real_escape_string(HTMLSpecialChars($nick))));
		$pass = trim(strip_tags(mysql_real_escape_string(HTMLSpecialChars($pass))));
 
		$result = mysql_query("SELECT * FROM users WHERE nick='$nick'");
 
		if(mysql_num_rows($result)!=0) echo 'Istnieje już konto z takim loginem.';
		else
		{
			$data = time();
 
			$pass = md5($pass);
 
			$query = "INSERT INTO `users` (`nick` , `pass`, `mail` , `data_rejestracji`) VALUES ('$nick', '$pass', '$mail' , '$data')";
 
			if(mysql_query($query)) echo 'Gratulacje! Zostałeś poprawnie zarejestrowany. Już teraz możesz się <a href="index.php">zalogować</a>.';
		}
	}
}
[PHP] pobierz, plaintext 

I gdzie to mam wstawić bo nie za bardzo wiem?

Pozdrawiam i dzięki za każdą pomoc

[Fifi209]

Jejku, użyj jakiegoś PDO bo na trimy, strip_tags etc patrzeć się nie da

W ogóle czemu HASŁO PRZECHOWUJESZ W POSTACI JAWNEJ?!

[JustHuman4]

Fifi, daruj sobie tutaj takie rzeczy. Problem rozwiązany, temat do zamknięcia

Pozdrawiam i dzięki za nakierowanie