Witam

Mam skrypt ban listy na swojej stronie, korzysta ona z pliku ban.txt niby wszystko ok jak mam dopisane 1 ip do działa ale jak dopiszę następne np. po spacji albo enterze to już nie działa(osoby z tymi ip mogą wejść na stronę) czy ktoś może zobaczyć ten skrypt i w razie czego przerobić go aby można było dopisywać ip po spacji czyli np. 53.21.252.253 53.21.252.257

Oto ten skrypt:

-kod wklejany do index.php

[PHP] pobierz, plaintext 
<?php
include( 'cfg/ban.lib.php' );
   if( ban( file( 'cfg/ban.txt' ), $_SERVER['REMOTE_ADDR'] ) ) {
	  die( 'Twoje ip jest zbanowane'.$_SERVER['REMOTE_ADDR'].'' );
   }
?>
[PHP] pobierz, plaintext 

-plik ban.lib.php

[PHP] pobierz, plaintext 
<?php
function ban( $arrList, $ip ) {
	  if( in_array( $ip, $arrList ) ) {
		 return TRUE;
	  }
	  $replace = array( '*' => '([0-9]{1,3})', '?' => '([0-9]{1})', '.' => '.');
 
	  foreach( $arrList AS $key => $value ) {
		  if( preg_match( '#^'.strtr( $value, $replace ).'$#', $ip ) ) {
			 return TRUE;
		  }
 
	  }
 
	  return FALSE;
   }
?>
[PHP] pobierz, plaintext 

Z góry dziękuję za pomoc

Spróbuj tak

[PHP] pobierz, plaintext 
<?php
include( 'cfg/ban.lib.php' );
   if( ban( file_get_contents( 'cfg/ban.txt' ), $_SERVER['REMOTE_ADDR'] ) ) {
	  die( 'Twoje ip jest zbanowane'.$_SERVER['REMOTE_ADDR'].'' );
   }
?>
 
//drugi plik
 
<?php
function ban( $arrList, $ip ) 
{
	 $arrIP = explode(" ",$arrList); 
 
	 if( in_array( $ip, $arrIP ) ) 
		 return TRUE;
	 else
		 return FALSE;
  }
?>
[PHP] pobierz, plaintext 

funkcja in_array() nie znajduje ci IP ponieważ standardowo file() w każdym wierszu zwraca też znaki końca linii, zmień na:

[PHP] pobierz, plaintext 
<?php
if( ban( file( 'cfg/ban.txt', FILE_IGNORE_NEW_LINES), $_SERVER['REMOTE_ADDR'] ) ) {
?>
[PHP] pobierz, plaintext 

i będzie działać. kolejne wpisy IP umieszczaj w osobnych liniach pliku, bez żadnych spacji przed lub po... powodzenia.

To tak kod podany przez @Lonas troszkę mniej mniej interesuje, ponieważ chciałbym mieć banowanie z maskami (np. 54.65.*) a po zmianie podanej przez @nevt jest tak, że wstawiam np. tak ip do txt:

[HTML] pobierz, plaintext 
83.21.253.52
83.21.253.54
83.21.253.51
[HTML] pobierz, plaintext 

To wtedy banuje tylko podany jako ostatni ip, gdy wstawię swój ip jako np. pierwszy albo drugi od końca to normalnie wchodzę na stronę.

czy ten plik tworzysz pod windozą jeżeli tak, to wróć do pierwotnej wersji (z pierwszego posta), a pierwszej linijce funkcji ban() dopisz:

[PHP] pobierz, plaintext 
<?php
function ban( $arrList, $ip ) {
  $arrList = str_replace(array("r", "\n"), array('', '') $arrList);
  if( in_array( $ip, $arrList ) ) {
...
?>
[PHP] pobierz, plaintext 

Kurcze zmieniłem tak jak podał kolega @nevt, przynajmniej z tego co ja zrozumiałem miałem zmienić w pliku ban.lib.php i zmieniłem tak:

[PHP] pobierz, plaintext 
<?php
function ban( $arrList, $ip ) {
$arrList = str_replace(array("r", "\n"), array('', '') $arrList);
	  if( in_array( $ip, $arrList ) ) {
		 return TRUE;
	  }
	  $replace = array( '*' => '([0-9]{1,3})', '?' => '([0-9]{1})', '.' => '.');
 
	  foreach( $arrList AS $key => $value ) {
		  if( preg_match( '#^'.strtr( $value, $replace ).'$#', $ip ) ) {
			 return TRUE;
		  }
 
	  }
 
	  return FALSE;
   }
?>
[PHP] pobierz, plaintext 

Jeśli to miało być zrobione w ten sposób to lipa, bo wywala błąd i strona się nie wyświetla

[PHP] pobierz, plaintext 
<?php
 
include( 'cfg/ban.lib.php' );
if(ban('cfg/ban.txt', $_SERVER['REMOTE_ADDR'])){ 
  die( 'Twoje ip jest zbanowane: '.$_SERVER['REMOTE_ADDR'].'</b>' ); 
}
 
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
 
function ban($filename, $remoteAddr){
  $fileContents = file($filename, FILE_IGNORE_NEW_LINES);
  $replace = array( '*' => '([0-9]{1,3})', '?' => '([0-9]{1})');  
 
  foreach($fileContents as $line){ 
	if(preg_match( '#^' . strtr($value, $replace) . '$#', $remoteAddr)) 
	  return true; 
 
  return false;
}
 
?>
[PHP] pobierz, plaintext 

W tym kodzie co podałeś brakowało { za

[PHP] pobierz, plaintext 
<?php
if(preg_match( '#^' . strtr($value, $replace) . '$#', $remoteAddr))
?>
[PHP] pobierz, plaintext 

i

}
}
za

[PHP] pobierz, plaintext 
<?php
return true;
?>
[PHP] pobierz, plaintext 

Bo jak ich nie było wywalało bład ale mimo, że poprawiłem dodając te znaki niby juz nie było blędów na stronie ale mimo dopisywania do txt ip nie banowało

zabrakło jednego przecinka!!! spróbuj raz jeszcze:

[PHP] pobierz, plaintext 
<?php
$arrList = str_replace(array("r", "\n"), array('', ''), $arrList);
?>
[PHP] pobierz, plaintext 

Dzięki @nevt teraz jest ok