Witam

chcę zrobić banowanie przyczym mam formularz gdzie recznie wpisuję adres IP i chodzi mi aby byly przyjmowane tylko poprawne adresy ip.

W sieci znalazłem cos takiego:

[PHP] pobierz, plaintext 
<?php
if (!eregi("^[_a-z0-9-]+(.[_a-z0-9-]+)*@[a-z0-9-]+(.[a-z0-9-]+)*(.[a-z]{2,4})$", $email))
?>
[PHP] pobierz, plaintext 

wstawiłem sobie to w jedno miejsce i maila sprawdza poprawnie. Poprzerabiałem to troszkę ale nie do końca to sprawdzanie IP jest poprawne.

Pomoże ktoś ?

Nie za bardzo wiem co sobie poprzerabiales, bo Twoje wyrazenie regularne adresu IP na 100% nie sprawdza, no chyba, ze w IP masz znaczek '@' no i adres zaczyna sie od litery :-) .

Moze cos takiego zadziala:

[PHP] pobierz, plaintext 
<?php
 
$ip="192.168.11.255";
 
if(eregi("^[0-9]{1,3}.[0-9]{1,3}.[0-9]{1,3}.[0-9]{1,3}$",$ip))
echo "jest ok";
else
echo "nie jest ok";
 
?>
[PHP] pobierz, plaintext 

A tak przy okazji moze warto w googlach wpisac "wyrazenia regularne"

niestety to działą jeszcze gorzej niż to ja co przerobiłem gdyż np IP 1.1 które jest niepoprawne jest uwazane za poprawne.


Chodzi o to aby tylko IP 0.0.0.0 do 255.255.255.255 (nawet jakby bylo 999.999.999.999) bylo uwazane za poprawne

Podobny problem rozwiązywaliśmy tutaj. Do sprawdzania czy jest to poprawne powinna wystarczyć zmodyfikowana funkcja z tamtego tematu.

[PHP] pobierz, plaintext 
<?php
function isIP( $ip )
{ 
	$pattern = "/^([0-9]{1,3}).([0-9]{1,3}).([0-9]{1,3}).([0-9]{1,3})$/";
 
	if( !preg_match( $pattern, trim($ip), $ip ) )
		return false;
 
	foreach( $ip as $key => $val )
		$ip[$key] = intval($val);
	if( ($ip[1] >= 0 && $ip[1] < 256) && ($ip[2] >= 0 && $ip[2] < 256) && ($ip[3] >= 0 && $ip[3] < 256) && ($ip[4] >= 0 && $ip[4] < 256) )
		return true;
	return false;
}
?>
[PHP] pobierz, plaintext 

i przykład

[PHP] pobierz, plaintext 
<?php
$ip[] = '0.0.0.0';
$ip[] = '255.255.255.255';
$ip[] = '999.999.999.999';
$ip[] = '1.1.1.1';
$ip[] = '2.2';
$ip[] = '192,165.1';
$ip[] = '234.10.5';
$ip[] = '12.34.56.12';
 
 
foreach( $ip as $ipItem )
	if( isIP($ipItem) )
		echo "$ipItem\n";
?>
[PHP] pobierz, plaintext 

a jego wynik to:

0.0.0.0
255.255.255.255
1.1.1.1
12.34.56.12

Choć ja wcale nie jestem taki pewien czy 0.0.0.0 to faktycznie poprawny adres IP.

edit --> Poprawiłem błąd jaki miałem w kodzie. Dodałem warunek w linii 6.

no racja, moj blad - z rozpedu zapomnialem ze nie chodzi o cyfry a o liczby do 255.
Sorki, za blad. Musze bardziej przemyslec post zanim klikne "dodaj odpowiedz"

Ale tak przy okazji '1.1' jest traktowane jako niepoprawny adres, wlasie testowalem:

[PHP] pobierz, plaintext 
<?php
 
$adr=array('1.1.1.1','2.2','0.0.0.0.0','192,165.1','234.10.5','12.34.56.12');
 
foreach($adr as $ip)
{
  if(eregi("^[0-9]{1,3}.[0-9]{1,3}.[0-9]{1,3}.[0-9]{1,3}$",$ip))
	  echo "$ip -- jest ok<br>";
  else
	 echo "$ip -- nie jest ok<br>";
}
?>
[PHP] pobierz, plaintext 

a wynik to :

1.1.1.1 -- jest ok
2.2 -- nie jest ok
0.0.0.0.0 -- nie jest ok
192,165.1 -- nie jest ok
234.10.5 -- nie jest ok
12.34.56.12 -- jest ok

Mam nadzieje, ze nie zostanie to potraktowane jako nabijanie postow.

Heh dziwne bo u mnie wyświetlało na odwrót


przestawiłem tylko u siebie na odwrót i jest teraz git (hehe dodaje nawet 999.999.999.999 ale tutaj trzeba by zbudować dużo dłuższy warunek)

Można prościej:

[PHP] pobierz, plaintext 
<?php
 
	// PRZYKŁADOWE TESTY:
	#$ip = '0.0.0.0';
	#$ip = '127.0.0.1';
	#$ip = '255.255.255.255';
	$ip = '123.12.3.123';
	#$ip = '0.0.0.256';
	#$ip = '999.999.999.999';
	#$ip = 'ala ma kota';
 
 
	if( $ip == long2ip( ip2long( $ip ) ) )
	{
		echo $ip . ' - poprawny adres IP';
	}
	else
	{
		echo $ip . ' - niepoprawny adres IP';
	}
 
 
?>
[PHP] pobierz, plaintext 

Jejku dzięki wielkie  teraz jest lux  przedział 0.0.0.0-255.255.255.255 inne są niepoprawne