[PHP] IP bezoeker vergelijken met IP's in banfile.txt

[GoodspeeD]

Ik heb een textbestand gemaakt, genaamd banfile.txt, waarin ik de IP's van gebruikers in heb gezet, waarvan ik niet wil dat ze een bepaalde actie uitvoeren. Om te bekijken of de bezoeker z'n IP voorkomt in het banfile.txt, heb ik een functie gemaakt, genaamd isbanned(), die 1 retourneert als het IP wel voorkomt in het bestand en 0 als het IP dus niet voorkomt.

Dit is de code voor die functie:

code:

function isbanned() {
    $banfile = "banfile.txt";
    $localip = $_SERVER['REMOTE_ADDR'];
    if (!file_exists("$banfile")) {
        $bfp = fopen("$banfile", "a");
        fclose($bfp);
    }

    $bfp = fopen("$banfile","r") or die ("error when opening banfile.txt");
    $buffer = fgets($bfp);
    while(!feof($bfp) && $buffer != $localip) {
        $buffer = fgets($bfp);
    }
    fclose($bfp);   
    if ($buffer == $localip) {
        return 1;
    }
    else {
        return 0;
    }
}

Het probleem is, dat wanneer ik m'n script uitvoer en de functie aangeroepen wordt, het script continu in het while loopje blijft hangen en ik na 10 seconden dus een foutmelding krijg, omdat het allemaal te lang duurt. Ik heb van alles bekeken, maar ik begrijp echt niet, waarom hij niet gewoon dat while loopje uitgaat.

Kent iemand dit probleem of ziet iemand wat ik fout doe????

Gr.

[whoami]

Ik stel voor dat je jouw functie gewoon eens debugged. That's part of the job.... Dan zie je zelf waar het misgaat en waarom, en kan je het eenvoudig aanpassen.

Het zal dus bij de conditie in uw while statement liggen. Niet moeilijk eigenlijk om te zien wat er fout is hoor....

[thomaske]

makkelijkere oplossing:

- lees banfile.txt in met file
- controleer array met in_array

[D2k]

en als je een grootte opgeeft bij die fgets?
ik heb het idee dat je steeds dezelfde regel uitleest zonder door je file te wandelen

[cosmoOo]

is && niet gewoon AND ?, je hebt toch OR nodig anders wacht je tot je het einde hebt bereikt en je het ip in de lijst ziet en dat gebeurd natuurlijk nooit

foutje

[GoodspeeD]

blegh. :-)

[GoodspeeD]

D2k schreef op 08 oktober 2002 @ 13:52:
en als je een grootte opgeeft bij die fgets?
ik heb het idee dat je steeds dezelfde regel uitleest zonder door je file te wandelen

fgets, leest toch als je niets opgeeft maximaal 1024 KB en tot aan een newline karakter? Vervolgens schuift de filepointer toch een regel verder of moet ik die handmatig verder schuiven?

[GoodspeeD]

whoami schreef op 08 oktober 2002 @ 13:50:
Ik stel voor dat je jouw functie gewoon eens debugged. That's part of the job.... Dan zie je zelf waar het misgaat en waarom, en kan je het eenvoudig aanpassen.

Het zal dus bij de conditie in uw while statement liggen. Niet moeilijk eigenlijk om te zien wat er fout is hoor....

Ik heb geen debuggingtooltje en ik heb zelf geen webserver met PHP, dus ik moet m'n scripts ook steeds uploaden naar de webserver van m'n provider. En als het niet moeilijk is om te zien wat er fout is, zou je mij dat dan aub kunnen zeggen??? Daar startte ik dit topic namelijk voor.

[whoami]

Ooit al eens van de edit-knop gehoord?

[D2k]

hmmz
krijg je geen false trug dan in $buffer?

[GoodspeeD]

thomaske schreef op 08 oktober 2002 @ 13:51:
makkelijkere oplossing:

- lees banfile.txt in met file
- controleer array met in_array

Spank you very much. Werkt perfect.