Spara besökares IP-adresser?

Du stänger aldrig första if-satsen väl?

Du behöver inte köra else efter if (!$con), eftersom att om die() körs så exekveras ändå inte koden nedanför

EDIT: du kan till och med ta bort hela den if-satsen eftersom du gör den kontrollen längre upp $con = mysql_connect("localhost","root","") or die(mysql_error())

Else-blocket har inget slut.

Du använder funktionen ip2long: http://php.net/manual/en/function.ip2long.php

Finns andra metoder för IPv6 dock

Du kör din fråga två gånger, eller ja, en tom fråga andra gången.
Kör det här istället:

if (!mysql_query("INSERT INTO ipadresser ('','".$ip."','".$datum."','".$unix."')"))
  {
  die('Error: ' . mysql_error());
  }

Det saknas även ett ; efter mysql_close($con) på näst sista raden.

Antingen gör du som havsmonstret skriver, eller så ändrar du
$query = mysql_query ("INSERT INTO ipadresser ('','".$ip."','".$datum."','".$unix."')");

till
$query = "INSERT INTO ipadresser ('','".$ip."','".$datum."','".$unix."')";

tycker det är kul när det dyker om trådar som man inte fattar ett skvatt av

Hej tänkte jag kunde låna tråden lite, eftersom många har dynamisk IP är det bättre att spara hostname istället?

Vet inte riktigt, jag har gjort en poll där rösterna verifieras med IP men vissa personer kan rösta igen när deras IP ändras. Så jag tänkte använda hostname istället för den ändras väl inte alla fall den man får från sin ISP? Nu använder jag Cookies + IP men finns det något bättre?

Är det inte bättre och skippa datum helt och hållet och bara använda unix? Sen kanske du måste välja vilka timezone du är i när du kör med date.

Funkar datumet eller, du bör köra med PHP strict så blir det bättre i framtiden sen så använd date("r") istället.

Vet inte varför du får fel IP men jag kör med den här funktionen sen så är 127.0.0.1 nästan samma sak som localhost så om du använder localhost i din adress så får du så klart 127.0.0.1 som din IP.

	function get_ip_address()
	{
		if(!empty($_SERVER['HTTP_CLIENT_IP']))
	    {
			$ip = $_SERVER['HTTP_CLIENT_IP'];
	    }
	    else if(!empty($_SERVER['HTTP_X_FORWARDED_FOR']))
	    {
			$ip = $_SERVER['HTTP_X_FORWARDED_FOR'];
	    }
	    else
	    {
			$ip = $_SERVER['REMOTE_ADDR'];
	    }
		
	    return $ip;
	}

Observera att du måste starta en session före.

<?php
// Kollar ifall IP-adressen finns i session
if (isset($_SESSION['Ip']))
{
$ip_first = $_SESSION['Ip'];
}

//finns den inte kollar den om den finns i databasen
else
{
$ip_first = $_SERVER['REMOTE_ADDR'];
$_SESSION['Ip'] = $ip_first;

$Time_first = date('U');

require("connect.php");
$check = mysql_query("SELECT * FROM connections WHERE Ip='$ip_first'");
$numrows_ip = mysql_num_rows($check);

while ($row2 = mysql_fetch_assoc($check))
{
$Ip_store = $row2['Ip'];
}

//ifall den inte finns så skriver den in det
if ($Ip_store != $ip_first)
{
$write = mysql_query("INSERT INTO connections VALUES ('','$ip_first','0','$Time_first',NULL)");
}

}
?>

connect.php innehåller förljande:

<?php



$connect = mysql_connect("localhost","namn","pass") or die("Connection failed! Contact admin.");
mysql_select_db("databasnamn") or die("Connection failed! Contact admin.");



?>

På detta sätt slipper du ens göra en förfrågan till databasen ifall sessionen finns. Detta borde minska trafiken lite grann. För ifall du gör en förfrågan varje gång kommer det ju ändå gå åt en del prestanda varje gång någon laddar en sida.