[PHP] download counter

[Verwijderd]

Heyz mensen,

ik heb een download counter geschreven en hierdoor zou er in een simpel tekstbestand elke keer op een nieuwe regel een IP moeten worden opgeslagen, en als er met dat IP al gedownload is zou er dan een '2' of een '3' (afhankelijk van hoe vaak er gedownload is met dat IP) voor moeten komen te staan..

Het probleem is alleen dat de 1e regel telkens wordt overschreven met het laatste IP dat gedownload heeft... Maar hoe vaak het is gedownload wordt wel telkens 1 meer...

PHP:

<?php

$expire= 1; 

$logfile= "logbestand.log"; 
    
$ip= getenv('REMOTE_ADDR');
$visits=0;
$badhit= false;
$now= time();

$ips = array(array());
if (file_exists($logfile)){
    if ($loggedips=file($logfile)){
        $visits=trim($loggedips[0]);
        for ($i=1; $i< count($loggedips); $i++){
            $loggedips[$i]=trim($loggedips[$i]);
            $ips[$i] = explode('||', $loggedips[$i]);
            if (($ips[$i][0]==$ip) && ($now-$ips[$i][1]< $expire)) 
                $badhit= true;
        }
        if ($badhit)
            echo $visits;
        else{
            $visits++;
            $fp= fopen($logfile, 'w');
            fputs($fp,"$visits\n");
            for ($i=1; $i< count($loggedips); $i++){
                if ($now-$ips[$i][1] < $expire)
                    fputs($fp, $ips[$i][0]."||".$ips[$i][1]."\n");
            }
            fputs($fp, "$ip||$now\n");
            fclose($fp);
            echo $visits;           
        }
    }
}else
    echo "Logfile bestaat niet...";



?>

[Jelle Niemantsverdriet]

En je vraag is?

De if in regel 14 klopt in ieder geval niet Sorry, las niet goed

[ Voor 21% gewijzigd door Jelle Niemantsverdriet op 19-09-2003 16:55 ]

[Limhes]

Jelle Niemantsverdriet schreef op 19 September 2003 @ 16:46:
En je vraag is?

De if in regel 14 klopt in ieder geval niet

Waarom zou het if-statement in regel 14 niet kloppen volgens jou? Kijk eens goed wat hij ermee wil bereiken.

edit:
@djluc: niet snel dingen wegediten he

[ Voor 10% gewijzigd door Limhes op 19-09-2003 16:49 ]

[djluc]

Zijn vraag is waarom steeds de regel wordt overschreven ipv een nieuwe regel die wordt toegevoegd. In Perl kon je zoiets doen met >> in je handler o.i.d. Met PHP zou ik het echter niet weten.

Probeer eens een print_r van de array te maken zodat je zeker weet dat die klopt.

[ Voor 37% gewijzigd door djluc op 19-09-2003 16:49 ]

[Suepahfly]

Limhes schreef op 19 September 2003 @ 16:48:
[...]


Waarom zou het if-statement in regel 14 niet kloppen volgens jou? Kijk eens goed wat hij ermee wil bereiken.

Er staat maar één ``='' dat moet ern twee zijn in een vergelijking

[Limhes]

Suepahfly schreef op 19 september 2003 @ 16:50:
Er staat maar één ``='' dat moet ern twee zijn in een vergelijking

Kijk dan eens goed of iets een vergelijking is of niet.

[djluc]

Dit is een filehandler, daar hoeft dat niet volgens mij.

edit:
Damn weer te laat.

[ Voor 26% gewijzigd door djluc op 19-09-2003 17:00 ]

[Limhes]

djluc schreef op 19 September 2003 @ 16:51:
Dit is een filehandler, daar hoeft dat niet volgens mij.

En nee, file() geeft een array terug...

[Verwijderd]

Mijn vraag is idd wat djluc zegt... waarom komt er niet mooi netjes een nieuwe regel...

[Suepahfly]

De fout zit em op regel 26.

Daar staat ``fp= fopen($logfile, 'w'); ''
Dit moet zijn fp= fopen($logfile, 'a');

PHP: fopen - Manual

[djluc]

Limhes schreef op 19 September 2003 @ 16:52:
[...]
En nee, file() geeft een array terug...

Ja, maar daar agat het niet om. PHP is gebaseerd op als het niet onwaar is is het waar. File geeft de waarde false terug als er iets niet goed ging, dat is dus de controle die er wordt uitgevoerd. Het is dus de bedoeling om de array die je terug krijgt in de var te plaatsen. Maar alleen als file goed gaat.

Suepahfly schreef op 19 September 2003 @ 16:54:
De fout zit em op regel 26.
Daar staat ``fp= fopen($logfile, 'w'); ''
Dit moet zijn fp= fopen($logfile, 'a');
PHP: fopen - Manual

Dat was idd wat ik bedoelde met:

In Perl kon je zoiets doen met >> in je handler o.i.d. Met PHP zou ik het echter niet weten.

[ Voor 33% gewijzigd door djluc op 19-09-2003 16:57 ]

[Limhes]

djluc schreef op 19 September 2003 @ 16:55:
[...]

Ja, maar daar agat het niet om. PHP is gebaseerd op als het niet onwaar is is het waar. File geeft de waarde false terug als er iets niet goed ging, dat is dus de controle die er wordt uitgevoerd. Het is dus de bedoeling om de array die je terug krijgt in de var te plaatsen. Maar alleen als file goed gaat.

offtopic:
Dat snap ik

Feit was alleen dat jouw uitleg niet correct was, en daar was mijn commentaar op gebaseerd

[djluc]

offtopic:
Er was helemaal geen uitleg, misschien dat je dat bedoeld ?

Er zijn trouwens wat betere manieren om het juiste ip-adres te achterhalen. Ook kun je eens kijken naar de mogelijkheden van sessions, zodat je de gebruikers misschien wat beter kunt onderscheiden. Na enkele jaren zal je file namelijk erg veel ip's bevatten die hergebruikt worden.

[MisterData]

getenv is trouwens redelijk vies, kun je beter de juiste var uit $_SERVER pakken of gewoon direct $REMOTE_ADDR

[Verwijderd]

ok.. bedankt het werkt nu..! het is gewoon om te kijken hoeveel mensen het downen en dan schrijf ik wel een apart progje om de log een beetje schoon en overzichtelijk te maken...