Blokowanie dostępu do strony, sprawdzenie poprawności kodu... Opcje

[Forgoot]

Witam!
Jako że jestem początkujący proszę Was, abyście powiedzieli czy ten skrypt jest poprawnie napisany, jeśli nie to proszę o ewentualną poprawkę i komentarz "co źle" zrobilem... Może da się ten skrypt rozbudować, dodać "coś", jeśli tak to proszę o podpowiedzi...

[PHP] pobierz, plaintext 
<?php
/**************************************************************************
 *  Prosty skrypt zabraniający dostępu do strony określonemu adresowi IP  *
 *  zapisanego w pliku ban.txt.                                           *
 **************************************************************************/
 
$adres = strval($_SERVER['REMOTE_ADDR']); 
$ip_zbanowane = @file("ban.txt"); 
    if($_SERVER['REMOTE_ADDR'] == "$ip_zbanowane")
  {
          /*Kod strony*/ 
    }else{
echo("<b>ERROR!</b> Nie masz uprawnień do oglądania tej strony.<br /> Twój IP: $adres został zbanowany.");
  }
 
?>
[PHP] pobierz, plaintext 

Plik ban.txt wygląda (przykładowo) tak:

Kod

127.0.0.1
127.0.0.2
127.0.0.3
127.0.0.4

Pozdrawiam.

(IMG:http://forum.php.pl/uwaga.gif)
~mike_mech

Ten post edytował Forgoot 27.10.2005, 10:11:57

[Forgoot]

1. Czy kod powinien wyglądać tak?? :

[PHP] pobierz, plaintext 
<?php
 
/**************************************************************************
 *  Prosty skrypt zabraniający dostępu do strony określonemu adresowi IP  *
 *  zapisanego w pliku ban.txt.                                           *
 **************************************************************************/
 
$adres = strval($_SERVER['REMOTE_ADDR']); 
$ip_zbanowane = @file("ban.txt"); 
   
   function GetClientIP()
{
    $ip = 0;
    
    if( ! empty( $_SERVER[ 'HTTP_CLIENT_IP' ] ) )
    {
        $ip = $_SERVER[ 'HTTP_CLIENT_IP' ];
    }
    
    if( ! empty( $_SERVER[ 'HTTP_X_FORWARDED_FOR' ] ) )
    {
        $ipList = explode( ", ", $_SERVER[ 'HTTP_X_FORWARDED_FOR' ] );
        if( $ip )
        {
            array_unshift( $ipList, $ip );
            $ip = 0;
        }
        foreach( $ipList as $v )
        {
            if( ! eregi( "^(192.168|172.16|10|224|240|127|0.", $v ) )
            {
                return $v;
            }
        }
    }
 
    return $ip ? $ip : $_SERVER[ 'REMOTE_ADR' ] == $ip_zbanowane;
} 
echo("<b>ERROR!</b> Nie masz uprawnień do oglądania tej strony.<br /> Twój IP: $adres został zbanowany.");
 
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
 
return $ip ? $ip : $_SERVER[ 'REMOTE_ADR' ] == $ip_zbanowane;
 
?>
[PHP] pobierz, plaintext 

użyłem tutaj operatora przypisania, niewiem czy dobrze i we właściwym miejscu...

LUB (drugi sposób, w jaki starałem się do skryptu dopisać Twoją funkcję @mike_mech):

[PHP] pobierz, plaintext 
<?php
/**************************************************************************
 *  Prosty skrypt zabraniający dostępu do strony określonemu adresowi IP  *
 *  zapisanego w pliku ban.txt.                                           *
 **************************************************************************/
 
include ("funkcja.php");
$adres = strval($_SERVER['REMOTE_ADDR']); 
$ip_zbanowane = @file("ban.txt"); 
    if( in_array( $_SERVER['REMOTE_ADDR'], $ip_zbanowane ) )
  {
          /*Kod strony*/ 
    }else{
echo("<b>ERROR!</b> Nie masz uprawnień do oglądania tej strony.<br /> Twój IP: $adres został zbanowany.");
  }
 
?>
[PHP] pobierz, plaintext 

Plik funkcja.php wygląda tak:

[PHP] pobierz, plaintext 
<?php
/******************
*  funkcja.php    *
*******************/
 
function GetClientIP()
{
    $ip = 0;
    
    if( ! empty( $_SERVER[ 'HTTP_CLIENT_IP' ] ) )
    {
        $ip = $_SERVER[ 'HTTP_CLIENT_IP' ];
    }
    
    if( ! empty( $_SERVER[ 'HTTP_X_FORWARDED_FOR' ] ) )
    {
        $ipList = explode( ", ", $_SERVER[ 'HTTP_X_FORWARDED_FOR' ] );
        if( $ip )
        {
            array_unshift( $ipList, $ip );
            $ip = 0;
        }
        foreach( $ipList as $v )
        {
            if( ! eregi( "^(192.168|172.16|10|224|240|127|0.", $v ) )
            {
                return $v;
            }
        }
    }
 
    return $ip ? $ip : $_SERVER[ 'REMOTE_ADR' ];
}
 
?>
[PHP] pobierz, plaintext 

Jeśli jest źle to prosze o nakierowanie mnie na właściwą drogę, bo szczerze mówiąc niewiedziałem zbytnio jak to zrobić poprawnie :/

2. Chciałbym też stworzyć skrypt wyświetlający losowe cytaty z pliku tekstowego. Proszę Was abyście podali mi kolejno jakie funkcje trzeba wykorzystać (czy mam zastosować randomizację?), postaram się zaprezentować potem moje "wypociny" (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)

Ten post edytował Forgoot 28.10.2005, 15:00:14