Hjelp til en simpel hitcounter

[The_Lozer]

denne hitcounteren teller kun en gang per ip, eller det vil si den skal telle en gang per ip

 

<?php

$fil = file("ip.txt");
$tell = count($fil);

$ip = $_SERVER["REMOTE_ADDR"];


foreach($fil as $linje) {
$ex = explode("*", $linje);
if ($ip !== $ex[0]) {

$skrive = fopen("ip.txt", "a");
$text = "{$ip}*ok*\r\n";
fwrite($skrive, $text);
fclose($skrive);
header("refresh:1");
break;
}

else

echo $tell;
}
?>

 

men hva er galt?

 

takker for alle svar

[Ståle]

<?php

$fil = file("ip.txt");

$tell = count($fil);

$ip = $_SERVER['REMOTE_ADDR'];

foreach($fil as $linje) {

$ex = explode("*", $linje);

if ($ip != $ex[1]) {

$skrive = fopen("ip.txt", "a");

$text = "*$ip*ok*\r\n";

fwrite($skrive, $text);

fclose($skrive);

header("refresh:1");

}

 

else

 

echo $tell;

}

?>

[Ric_H]

foreach($fil as $linje) {
$ex = explode("*", $linje);
if ($ip !== $ex[0]) {

$skrive = fopen("ip.txt", "a");
$text = "{$ip}*ok*\r\n";
fwrite($skrive, $text);
fclose($skrive);
header("refresh:1");
break;
}

 

Jeg antar at det er en feil i if-setningen, som resulterer i at du skriver IP'en til fil med en gang du finner en IP som er ulik. Du sjekker ikke egentlig om IP-adressen er der, eller ikke.

 

Du bør kanskje vente med all skriving til fil til etter foreach-loopen er ferdig. Forslag (ikke testet):

 

$exists = false;

foreach($fil as $linje) {
   $ex = explode("*", $linje);
   if ($ip == $ex[0])
       $exists = true;
}

if ($exists)
   echo $tell;
else {
   $skrive = fopen("ip.txt", "a");
   $text = "{$ip}*ok*\r\n";
   fwrite($skrive, $text);
   fclose($skrive);
   header("refresh:1");
}

[The_Lozer]

EDIT: ... igjen jeg dreit meg ut litt, det funka takker for svar

Endret 5. februar 2007 av The_Lozer

[ZoRaC]

Hvorfor vil dere ikke kjøre echo $tell; uansett?

Slik den er nå så skriver den bare ut antallet om IPen er telt fra før...