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]

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.