[PHP] Wyrażenie Regularne php, Problem z walidiacją numeru Tel przy użyciu wyrażenia regularnego Opcje

[korey]

Witam!
Napisałem coś takiego:

[PHP] pobierz, plaintext 
 
//Sprawdzam pole "szkola_tel"
if(empty($szkola_tel)){
		$info_txt_szkola_tel.='<font color="#B20000"> Pole nie zostało wypełnione.</font>';
		$blad=true;
}
else if(!ereg('^[0-9]{3}-[0-9]{2,3}-[0-9]{2,3}$'  , $szkola_tel )){
	$info_txt_szkola_tel.='<font color="#B20000">Nieprawidłowy numer telefonu.</font>';
	$help_txt.='~ <font color="#B20000">Nr.Tel podaj w/g przykładu: <b>000-000-000</b> lub <b>000-00-00</b></font><br />';	
	$blad=true;
}
else{
	$info_txt_szkola_tel.='<font color="#207C07"> OK.</font>';	
}
[PHP] pobierz, plaintext 

Wyrażenie działa poprawnie jak wpisuje sie numrey tel a takich formatach:
000-00-00
lub
000-000-000

Chciałbym do tego wszystkiego dodać numer kierunkowy, czyli np: +XX\s
lecz gdy próbuje to robię wyrażenie zwraca błąd.

Z góry dziękuje za pomoc

[TNT]

+ chyba musi być escapowany, spróbuj:

Kod

^\+[0-9]{2}\s[0-9]{3}-[0-9]{2,3}-[0-9]{2,3}$

PS
ereg() jest przestarzała - http://pl.php.net/manual/en/function.ereg.php

[korey]

Hmm to jakiej funkcji używać.?

przerobiłem kod na:

[PHP] pobierz, plaintext 
	$tel = trim($szkola_tel);
	if(empty($szkola_tel)){
		$info_txt_szkola_tel.='<font color="#B20000"> Pole nie zostało wypełnione.</font>';
		$blad=true;
	}
	else if(!preg_match('^\+[0-9]{2}\s[0-9]{3}-[0-9]{2,3}-[0-9]{2,3}$'  , $tel )){
		$info_txt_szkola_tel.='<font color="#B20000">Nieprawidłowy numer telefonu.</font>';
		$help_txt.='~ <font color="#B20000">Nr.Tel podaj w/g przykładu: <b>000-000-000</b> lub <b>000-00-00</b></font><br />';	
		$blad=true;
	}
	else{
		$info_txt_szkola_tel.='<font color="#207C07"> OK.</font>';	
	}
[PHP] pobierz, plaintext 

wstawilem preg_match()
w formularzy wpisywałem:
+23 324-343-434
i
+99 234-12-12
funkcja nadal zwraca błąd

Ten post edytował korey 28.02.2010, 13:50:58

[TNT]

preg_match ma inne ograniczniki(?) - nie wiem jak to nazwać
Zobacz

Kod

else if(!preg_match('/\+[0-9]{2}\s[0-9]{3}-[0-9]{2,3}-[0-9]{2,3}/'  , $tel )){

[korey]

użyłem twojego kodu i nic.
usunąłem trim() i nic.
dodałem ^ i $ przed / i nic
dodałem ^ i $ po / i nic
nie ma jakiegoś innego sposobu na walidiacje numeru tel.?

[TNT]

Zobacz co u ciebie wyświetli

[PHP] pobierz, plaintext 
<?php
$tel = '+48 000-09-32';
if (preg_match('/\+[0-9]{2}\s[0-9]{3}-[0-9]{2,3}-[0-9]{2,3}/'  , $tel))
  print 'ogien';
else
  print 'failur';
?>
[PHP] pobierz, plaintext 

[pyro]

Nawet nie patrzyłem na przykłady innych, po prostu ułożyłem nowe:

[PHP] pobierz, plaintext 
$phone_no = '+48 503-500-999';
if(preg_match('|(\+[0-9]{2} )?[0-9]{3}-[0-9]{2,3}-[0-9]{2,3}|', $phone_no))
{
echo 'Numer telefonu poprawny';
}
else
{
echo 'Nieprawidłowy numer telefonu';
}
[PHP] pobierz, plaintext 

Ten post edytował pyro 28.02.2010, 14:56:33

[korey]

Nawet nie patrzyłem na przykłady innych, po prostu ułożyłem nowe:

[PHP] pobierz, plaintext 
$phone_no = '+48 503-500-999';
if(preg_match('|(\+[0-9]{2} )?[0-9]{3}-[0-9]{2,3}-[0-9]{2,3}|', $phone_no))
{
echo 'Numer telefonu poprawny';
}
else
{
echo 'Nieprawidłowy numer telefonu';
}
[PHP] pobierz, plaintext 

Kod działa poprawnie ale gdy np podaje taki numer:
+3323 234-234-234 zwraca mi OK.
da sie to jakoś ulepszyć.?

teraz tak wygląda kod:

[PHP] pobierz, plaintext 
	//Sprawdzamy pole "szkola_tel"
	if(empty($szkola_tel)){
		$info_txt_szkola_tel.='<font color="#B20000"> Pole nie zostało wypełnione.</font>';
		$blad=true;
	}
	else if(!preg_match('|(\+[0-9]{2} )?[0-9]{3}-[0-9]{2,3}-[0-9]{2,3}|'  , $szkola_tel )){
		$info_txt_szkola_tel.='<font color="#B20000">Nieprawidłowy numer telefonu.</font>';
		$help_txt.='~ <font color="#B20000">Nr.Tel podaj w/g przykładu: <b>+00 000-000-000</b> lub <b>+32 000-00-00</b></font><br />';	
		$blad=true;
	}
	else{
		$info_txt_szkola_tel.='<font color="#207C07"> OK.</font>';	
	}
[PHP] pobierz, plaintext 

[thek]

Po pierwsze czy ktoś zwrócił uwagę na fakt, że + jest znakiem specjalnym wyrażeń i nie może być wstawiany ot tak sobie, gdyż oznacza, że poprzedzany element musi wystąpić przynajmniej jeden raz? Inna sprawa, że nawet gdy to poprawisz to nie uwzględniasz, że brak kierunkowego znowu wywali błąd. Ta część ma być opcjonalna, więc bez użycia ? się nie obejdzie.
EDIT: Chyba mi się na oczy już rzuciło dziś, albo za długo siedzę przy kompie, ale nie zauważyłem Twojego i późniejszego. Opisywałem wcześniejsze z postem TNT włącznie. Moja wina i kajam się za błąd z warunkowym kierunkowym.

Zastanawiam się, czy aby nie lepszym byłoby ewentualne "chwilowe" usunięcie kierunkowego, czyli +XX i pozbawienie ciągu wszelkich znaków innych niż cyfry bo przecież minus lub spacja mogą wystąpić niekoniecznie w podanych miejscach. Normą jest podawanie numerów nie tylko trójkami, ale także parami, a więc także w postaci: +48 503 50 09 99. Może użyć filter_var i zrobić "satanizację"? W ten sposób nawet głupoty w stylu 7f5s-7f7s76 przeczyści do 75-7776 lub napisać samemu funkcję, która obetnie ewentualny kierunkowy i zostawi same cyfry by myślniki i spacje nie robiły bajzlu. No ale to już kwestia potrzeb i własnego pomysłu na przechowywanie numeru.

Ten post edytował thek 2.03.2010, 17:04:36

[pyro]

Po pierwsze czy ktoś zwrócił uwagę na fakt, że + jest znakiem specjalnym wyrażeń i nie może być wstawiany ot tak sobie, gdyż oznacza, że poprzedzany element musi wystąpić przynajmniej jeden raz? Inna sprawa, że nawet gdy to poprawisz to nie uwzględniasz, że brak kierunkowego znowu wywali błąd. Ta część ma być opcjonalna, więc bez użycia ? się nie obejdzie.

Czytałeś w ogóle to wyrażenie regularne? I jedno i drugie zostało uwzględnione.

@korey, zobacz to:

[PHP] pobierz, plaintext 
 
preg_match('|^(\+[0-9]{2} )?[0-9]{3}-[0-9]{2,3}-[0-9]{2,3}$|'  , $szkola_tel )
[PHP] pobierz, plaintext 

Ten post edytował pyro 2.03.2010, 16:31:59

[Fifi209]

Proszę wyrażenie + przykład:

[PHP] pobierz, plaintext 
<?php
 
	$tel[] = '+32 250-250-45';
	$tel[] = '+3423 234-34-56';
	$tel[] = '250-78-455';
 
	foreach ($tel as $val) {
		if (preg_match('#^(\+[\d]{2}+)?\s?([\d]{3}-[\d]{2,3}-[\d]{2,3})$#', $val)) {
			echo $val.'<br/>';
		}
	}
 
?>
[PHP] pobierz, plaintext 

[korey]

Noo.
Użyłem wyrażenia od fifi209 sprawdziłem numery na różne sposoby i tym razem wszystko jest dobrze.
Dziękuje wam wszystkim.
ale skoro jesteśmy już w temacie wyrażeń to da sie jakoś przerobić to:
Tak by akceptowało adres w postaci "www.strona strony ze uwzględnionymi znakami - _ (i innymi używanymi w adresie). pl,com" itp

[PHP] pobierz, plaintext 
	$url = trim($szkola_www);
	if(empty($szkola_www)){
	// POLE NIE OBOWIĄZKOWE	
	}
	else if (!preg_match('/^(http|ftp)([s]{0,1}):\/\/([a-z0-9]{1})((([a-z0-9-]*[-]{2})|([a-z0-9])*|([a-z0-9-]*[-]{1}[a-z0-9]+))*)((\.[a-z0-9](([a-z0-9-]*[-]{2})|([a-z0-9]*)|([a-z0-9-]*[-]{1}[a-z0-9]+))+)*)(\.([a-z0-9]{2,6})){0,1}((:[0-9]){0}|(:[1-9]{1}[0-9]*))\//iu', $url)){
		$info_txt_szkola_www.='<font color="#B20000">Nieprawidłowy adres url.</font>';
		$help_txt.='~ <font color="#B20000">Url podaj w/g przykładu: <b>http://</b>www.strona.com<b>/</b></font><br />';
		$blad=true;
	}
	else{
		$info_txt_szkola_www.='<font color="#207C07"> OK.</font>';	
	}
[PHP] pobierz, plaintext 

Bo w tym przykładzie działa tylko tak:
http://www.strona.pl/
albo
http://www.strona.com/ itp.
prościej mówiąc chce usunąć http:// i / na koncu

Ten post edytował korey 3.03.2010, 08:30:16

[thek]

Może to tylko mój pomysł, ale połącz walidację z poprawianiem Jeśli brak jest protokołu, to dopisz go i potem już użyj filter_var( $test, FILTER_VALIDATE_URL ). Dopisanie brakującego http:// jest akurat tutaj konieczne, gdyż filtr ten wymaga posiadania protokołu przez ciąg znakowy. Oczywiście wcześniej możesz jeszcze potraktować całość FILTER_SANITIZE_URL

Ten post edytował thek 3.03.2010, 08:58:18

[korey]

Yyyy.. nie rozumiem. ze niby jak?

[thek]

FILTER_SANITIZE_URL usuwa wszystkie snaki, które nie mogą pojawić się w URL. Według specyfikacji zostawia jedynie liczby, cyfry oraz znaki $-_.+!*'(),{}|\\^~[]`<>#%";/?:@&=. a reszta wylatuje. Teraz jedynie sprawdzamy czy ma "z przodu" protokół jakikolwiek jeśli go brak, doklejamy http:// a potem jeszcze walidacja z użyciem FILTER_VALIDATE_URL. Wiem, że można od razu preg_match walnąć, ale jakoś mam większe zaufanie dla wbudowanych reguł filtrujących niż pisanie wyrażenia "z głowy" i preferuję zdać się podczas walidacji na pewne funkcje. Jedyne co trzeba jeszcze sprawdzić to czy da się przepchnąć adres ip zamiast domeny http://80.56.200.43/index.php da się zwalidować czy nie? Bo i takie adresy mogą się przecież trafić...

EDIT: Żeby nie było. Adres podany przeze mnie jest czysto przypadkowy i nie prowadzi nigdzie. To ślepa uliczka mająca jedynie na celu zaprezentowanie wariantu z możliwą sytuacją walidacji IP zamiast domeny

Ten post edytował thek 3.03.2010, 13:28:58

[korey]

Napisałem coś takiego:

[PHP] pobierz, plaintext 
	//Sprawdzamy pole "szkola_www"
	if(empty($szkola_www)){
	// POLE NIE OBOWIĄZKOWE	
	}
	else if(!filter_var($szkola_www, FILTER_VALIDATE_URL)){
		$info_txt_szkola_www.='<font color="#B20000">Nieprawidłowy adres url.</font>';
		$help_txt.='~ <font color="#B20000">Url podaj w/g przykładu: <b>http://</b>www.strona.com<b>/</b></font><br />';
		$blad=true;
	}
	else{
		$info_txt_szkola_www.='<font color="#207C07"> OK.</font>';	
	}
[PHP] pobierz, plaintext 

Po wysłaniu formularza mam taki komunikat:
Fatal error: Call to undefined function: filter_var() in C:\Program Files\WebServ\httpd-users\Szkolne_Kola\administrator\add_szkola.php on line 138

Nie mam pojęcia czemu

Ten post edytował korey 6.03.2010, 13:15:40

[tr0lit]

Musisz posiadać na serwerze wersję PHP 5.2.0, lub wyższą. Sprawdź używając funkcji phpinfo().

[strife]

Napisałem coś takiego:

[PHP] pobierz, plaintext 
	//Sprawdzamy pole "szkola_www"
	if(empty($szkola_www)){
	// POLE NIE OBOWIĄZKOWE	
	}
	else if(!filter_var($szkola_www, FILTER_VALIDATE_URL)){
		$info_txt_szkola_www.='<font color="#B20000">Nieprawidłowy adres url.</font>';
		$help_txt.='~ <font color="#B20000">Url podaj w/g przykładu: <b>http://</b>www.strona.com<b>/</b></font><br />';
		$blad=true;
	}
	else{
		$info_txt_szkola_www.='<font color="#207C07"> OK.</font>';	
	}
[PHP] pobierz, plaintext 

Po wysłaniu formularza mam taki komunikat:
Fatal error: Call to undefined function: filter_var() in C:\Program Files\WebServ\httpd-users\Szkolne_Kola\administrator\add_szkola.php on line 138

Nie mam pojęcia czemu

Może masz starą wersję php? http://pl.php.net/filter_var sprawdź jaką masz wersje korzystając z phpinfo();

[korey]

no macie racje.. ;/ To moze mógłby mo ktoś powiedzieć jak zaktualizować php w programie WebServ 1.3.?
Albo ewentualnie zmienić serwer na którym mógłbym te stronkę testować..?

[tr0lit]

Ściągnij najnowszą wersję PHP z oficjalnej strony i podmień pliki, albo zmień WebServ na WAMP, albo XAMPP.

Ten post edytował tr0lit 6.03.2010, 20:35:32