Forum PHP.pl > [PHP] walidacja pól

Pomoc - Szukaj - Użytkownicy - Kalendarz

krystian1991x

18.09.2010, 12:03:33

Witam mam pytanie jak inaczej czyli prościej zrobić walidacje która sprawdza czy pola formularza zostały wszystkie wypełnione ja zrobiłem coś takiego ale niezbyt mi to odpowiada

[PHP] pobierz, plaintext 
 
if($_POST['login'] == ""|| $_POST['login2'] == "" || $_POST['login3'] == ""  || $_POST['login4'] == "") {
 
 
 
 
}else {
 
}
[PHP] pobierz, plaintext

Fifi209

18.09.2010, 12:23:51

Zrób tablicę z kluczami, które chcesz sprawdzić potem przeleć foreach + isset ew. empty

krystian1991x

18.09.2010, 12:32:45

jakiś przykład ?

Ramzaa

18.09.2010, 12:38:33

Cytat(krystian1991x @ 18.09.2010, 13:32:45 )

jakiś przykład ?

if (isset($_POST['cos'])) { }

krystian1991x

18.09.2010, 13:31:32

raczej chodziło mi o to jak stworzyć te tablice z kluczem ;P

Zdecydowałem że jednak zrobię sobie to tak ale ta konstrukcja nie wiedzieć czemu nie jest prawidłowa

krispak

18.09.2010, 14:44:42

Cytat

Zdecydowałem że jednak zrobię sobie to tak ale ta konstrukcja nie wiedzieć czemu nie jest prawidłowa

Tzn jak bo nie wiem, o ktora konstrukcje chodzi?

krystian1991x

18.09.2010, 15:01:39

[PHP] pobierz, plaintext 
<?php
 
   require('libs/Smarty.class.php');
   require('cfg.php');
 
  $smarty = new Smarty;
 
 $login     = htmlspecialchars(mysql_escape_string($_POST['login']));           
 $name      = htmlspecialchars(mysql_escape_string($_POST['name']));  
 $last_name = htmlspecialchars(mysql_escape_string($_POST['last_name']));
 $password1 = htmlspecialchars(mysql_escape_string($_POST['pw1']));  
 $password2 = htmlspecialchars(mysql_escape_string($_POST['pw2']));
 $email1    = htmlspecialchars(mysql_escape_string($_POST['email1']));
 $email2    = htmlspecialchars(mysql_escape_string($_POST['email2']));
 $phone1    = htmlspecialchars(mysql_escape_string($_POST['phone1']));
 $phone2    = htmlspecialchars(mysql_escape_string($_POST['phone2']));
 
   if(isset($_POST['register'])) {
 
    if(!empty($_POST['login']))     { if(!empty($_POST['name'])) {
    if(!empty($_POST['last_name'])) { if(!empty($_POST['pw1'])) {
    if(!empty($_POST['pw2']))       { if(!empty($_POST['email1'])) {
    if(!empty($_POST['email2']))    { if(!empty($_POST['phone1'])) {
    if(!empty($_POST['phone2']))    { if(empty($_POST['adres'])) {
 
     if(preg_match('#^[A-Za-z0-9_-]+$#', $login) && strlen($login) > 5 || strlen($login) < 17 ) {
     if(preg_match('#^[A-Za-z_-]+$#', $name) && strlen($name) > 3 || strlen($name) < 17) {
     if(preg_match('#^[A-Za-z_-]+$#', $last_name) && strlen($last_name) > 3 || strlen($last_name) < 32) {
     if(preg_match('#^[A-Za-z0-9_-]+$#', $pw1) && strlen($pw1) > 5 || strlen($pw1) < 17 ) {
     if(preg_match('#^[A-Za-z0-9_-]+$#', $pw2)) {
     if(preg_match('#^[+0-9_-]+$#', $phone1) && strlen($phone1) == 9) {
     if(preg_match('#^[+0-9_-]+$#', $phone2) && strlen($phone1) > 8 || strlen($phone1) < 12) {
 
      echo "Teoretycznie dziala";
 
     } else{ echo "Numer telefonu stacjonarnego nie jest poprawny";}
     } else{ echo "numer telefonu komórkowego nie jest poprawny";}
     } else{ echo "Haslo nie jest poprawne";}
     } else{ echo "Haslo nie jest poprawne";}
     } else{ echo "Nazwisko nie jest poprawne";}
     } else{ echo "Imie nie jest poprawne";}
     } else{ echo "login nie jest poprawny";}
 
 
    } else { echo "Podaj adres"; } }   else { echo "Podaj telefon stacjonarny"; }
    } else { echo "Podaj telefon"; } } else { echo "Powtórz email"; } 
    } else { echo "Podaj email"; } }   else { echo "Powtórz haslo"; }
    } else { echo "Podaj haslo"; } }   else { echo "Podaj nazwisko"; } 
    } else { echo "Podaj imie"; } }    else { echo "Podaj login"; }
 
   }
 
  $smarty->assign('login', $login);
  $smarty->assign('name', $name);
  $smarty->assign('last_name', $last_name);
  $smarty->assign('password1', $password1);
  $smarty->assign('password2', $password2);
  $smarty->assign('email1', $email1);
  $smarty->assign('email2', $email2);
  $smarty->assign('phone1', $phone1);
  $smarty->assign('phone2', $phone2);
 
  $smarty->display('register.tpl');
 
?>
[PHP] pobierz, plaintext

jak do tej pory tak u mnie to wygląda jak to zrobię to będę myślał czy da się łatwiej i jak Bo mi nic innego do głowy nie przychodziło

Wicepsik

18.09.2010, 15:28:31

1. Po co używasz funkcji mysql_escape_string, htmlspecialchars jak i tak potem sprawdzasz zmienne wyrażeniami regularnymi ?
2. Sprawdzenie długości znaków możesz załatwić na poziomie wyrażeń regularnych: http://pl.wikibooks.org/wiki/PHP/Podstawy_...%84_regularnych

Fifi209

18.09.2010, 15:40:28

Bardziej chodziło mi o coś takiego:

[PHP] pobierz, plaintext 
	$klucze = array('login', 'password', 'last_name', 'pw');
 
	foreach ($_POST as $key => $value) {
		if (in_array($key, $klucze)) {
			if (empty($value)) {
				die('Wypełnij wszystkie pola!');
			}
		}
	}
[PHP] pobierz, plaintext

Zamiast wyrażeń w wielu przypadkach możesz użyć: ctype_ lub filter_var

Ale trzymając się pierwszej koncepcji możesz zrobić np.

[PHP] pobierz, plaintext 
	$keys = array('login' => '#^[\w\d]{5,20}$#', 'password' => '#^[\w\d]{5,20}$#');
 
	foreach ($_POST as $key => $value) {
		if (array_key_exists($key, $keys)) {
			if (!preg_match($keys[$key], $value)) {
				die('Coś poszło nie tak');
			}
		}
	}
[PHP] pobierz, plaintext

Albo dodatkowo rozbudować o błędy:

[PHP] pobierz, plaintext 
	$keys = array('login' => array('reg' => '#^[\w\d]{5,20}$#', 'err' => 'Login zawiera niedozwolone znaki!'), 'password' => array('reg' => '#^[\w\d]{5,20}$#', 'err' => 'Hasło zawiera niedozwolone znaki'));
 
	foreach ($_POST as $key => $value) {
		if (array_key_exists($key, $keys)) {
			if (!preg_match($keys[$key]['reg'], $value)) {
				die($keys[$key]['err']);
			}
		}
	}
[PHP] pobierz, plaintext

Wszystko pisane z palca, nie testowałem.

krystian1991x

18.09.2010, 19:10:22

wytłumaczy mi ktoś ten zapis ?

[PHP] pobierz, plaintext 
 '#^[\w\d]{5,20}$#'
[PHP] pobierz, plaintext

Fifi209

18.09.2010, 23:12:49

http://pl.php.net/manual/pl/regexp.reference.delimiters.php

http://pl.php.net/manual/pl/regexp.reference.backslash.php

Ogólnie poczytaj sobie:
http://pl.php.net/manual/pl/reference.pcre...tern.syntax.php

krystian1991x

19.09.2010, 00:41:24

oh nie prosiłem o linki bo to sobie umiem wyszukać ... od tego mamy google ! a forum chyba służy do czegoś innego ? szkoda że niektórzy mylą wyszukiwarki z forami ...

Fifi209

19.09.2010, 09:04:56

Skoro potrafisz sobie sam wyszukać to po co na forum zadajesz takie pytania?

Jakiś większy problem, że to jest po angielsku? Trudno, taka praca programisty.

krystian1991x

19.09.2010, 10:40:00

Wujek ! ty se uważaj

Po co zadaje na forum ? bo od tego ono jest nie wiem po co było zakładane forum mogli po prostu zrobić przekierowanie do manuala

Fifi209

19.09.2010, 10:44:33

Cytat

any "word" character

Cytat

any decimal digit

# #

Cytat

When using the PCRE functions, it is required that the pattern is enclosed by delimiters. A delimiter can be any non-alphanumeric, non-backslash, non-whitespace character.

Often used delimiters are forward slashes (/), hash signs (#) and tildes (~). The following are all examples of valid delimited patterns.

^ $

{1,3}
Zapis taki oznacza: minimum jeden znak, maksimum trzy znaki.

krystian1991x

19.09.2010, 14:45:51

da się w zakresie tej tablicy jeszcze sprawdzić takie rzeczy jak czy login jest zajęty oraz czy pola z haslami sie zgadzają ?

a i gdzie mam teraz umieścić zapytanie ?

[PHP] pobierz, plaintext 
	foreach ($_POST as $key => $value) {
		if (array_key_exists($key, $keys)) {
			if (!preg_match($keys[$key]['reg'], $value)) {
		   $error = $keys[$key]['err'];
			}
		}
	}
[PHP] pobierz, plaintext

utworzyć else dla tego warunku ?

[PHP] pobierz, plaintext 
			if (!preg_match($keys[$key]['reg'], $value)) {
		   $error = $keys[$key]['err'];
			}
[PHP] pobierz, plaintext

Fifi209

19.09.2010, 15:57:55

Ja to podsunąłem jako pomysł do sprawdzania czy pole jest puste, ew. czy zawiera dozwolone znaki.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.