Wyrazenia regularne, znowu ? :) Opcje

[Rafiki]

Witam, mam jedna prosbe, otoz napisalem sobie warunki z preg_match walidujace poprawnosc pol w formularza , wiekszosc dziala ok ale nie wszystkie :/ kod wyglada tak:

[PHP] pobierz, plaintext 
<?
		if ( $_POST['password'] == $_POST['password_retype'] )
		{
			if ( $_POST['email'] == $_POST['email_retype'] )
			{
				if ( preg_match('/[w.]+/', $_POST['login']) )
				{
					if ( preg_match('/[w.]+/', $_POST['password']) )
					{
						if ( preg_match('/[a-zA-ZąćęłńóśźżĄĆŁŃÓŚŹŻ]+/', $_POST['first_name']) )
						{
							if ( preg_match('/[a-zA-ZąćęłńóśźżĄĆŁŃÓŚŹŻ]+/', $_POST['last_name']) )
							{
								if ( preg_match('/([w.]+)(@)([w.]+)(.)([a-zA-Z]){2,4}/', $_POST['email']) )
								{
									if ( is_numeric($_POST['gg']) || empty($_POST['gg']) )
									{
										if ( preg_match('/[w.]+/', $_POST['tlen']) || empty($_POST['tlen']) )
										{
											$query = // zapytanie do bazy mysql
 
												if ( $query )
												{
													$error = $error.'Użytkownik został dodany do bazy pomyślnie ! '.$br.' <a href="admins.php">Powrót do poprzedniej strony.</a>'.$br;
												}
												else
												{
													$error = $error.'Użytkownik nie został dodany do bazy, skontaktuj się z administratorem ! ';
												}
											}
										}
										else
										{
											$error = 'Login tlen zawiera niedozwolone znaki ( nie podawaj znaków specjalnych oraz polskich liter ) ! ';
										}
									}
									else
									{
										$error = 'Numer gg zawiera niedozwolone znaki ( używaj tylko cyfr ) ! ';
									}
								}
								else
								{
									$error = 'Adresy e-mail jakie podałeś nie są zapisane w formacie: <b>login@serwer.domena</b> ! ';
								}
							}
							else
							{
								$error = 'Podane nazwisko jest zapisane w złej postaci ( używaj tylko liter ) ! ';
							}
						}
						else
						{
							$error = 'Podane imię jest zapisane w złej postaci ( używaj tylko liter ) ! ';
						}
					}
					else
					{
						$error = 'Hasło zawiera niedozwolone znaki ( nie podawaj znaków specjalnych oraz polskich liter ) ! ';
					}
				}
				else
				{
					$error = 'Login zawiera niedozwolone znaki ( nie podawaj znaków specjalnych oraz polskich liter ) ! ';
				}
			}
			else
			{
				$error = 'Adresy e-mail ,które podałeś nie są identyczne ! ';
			}
		}
		else
		{
			$error = 'Hasła ,które podałeś nie są identyczne ! ';
		}		
	}
	else
	{
		$error = 'Nie wypełniłeś wymaganych pól formularza ! ';
	}
?>
[PHP] pobierz, plaintext 

nie ktore pola zle mi waliduje :| chociazby adres tlen, gdy wpisze znaki specjalne to i tak przechodzi walidacje i uzytkownik jest dodawany :| Nie mam pojecia gdzie sa bledy

[crash]

Jak na mój gust to każde wyrażenie jest tutaj błędne... Zapomniałeś o ograniczeniu całości przez ^ $... I dla wartości liczbowych proponuję jednak [0-9]+...

[FiDO]

Pozatym struktura tego kodu moze przyprawic o dreszcze.. Jak wciecia siegaja polowy strony to z reguly znaczy, ze cos jest nie tak z kodem i ja sie z tym zgadzam. Wciecia sa oczywiscie niezbedne, ale jesli ktos robi takie pietrusy to przestaje to byc ladne..
Strach pomyslec, jakby to wygladalo, gdybys mial w ten sposob walidowac formularz niebanalnych rozmiarow..
Lepiej to zaneguj.. sprawdzasz kazdy warunek po kolei tylko, ze zanegowany i jesli zostanie spelniony to ustawiasz $error. W ten sposob unikasz takich kosmicznych wciec.

[Rafiki]

crash i FIDO dzieki za rady (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Niestety czasem nie mysle (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) trzeba to zmienic (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Aktualnie kod ( a wlasciwie juz cala funkcja do walidacji wyglada tak, chcialbym sie was poradzic ( bo jak widac z php mam od nie dawna stycznosc ) czy wszystko jest ok ? czy moze jeszcze cos nalezy zmienic ?

[PHP] pobierz, plaintext 
<?php
function set_inspection_add_admins()
{
	global $error, $br;
 
	if ( empty($_POST['login']) && empty($_POST['password']) && empty($_POST['password_retype']) && empty($_POST['first_name']) && empty($_POST['last_name']) && empty($_POST['email']) && empty($_POST['email_retype']) )
	$error = 'Nie wypełniłeś wymaganych pól formularza ! ';
 
	if ( $_POST['password'] !== $_POST['password_retype'] )
	$error = $error.'Hasła ,które podałeś nie są takie same ! ';
 
	if ( $_POST['email'] !== $_POST['email_retype'] )
	$error = $error.'Adresy e-mail ,które podałeś nie są takie same ! ';
 
	if ( !preg_match('/^[w.]+$/', $_POST['login']) )
	$error = $error.'Login zawiera niedozwolone znaki ( nie podawaj znaków specjalnych oraz polskich liter ) ! ';
 
	if ( !preg_match('/^[w.]+$/', $_POST['password']) )
	$error = $error.'Hasło zawiera niedozwolone znaki ( nie podawaj znaków specjalnych oraz polskich liter ) ! ';
 
	if ( !preg_match('/^[a-zA-ZąćęłńóśźżĄĆŁŃÓŚŹŻ]+$/', $_POST['first_name']) )
	$error = $error.'Podane imię jest zapisane w złej postaci ( używaj tylko liter ) ! ';
 
	if ( !preg_match('/^[a-zA-ZąćęłńóśźżĄĆŁŃÓŚŹŻ]+$/', $_POST['last_name']) )
	$error = $error.'Podane nazwisko jest zapisane w złej postaci ( używaj tylko liter ) ! ';
 
	if ( !preg_match('/^([w.]+)(@)([w.]+)(.)([a-zA-Z]){2,4}$/', $_POST['email']) )
	$error = $error.'Adresy e-mail jakie podałeś nie są zapisane w formacie: <b>login@serwer.domena</b> ! ';
 
	if ( !preg_match('/^[0-9]+]$/', $_POST['gg']) || empty($_POST['gg']) )
	$error = $error.'Numer gg zawiera niedozwolone znaki ( używaj tylko cyfr ) ! ';
 
	if ( !preg_match('/^[w.]+$/', $_POST['tlen']) || empty($_POST['tlen']) )
	$error = $error.'Login tlen zawiera niedozwolone znaki ( nie podawaj znaków specjalnych oraz polskich liter ) ! ';
 
	if ( empty($error) )
	{
		$query = mysql_query('SELECT login, password, chmod FROM admins WHERE login=''.$_POST['login'].''');
		$result = mysql_fetch_assoc($query);
 
		if ( $result )
		{
			$error = 'Użytkownik o takim loginie istnieje ! ';
		}
		else
		{
			if( $_POST['chmod_root'] == '' ) $_POST['chmod_root'] = 0;
			if( $_POST['chmod_admins'] == '' ) $_POST['chmod_admins'] = 0;
			if( $_POST['chmod_users'] == '' ) $_POST['chmod_users'] = 0;
			if( $_POST['chmod_categories'] == '' ) $_POST['chmod_categories'] = 0;
			if( $_POST['chmod_news'] == '' ) $_POST['chmod_news'] = 0;
			if( $_POST['chmod_articles'] == '' ) $_POST['chmod_articles'] = 0;
			if( $_POST['chmod_comments'] == '' ) $_POST['chmod_comments'] = 0;
 
			$true_chmod = $_POST['chmod_root'].'.'.$_POST['chmod_admins'].'.'.$_POST['chmod_users'].'.'.$_POST['chmod_categories'].'.'.$_POST['chmod_news'].'.'.$_POST['chmod_articles'].'.'.$_POST['chmod_comments'];
 
			$query = mysql_query('INSERT INTO admins VALUES (''.NULL.'', CURDATE(), ''.$_POST['login'].'', ''.SHA1($_POST['password']).'', ''.$true_chmod.'', ''.$_POST['first_name'].'', ''.$_POST['last_name'].'', ''.$_POST['email'].'', ''.$_POST['gg'].'', ''.$_POST['tlen'].'')');
 
			$send_mail = @mail($_POST['email'], 'Zostałes nowym użytkownikiem: '.$config['site_name'].'', 'Witaj !'.$br.'Zostałeś nowym użytkownikiem na stronie: '.$config['site_name'].'. Aby się zalogować odwiedź stronę: '.$$config['url_site'].$br.$br.' Podając dane: '.$br.$br.' Login: '.$_POST['login'].$br.'  Hasło: '.$_POST['password'].$br.$br.' W razie jakich kolwiek problemów skontaktuj się z administratorem serwisu !. '.$br.$br.' Życzymy miłego dnia, zespół: '.$config['site_name'].'','From: '.$config['email_admin'].''); 
 
			if ( !$send_mail )
			$error = 'E-mail do użytkownika z zawiadomieniem iż został dodany do bazy nie dotarł ! '.$br.' Najprawdopodbniej serwer SMTP w konfiguracji php jest źle skonfigurowany !'.$br.$br;
 
			if ( $query )
			{
				$error = $error.'Użytkownik został dodany do bazy pomyślnie ! '.$br.$br.' <a href="admins.php">Powrót do poprzedniej strony.</a>'.$br;
			}
			else
			{
				$error = $error.'Użytkownik nie został dodany do bazy, skontaktuj się z administratorem ! ';
			}
		}
	}
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Rafiki 13.01.2006, 22:03:06