[php][mysql]System Zabezpieczający. Opcje

[Rash]

Zrobiłem Skrypt.
Skrypt Ten Zapisuje ip odwiedzającego,date i godzine wejscia na stronę:

[PHP] pobierz, plaintext 
<?php
$ip = $_SERVER['REMOTE_ADDR']; //Przypisujesz Zmiennej IP.
$czas = date("H:i:s"); //Przypisujesz zmiennej czas.
$data = date("d-m-Y"); //Przypisujesz Zmiennej Date..
mysql_connect ("host", "login", "haslo"); //Łączysz Się z Bazą
mysql_select_db ("baza"); //Wybierasz Bazę
$zapytanie = "INSERT INTO `ip` (`id`, `ip`, `czas`, `data`) VALUES ('', '$ip', '$czas', '$data')";//Przypisujesz zmiennej Zapytanie
mysql_query($zapytanie); //Wykonujesz Zapytanie
mysql_close(); //Zamykasz połączenie
?>
[PHP] pobierz, plaintext 

Możesz wstawić to do dowlnej podstrony za pomocy:

[PHP] pobierz, plaintext 
<?php
include("ip.php");
?>
[PHP] pobierz, plaintext 

To zapytanie Sql:

[SQL] pobierz, plaintext 
CREATE TABLE `ip` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `ip` varchar(200) collate utf8_polish_ci DEFAULT NULL,
  `czas` varchar(200) collate utf8_polish_ci DEFAULT NULL,
  `data` varchar(200) collate utf8_polish_ci DEFAULT NULL,
  PRIMARY KEY  (`id`))
[SQL] pobierz, plaintext 

Mam nadzieje że się wam przyda

[cckoles]

1. Jakie IP otrzymasz jak ktos będzie się łaczył przez proxy ?
2. Jesli juz funkcja mysql to moze lepiej mysqli ? Mysql jest przestarzale i zalecane jest uzywac mysqli.


3.

[SQL] pobierz, plaintext 
CREATE TABLE `ip` (
`id` int(11) NOT NULL AUTO_INCREMENT,
`ip` varchar(200) collate utf8_polish_ci DEFAULT NULL,
`czas` varchar(200) collate utf8_polish_ci DEFAULT NULL,
`data` varchar(200) collate utf8_polish_ci DEFAULT NULL,
PRIMARY KEY (`id`))
[SQL] pobierz, plaintext 

czas i data jako varchar ?
ip moze miec 200 znakow ?

Uwierz mi, że nie czepiam się ale dobrze przeanalizuj swój kod (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Pozdrawiam