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

[mike]

Funkcja file() zwraca tablicę, co oznacza, że ta linia:

[PHP] pobierz, plaintext 
<?php
 
//...
if($_SERVER['REMOTE_ADDR'] == "$ip_zbanowane")
//...
 
?>
[PHP] pobierz, plaintext 

Jest błędna bo porównujesz stringa do tablicy i to jeszcze źle to robisz (po co Ci znaki cudzysłowiu ).
Lepiej będzie:

[PHP] pobierz, plaintext 
<?php
 
//...
if( in_array( $_SERVER['REMOTE_ADDR'], $ip_zbanowane ) )
//...
 
?>
[PHP] pobierz, plaintext 

Teraz php sprawdzi czy adres usera jest w tablicy IP zbanowanych userów.

P.S. $_SERVER może zawierać mało wiarygodny adres IP, aby mieć większą pewnośc co do adresu, użyj funkcji:

[PHP] pobierz, plaintext 
<?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 

[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

[Sabistik]

[PHP] pobierz, plaintext 
<?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' ];
}
 
$banArr = file("ban.txt");
if(in_array(GetClientIP(), $banArr) )
echo "<b>ERROR!</b> Nie masz uprawnień do oglądania tej strony.<br /> Twój IP:".GetClientIP()." został zbanowany.";
 
 
?>
[PHP] pobierz, plaintext 

[mike]

Co do 1. @Sabistik podał Ci odpowiedź.

A co do:

2. Chciałbym też stworzyć skrypt wyświetlający losowe cytaty z pliku tekstowego.

[PHP] pobierz, plaintext 
<?php
 
$arrQuotes = file( 'quotes.txt' );
echo array_rand( $arrQuotes );
 
?>
[PHP] pobierz, plaintext 

Gdzie plik quotes.txt wygląda tak:

I spraw Panie aby nam się chciało tak bardzo jak nam się nie chce.
Nie dyskutuj z debilem! Najpierw sprowadzi Cię do swojego poziomu, a potem pokona doświadczeniem.
Jak najdzie Cię ochota na pracę, usiądź i poczekaj aż przejdzie.

Wystarczy poczytać Manuala.
Albo użyć wyszukiwarki forum, wpisać do niej losowanie i znaleśc to: losowanie liczb z tablicy (temat identyczny, bo co za różnica co losujemy (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) )

Zaniedbywanie manuala i wyszukiwarki jest na tym forum karane (IMG:http://forum.php.pl/style_emoticons/default/aaevil.gif)

[Forgoot]

Znalazłem, napisałem, w ten sam dzień co zadałem pytanie dot. losowych cytatów, ale niestety nie mam (chwilowo) dostępu do internetu 24 godziny na dobę.... więc piszę dziś (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

Oto moje rozwiązania dotyczące losowych cytatów:
dwa kody wyświetlające losowe cytaty (pobierane z tablicy)
1.

[PHP] pobierz, plaintext 
<?php
/*********************************
 *  Prosty skrypt wyświetlający  * 
 *  losowy cytat tablicy		*
 *  version: 0.2				 *
 *********************************/
 
	echo("Losowy cytat: ");
//tablica zawierająca cytaty
$cytat = array(
'1' => 'Jakiś cytat 1',
'2' => 'Jakiś cytat 2',
'3' => 'Jakiś cytat 3',
'4' => 'Jakiś cytat 4',
'5' => 'Jakiś cytat 5'
); 
 
$numer=rand(1,5);
 
	echo ("$cytat[$numer]");
?>
[PHP] pobierz, plaintext 

2.

[PHP] pobierz, plaintext 
<?php
/*********************************
 *  Prosty skrypt wyświetlający  * 
 *  losowy cytatz tablicy		*
 *  version: 0.2				 *
 *********************************/
 
	echo("Losowy cytat: ");
 
//tablica zawierająca cytaty
$cytat = array(
'1' => 'Jakiś cytat 1',
'2' => 'Jakiś cytat 2',
'3' => 'Jakiś cytat 3',
'4' => 'Jakiś cytat 4',
'5' => 'Jakiś cytat 5',
'6' => 'Jakiś cytat 6',
'7' => 'Jakiś cytat 7',
'8' => 'Jakiś cytat 8',
'9' => 'Jakiś cytat 9',
'10' => 'Jakiś cytat 10'
); 
 
	srand((float) microtime() * 10000000); //inicjujemy generator liczb losowych 
 
$cytat = $cytat[array_rand($cytat)]; //wybieramy jeden losowy cytat z tablicy
 
	echo "[ $cytat ]"; //wyświetlamy losowy cytat
?>
[PHP] pobierz, plaintext 

3 Tutaj napisałem kod podobnego skryptu do tych w/w, tylko teraz cytaty pobierane są z pliku...

[PHP] pobierz, plaintext 
<?php
/*********************************
 *  Prosty skrypt wyświetlający  * 
 *  losowy cytat, pobierany z	*
 *  pliku: cytat.txt			 *
 *********************************/	
 
	echo("Losowy cytat: ");
$cytat = array();
 
$file = file('cytat.txt'); 
	srand((float) microtime() * 10000000); //inicjujemy generator liczb losowych 
 
$cytat = $file[array_rand($file)]; 
 
echo "[ $cytat ]"; //wyświetlamy losowy cytat 
?>
[PHP] pobierz, plaintext 

Plik cytat.txt wygląda tak:

Kod

Jakiś cytat 1
Jakiś cytat 22
Jakiś cytat 3
Jakiś cytat 45
Jakiś cytat 1200
Jakiś cytat 8

Prosiłbym abyście powiedzieli czy nie ma błędów (czy składnia jest zachowana itp...), jeśli są to proszę o komentarz... Dzięki bardzo.
Pozdrawiam!

[mike]

Dobry programista to leniwy programista, taki, który tobi wszystko jak najprościej. W przykładach, które podałeś jest za dużo kombinowania.
Po co robić tak:

[PHP] pobierz, plaintext 
<?
$file = file('cytat.txt'); 
srand((float) microtime() * 10000000); //inicjujemy generator liczb losowych 
$cytat = $file[array_rand($file)]; 
?>
[PHP] pobierz, plaintext 

Skoro można tak:

[PHP] pobierz, plaintext 
<?php
</php
$arrQuotes = file( 'quotes.txt' );
echo array_rand( $arrQuotes );
?>
[PHP] pobierz, plaintext 

:?:

Składnia jest zachowana i jako takich błędów nie ma. Ale są to rozwiązania bardzo kipskie.

Czytałeś w ogóle posta, którego napisałem powyżej :?:

Cytat z manuala funkcji srand()

Notatka: Począwszy od php 4.2.0, nie trzeba już inicjować generatora liczb losowych przed użyciem.

[Forgoot]

Ok, dzięki @mike_mech za pomoc (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) .

Dobry programista to leniwy programista..

na przyszłoś zapamiętam (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)