Blokowanie IP Opcje

[urbizdw]

Witam!
Chcialem napisac skrypt w php ktory czyta z pliku adresy IP (adresy umieszczone sa linijka pod linijka) i jesli jest taki adres w pliku to zeby wyswietlilo tekst zablokowane a jesli niema adresu to jestes. Co jest nie tak z moim skryptem (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

[PHP] pobierz, plaintext 
<?php
function ip_prawdziwe() {
if ($_SERVER['HTTP_X_FORWARDED']) {
	$ip_prawdziwe= $_SERVER['HTTP_X_FORWARDED_FOR'];
}
else {
	$ip_prawdziwe = $_SERVER['REMOTE_ADDR'];
}
 
return $ip_prawdziwe;
}
 
$ip= IP_prawdziwe();
 
$plik= file("blockip.dat");
$n= 0;
while($n < count($plik)){
if($plik[$n] == $ip){
echo 'zablokowane';
}
else{ echo 'jestes';
$n++;
}
	}
?>
[PHP] pobierz, plaintext 

[Jojo]

W pliku jest coś takiego jak znaki końca linii. Jeśli rozbijesz plik na tablicę przy pomocy funkcji file(), to te znaczki nadal tam są i porównanie zmiennej $ip z linijką pliku nigdy nie da Ci poprawnego rezultatu. Musisz przed porównaniem pozbyć się znaków końca linii z elementów tablicy $plik.

[nospor]

Ano to jest nie tak, ze nie zajrzales do manuala:
http://pl.php.net/manual/pl/function.file.php

Notatka: Każda linia w wynikowej tabeli będzie zawierać znak(i) końca linii, jeśli nie chcesz ich musisz uzyć rtrim().

Co oznacza ze te twoje IP z pliku nigdy sie nie bedzie rownalo z tym co koles aktualnie wszedl, bo to z pliku bedzie mialo jeszcze znak konca linii na koncu. Co nalezy zrobic w tej sytuacji masz napisane (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

pozatym zwiekszanie tego $n w warunku to baaaaaardzo zly pomysl. Jak ci trafi na zbanowane IP to z tej petli nigdy nie wyjdziesz (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[urbizdw]

Probowalem:

[PHP] pobierz, plaintext 
<?php
function ip_prawdziwe() {
if ($_SERVER['HTTP_X_FORWARDED']) {
	$ip_prawdziwe= $_SERVER['HTTP_X_FORWARDED_FOR'];
}
else {
	$ip_prawdziwe = $_SERVER['REMOTE_ADDR'];
}
 
return $ip_prawdziwe;
}
 
$ip= IP_prawdziwe();
 
$plik= file("blockip.dat");
$n= 0;
while($n < count($plik)){
rtrim($plik, 't');
if($plik[$n] == $ip){
echo 'jestes';
}
else{ echo 'nie jestes';
}
	}
?>
[PHP] pobierz, plaintext 

tez nic (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) wiec co jest jeszcze nie tak??

Ten post edytował urbizdw 8.09.2006, 14:11:25

[cinekz]

[PHP] pobierz, plaintext 
<?php
function ip_prawdziwe() {
if ($_SERVER['HTTP_X_FORWARDED']) {
	$ip_prawdziwe= $_SERVER['HTTP_X_FORWARDED_FOR'];
}
else {
	$ip_prawdziwe = $_SERVER['REMOTE_ADDR'];
}
 
return $ip_prawdziwe;
}
 
$ip= IP_prawdziwe();
 
$plik= file("blockip.dat");
$n= 0;
while($n < count($plik)){
if( rtrim( $plik[$n] ) == $ip){
echo 'zablokowane';
}
else{ echo 'jestes';
$n++;
}
	}
?>
[PHP] pobierz, plaintext 

[urbizdw]

To tez nie dziala bo az petla krazy caly czas w kolko i nie przestaje dzialac (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) az przegladarka pada (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[nospor]

NO ale czytaj chlopie cosie do ciebie pisze:

pozatym zwiekszanie tego $n w warunku to baaaaaardzo zly pomysl. Jak ci trafi na zbanowane IP to z tej petli nigdy nie wyjdziesz

Masz te zwiekszanie $n dac niezaleznie od warunku...

[urbizdw]

A wiec dalem :

[PHP] pobierz, plaintext 
<?php
function ip_prawdziwe() {
if ($_SERVER['HTTP_X_FORWARDED']) {
$ip_prawdziwe= $_SERVER['HTTP_X_FORWARDED_FOR'];
}
else {
$ip_prawdziwe = $_SERVER['REMOTE_ADDR'];
}
return $ip_prawdziwe;
}
$ip= IP_prawdziwe();
$plik= file("blockip.dat");
$n= 0;
while($n < count($plik)){
if( rtrim( $plik[$n] ) == $ip){
echo 'zablokowane';
}
else{ echo 'jestes';
}
$n++;
}
?>
[PHP] pobierz, plaintext 

Dałem $n++ poza warunek ale nadal cos nie tak. Powinno wyswietlac jedna z informacji ze albo zablokowane albo ze jestes. a mi wyswietla zablokowanejestes. co jest nie tak??

Ten post edytował urbizdw 8.09.2006, 15:13:53

[tiraeth]

Panowie, mamy do czynienia z "użytkownikiem"...

Przeanalizuj jeszcze raz działanie programu czyli napisz sobie graficzny algorytm....

[PHP] pobierz, plaintext 
<?php
$ip = ( $_SERVER['HTTP_X_FORWARDED'] ) ? $_SERVER['HTTP_X_FORWARDED_FOR'] : $_SERVER['REMOTE_ADDR'];
 
$plik = file('blockip.dat');
$n = 0;
 
$banned = false;
while ( $n < count($plik) )
{
	if ( rtrim( $plik[$n] ) == $ip )
	{
		$banned = true;
		break;
	}
}
 
if ( $banned )
{
	echo '<span style="color:#ff0000">Wstęp zablokowany</span>';
}
else
{
	echo '<span style="color:#00ff00">Wstęp dozwolony</span>';
}
?>
[PHP] pobierz, plaintext 

[Mystic007]

Ja bym w pętlę wywalił, i dał w if'ie funkcję in_array" title="Zobacz w manualu php" target="_manual. (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Powodzenia