Ip log : unik?

Ikke for noget men hvorfor er jeres koder så underlige, jeg vil have gjort sådan:

<?php
require_once('top.php');
    if(phpversion() >= "4.2.0") {
      extract($_SERVER);
    }
$ip  = $_SERVER[REMOTE_ADDR];
$nu = now();
include("sql.php");
mysql_query(INSERT INTO log (ip, dato) values ('$ip', '$nu'");
require_once('bund.php');
?>

mysql_query(INSERT INTO log (ip, dato) values ('$ip', '$nu'");

Rettes til:

mysql_query("INSERT INTO log (ip, dato) values ('$ip', '$nu'");

jeps :P men nu gjorde jeg sådan der :P ;)

web-geek >
1. Han extracter $_SERVER, så det er helt sort at definere en ekstra var med ip'en.
2. Når der bruges strenge som index i et array, bruger man "" eller ''
3. Man kan ligeså godt bruge MySQL's indbyggede funktioner. Less is more.

-padde- >
Hvad mener du når du skriver unikke hits? Skal den kun indsætte én post om dagen pr. ip, eller?

ja et hit pr ip... :)

Prøv med det her:

<?
require_once('top.php');
    if(phpversion() >= "4.2.0") {
      extract($_SERVER);
    }

include "sql.php";
$idag = mysql_query("SELECT COUNT(*) AS antal FROM log WHERE ip='$REMOTE_ADDR' AND date_add(dato,interval 24 HOUR) > NOW() ORDER BY id DESC LIMIT 0,1") or die(mysql_error());
if (!mysql_result($idag,0)) {
    mysql_query($sql = "INSERT INTO log (ip, dato) values ('$REMOTE_ADDR', now())") or die (mysql_error(). 'SQL: ' .$sql);
}
require_once('bund.php');
?>

luk sig..