[php/mysql] Problem z walidacją w skrypcie rejestracji userów Opcje

[free]

Poprzez formularz przesylam dane do ponizszego skryptu :

[PHP] pobierz, plaintext 
<?php
case 'zarejestruj':
$spr="SELECT ksywa, email FROM users88 WHERE ksywa='".$_POST['ksywa']."' OR email='".$_POST['email']."'";
 $wynik=mysql_query($spr);
 if (mysql_num_rows($wynik) > 0 ) echo "<br /> Podana nazwa lub email juz istnieje w bazie danych <br />";
	  if (!$_POST['ksywa'])  echo "<br /> Nie wypełniono pola "Imię"<br />";
	  if (!$_POST['email'])  echo "<br /> Nie wypełniono pola "Email"<br />";
	  if (!$_POST['haslo'])  echo "<br /> Nie wypełniono pola "Hasło"<br />";
	  if (!$_POST['haslo2']) echo "<br /> Nie wypełniono pola "Hasło2"<br />";
	  if ($_POST['haslo']!=$_POST['haslo2'])  echo "<br /> Podane hasła nie sa identyczne. Podaj takie same hasła";
 
	 else {
$ip = $_SERVER['REMOTE_ADDR'];
$data = date('Y-m-d G:i:s');
 $dataip ="$data.$ip";
 $md5=md5($dataip);
$ksywa=htmlspecialchars($_POST['ksywa']);
$email=htmlspecialchars($_POST['email']);
$haslo=htmlspecialchars($_POST['haslo']);
$haslo2=htmlspecialchars($_POST['haslo2']);
 
		$sql = "INSERT INTO tempusers88 (tempuser_id, email, ksywa, haslo, ip, md5)
VALUES ('', '$email', '$ksywa', '$haslo', '$ip', '$md5')";
 
 
		mysql_query($sql)
		  or die('Nie potrafię utworzyć konta: ' . mysql_error());
?>
[PHP] pobierz, plaintext 

Działa dobrze, tzn rejestruje uzytkownikow. Ale zastosowana przezemnie walidacja /LINIE 5-10/ błędów nie sprawdza się, tzn skrypt rejestruje nawet gdy sie nie poda nazwy usera lub hasla :-( Co tu poprawic ?

Ten post edytował free 5.09.2006, 10:44:26

[nospor]

Podpowiedź:
zauwaz, ze twoj ELSE, ktory robi wpis do bazy odnosi sie tylko do ostatniego ifa. Pozostale ify wywalają tylko tekst na ekran i nie mają z ELSE nic wspolnego...

proszę poprawić tytuł o znacznik zgodnie z zasadami forum Przedszkole:
Temat: Tematyka i zasady panujace na forum Przedszkole

[My4tic]

Zamiast

[PHP] pobierz, plaintext 
<?php
if (!$_POST['haslo'])
?>
[PHP] pobierz, plaintext 

używaj

[PHP] pobierz, plaintext 
<?php
if (empty($_POST['haslo'])) {...}
?>
[PHP] pobierz, plaintext 

http://pl2.php.net/manual/pl/function.empty.php" title="Zobacz w manualu php" target="_manual

[free]

Nospor. Działało mi do tej pory jak nie dodałem opcji sprawdzajacej w linii 3,4,5 . Do tej pory warunki były sprawdzane i w razie nie spełnienia obojetnie jakiego warunku nie dochodzilo do dodania do bazy tylko komunikat lub komunikaty. a teraz wywala komunikaty o braku jakiegod parametru z $_POST ale dodaje do bazy niepelne dane.
My4tic słuszna uwaga.

[nospor]

@free to ze ci dzialala do tej pory, przed zmianami, to ma malo do znaczenia. Jak juz ci pisalem, wczesniejsze ify tylko wyswietlają napis na ekran, w zaden sposob nie warunkuja wlozenia do bazy. Wlozenie do bazy warunkuje tylko ostatni if.
Mozesz dorobic, ze kazdy if ustawia zmienna $blad na true. A do bazy beddziesz wkladaj tylko wtedy gdy zmienna $blad jest false.

[free]

Więc jak to zrobić ?
Poprawiłem na

[PHP] pobierz, plaintext 
<?php
if (empty($_POST['haslo'])) {...}
?>
[PHP] pobierz, plaintext 

ale nic nie zmienia, poza kosmetyką.
Testuje i widze ze problem lezy w dodanych liniach 3,4,5 bez nich skrypt bez jakiejs zmiennej zatrzymywał sie i bylo OK. A teraz tylko wyswietla komunikaty ale dodaje dane do bazy. Głupieje juz.

[nospor]

To ja ci mowie, ze jesli masz blad danych to masz ustawiac jakac zmienna, a ty mi dopisujesz {...} i sie dziwisz ze nic sie nie zmienia.... czy ja nie szprecham po polsku?

[PHP] pobierz, plaintext 
<?php
$blad = false;
if (jakiswaruneknazledane1) {echo 'zledane1';$blad=true;}
if (jakiswaruneknazledane2) {echo 'zledane2';$blad=true;}
//.....
if (!$blad) {
//tu se wstaw do bazy
}
?>
[PHP] pobierz, plaintext 

I przestan powtarzac uparcie gdzie jest blad, ze wlini 3,4,5 bo ja to doskonale wiem.

edit (po poscie ponizej):
wszystkie odpowiedzi byly logiczne... ostatnia to gotowiec....

[free]

Nospor ostatnia odpowiedz brzmi logicznie :-)
Sprawdziłem i śmiga jak KUBICA :-) O to mi chodziło.
Ostatecznie kod wygląda tak :

[PHP] pobierz, plaintext 
<?php
case 'zarejestruj':
$blad = false;
$spr="SELECT ksywa, email FROM users88 WHERE ksywa='".$_POST['ksywa']."' OR email='".$_POST['email']."'";
 $wynik=mysql_query($spr);
if (mysql_num_rows($wynik) > 0 ) {echo "<br />Podana nazwa lub email juz istnieje w bazie danych "; $blad=true; }
if (empty($_POST['ksywa']))  { echo "<br /> Nie wypełniono pola "Ksywa"<br />";  $blad=true; }
if (empty($_POST['email']))  { echo "<br /> Nie wypełniono pola "Email"<br />";   $blad=true; }
if (empty($_POST['haslo']))  { echo "<br /> Nie wypełniono pola "Hasło"<br />";   $blad=true; }
if (empty($_POST['haslo2'])) { echo "<br /> Nie wypełniono pola "Hasło2"<br />";  $blad=true; }
if ($_POST['haslo']!=$_POST['haslo2'])  { echo "<br /> Podane hasła nie sa identyczne. Podaj takie same hasła"; $blad=true; }
 if (!$blad) {
?>
[PHP] pobierz, plaintext 

Mała nie ścisłość powstaje jedynie gdy uzytkownik nie poda nic w polu ksywa i email. wowczas skrypt oprocz tekstu ze nie wypelniono tych pol podaje rowniez ze taki user lub email juz istnieje. Co zmeinic by to dopracować ?

Ten post edytował free 5.09.2006, 11:30:59

[nospor]

Tak na chlopski rozum to najpierw powinienes sprawdzac, czy sa dane prawidlowe, a dopiero potem selecta walic Ale to juz teraz pokombinuj sam. KOlejnego "logicznego" gotowca ci nie dam

[free]

Tak se teraz pomyslalem, ze pytajac o dane z bazy poprzez SELECT gdzy nie ma spełnionych warunkow to dostajemy wszystko, wiec chyba lepiej bedzie jak zamiast

[PHP] pobierz, plaintext 
<?php
if (mysql_num_rows($wynik) > 0 )
?>
[PHP] pobierz, plaintext 

uzyje

[PHP] pobierz, plaintext 
<?php
if (mysql_num_rows($wynik) ==1 )
?>
[PHP] pobierz, plaintext 

bo jak od poczatku tak bede rejestrowal userow to w przypadku duplikacji nazw bedzie tylko 1 user o danym niku lub mailu. I jezeli bedzie ten warunek spełniony to warunek if bedzi działał optymalniej.

Ten post edytował free 5.09.2006, 12:05:25

[My4tic]

Dziwnie troche do tego podchodzisz....

Nadaj atrybut UNIQUE w SQL do nick i mail, później...

1. Sprawdź czy wypełniono wymagane pola,
2. Sprawdź poprawnośc danych/usuń niebezpieczny kod,
3. Dodaj dane do SQL,
4. Jeśli zapytanie zwróci błąd to znaczy, że taki nick/mail istnieje (nie pamiętam teraz numeru błedu, poszukaj sobie),
4a. Jeśli bład - Wyświetl ponownie formularz.
4b. Jeśli brak błedu - Komunikat, że wszystko ok.

[free]

Wystarczy dodac :

[SQL] pobierz, plaintext 
`ksywa` varchar(100) UNIQUE NOT NULL DEFAULT '',
  `email` varchar(255) UNIQUE NOT NULL DEFAULT '',
[SQL] pobierz, plaintext 

?

Dostrzegłem jeszcze jeden problem.
Tak jak napisalem wyzej

[PHP] pobierz, plaintext 
<?php
if (mysql_num_rows($wynik) ==1 )
?>
[PHP] pobierz, plaintext 

rozwiązuje problem.
Gdy użytkownik logujac sie wciska spacje przechodzi walidacje błędów :-(

Zastosowalem funkcje TRIM() ktora usuwa dodatkowe spacje z konca i poczatku tekstu, ale nie pomaga:

[PHP] pobierz, plaintext 
<?php
$ksywa=trim(htmlspecialchars($_POST['ksywa']));
$email=htmlspecialchars(trim($_POST['email']));
$haslo=htmlspecialchars(trim($_POST['haslo']));
$haslo2=htmlspecialchars(trim($_POST['haslo2']));
?>
[PHP] pobierz, plaintext