Forum PHP.pl

Witam. Postanowiłem sprawdzić swoje siły przy skrypcie rejestracji.

Otóż moje postanowienia są takie:
4 pola:
- Login
- Hasło
- Powtórz hasło
- Email

*Sprawdzanie czy pola nie są puste. (działa)

*Sprawdzanie czy hasła są identyczne. (działa)

*W przypadku poprawnej rejestracji wyświetla "Zarejestrowano poprawnie" nad formularzem. (nie działa)

*W przypadku niepowodzenia wyświetla "Błąd. Proszę spróbować za kilka sekund" nad formularzem. (nie działa)

*No i oczywiście jeżeli wszystkie pola są wypełnione dodaj rekordy do bazy. (nie działa)

Szyfrowanie hasła dodam później (IMG:style_emoticons/default/smile.gif) Tak samo sprawdzanie czy login i hasło już istnieje w bazie.

Kod rejestracji:

[PHP] pobierz, plaintext 
<?php
 
include('../conf.php');
 
$login = $_POST['login'];
$haslo = $_POST['haslo'];
$haslo2 = $_POST['haslo2'];
$email = $_POST['email'];
$dodaj = $_POST['dodaj'];
$zero = "";
$jeden = "1";
 
		if ($dodaj === $jeden) {
 
			if ($login === $zero) {
				$no_login = '<font color="red">Podaj login!</font>';
			}
 
			if ($haslo === $zero) {
				$no_haslo = '<font color="red">Podaj Hasło!</font>';
			}
 
			if ($email === $zero) {
				$no_email = '<font color="red">Podaj Email!</font>';
			}
 
			if ($haslo !== ($haslo2)) {
				$rozne = '<font color="red">Hasła się różnią!</font>';
			}
 
			if ($login !== $zero && $haslo !== $zero && $email !== $zero) {
				$zapytanie = "INSERT INTO `users` (`user_id`, `login`, `haslo`, `email`) VALUES ('', '$login', '$haslo', '$email')";
				$idzapytania = mysql_query($zapytanie);
			}
 
			if ($idzapytania) {
				$dodano = '<font color="green">Zarejestrowano poprawnie!</font>';
			}
				else {
					if (!isset($no_login) && ($no_haslo) && ($no_email)) {
						$nie_dodano = '<font color="red">Błąd! Spróbuj ponownie za kilka sekund :)';
					}
				}
		}
		?>
<!--/////////////////////////////////////////////////////////-->
<table>
	<form action="register.php" method="post">
	<tr>
		<td>
			<?php
				if (isset($dodano)) {
				echo $dodano;
				}
 
				if (isset($nie_dodano)) {
				echo $nie_dodano;
				}
			?>
		</td>
	</tr>
	<tr>
		<td>
			LOGIN: 
		</td>
		<td>
			<input type="text" name="login" />
		</td>
		<?php
			if (isset($no_login)) {
				echo '<td>';
				echo $no_login;
				echo '</td>';
			}
 
		?>
	</tr>
<!--////////////////////////////////////////////////////////-->
	<tr>
		<td>
			HASŁO: 
		</td>
		<td>
			<input type="password" name="haslo" />
		</td>
		<?php
			if (isset($no_haslo)) {
				echo '<td>';
				echo $no_haslo;
				echo '</td>';
			}
 
			if (isset($rozne)) {
				echo '<td>';
				echo $rozne;
				echo '</td>';
			}
		?>
	</tr>
<!--/////////////////////////////////////////////////////////-->
	<tr>
		<td>
			POWTÓRZ HASŁO: 
		</td>
		<td>
			<input type="password" name="haslo2" />
		</td>		
	</tr>
<!--/////////////////////////////////////////////////////////-->
	<tr>
		<td>
			EMAIL: 
		</td>
		<td>
			<input type="text" name="email" />
			<input type="hidden" name="dodaj" value="1" />
		</td>
		<?php
			if (isset($no_email)) {
				echo '<td>';
				echo $no_email;
				echo '</td>';
			}
		?>
	</tr>
	<tr><td><button type="submit">Rejestruj</button></td></tr>
		</form>
</table>
[PHP] pobierz, plaintext 

Podejrzewam że mam pomieszaną kolejność IF'ów i coś się żre z czymś, jednak nie mogę rozwikłać co takiego (IMG:style_emoticons/default/smile.gif)
Pomożecie?

EDIT:

Doszedłem do przyczyny (być może)

W kodzie jest tak: Jeżeli zmienne login, haslo i email nie są puste to zapisuje rekordy do bazy...

Jednak sprawdziłem połączenie z bazą i mam błąd:

Can't connect to local MySQL server through socket '/var/run/mysqld/mysqld.sock' (2)

Nie korzystam przy połączeniu z "localhost" tylko z "mysql3.ph-hos.osemka.pl" (baza na darmowym hostingu friko.pl)

Użytkownik i hasło do bazy sprawdzone 100 razy.

Czytales cos o operatorze "===" ? Prawda jeśli $a jest równe $b i są tego samego typu. (tylko PHP4) . Stosuj tylko == . Nie kapuje po co ci pusta zmienna . Mozeesz poprostu porwnac do zera NULL czy cos ala (IMG:style_emoticons/default/smile.gif) .

Tutaj

[PHP] pobierz, plaintext 
if (!isset($no_login) && ($no_haslo) && ($no_email))
[PHP] pobierz, plaintext 

isset tylko sie tyczy do no_login . W pierwszym zapytaniu daj sobie poprostu czy istnieje $dodaj (IMG:style_emoticons/default/smile.gif) . Glowny problem to pewnie jest to ze uzywasz PHP 5 a te tutaj operatory sa na PHP 4 .

Gotowego kodu ci nie podam (IMG:style_emoticons/default/smile.gif) Podalem ci tylko wskazowki . Jezeli masz jeszcze jakies pytania pisz .

Pozdro

PS: Sprawdz jaka wersje masz php za pomoca phpinfo();

Ten post edytował deadwheel 3.09.2012, 21:19:18

Ja się zastanawiam czy dałoby się uprościć tą rejestrację ze sprawdzaniem i nie lepiej byłoby pisać po if jeszcze elseif albo else jakby coś się źle zrobiło to wtedy można łatwiej wyczaić gdzie się popełniło błąd, mnie nie raz uratowało to że dopisywałem else do warunku jak coś było w do zrobienia, no i szybciej szło poprawianie. Jak jesteś początkujący i uczysz się dopiero php z kursu to sprawdź czy kurs jest dokładnie pod php5, jak wspomniał deadwheel o operatorach pod php4 to zastanawiam się,czy przerabia przestarzały kursu php(choć niewiele się różni ale różnice są (IMG:style_emoticons/default/tongue.gif) ).

Dzięki wielkie za naprowadzenie (IMG:style_emoticons/default/smile.gif) Napisałem od nowa połączenie z bazą, zmieniłem to co poradziłeś i poprawiłem jeszcze:

[PHP] pobierz, plaintext 
if (!empty ($login) && ($haslo) && ($email)) {
				if ($haslo == $haslo2) {
					$zapytanie = "INSERT INTO `users` (`id`, `login`, `haslo`, `email`) VALUES ('', '$login', '$haslo', '$email')";
					$idzapytania = mysql_query($zapytanie);
				}
			}
[PHP] pobierz, plaintext 

Ponieważ przed dodaniem rekordów nie sprawdzał czy hasła są takie same i dodawał nie zważywszy na to a jednocześnie wyświetlał że hasła są różne (IMG:style_emoticons/default/smile.gif)

Jeszcze raz dzięki.
Pozdro.

@olszam

Ten kod napisałem "z głowy" (IMG:style_emoticons/default/smile.gif) Oczywiście z niewielką pomocą kursów lub wcześniejszych prób w celu sprawdzenia składni lub czegoś w tym rodzaju (IMG:style_emoticons/default/tongue.gif) Jak każdy początkujący popełniam błędy, i jak to się mówi: "Człowiek uczy się na błędach", wychodzi na to że ja uczę się bardzo dużo ;D

spoko spoko każdemu się zdarza, nie każdy jest alfą i omegą. Czasem bardzo proste rzeczy mogą nam zawracać głowę i zastanawiamy się "czemu nie działa, co ja źle zrobiłem" (IMG:style_emoticons/default/tongue.gif) , ile to ja takich wpadek miałem.