Weiteres Problem mit selbstgeschriebenem Script

Hallo!

habe ein Problem mit folgendem Script:

PHP-Code:

$ip=getenv("REMOTE_ADDR") . "n";

$fp = fopen("ip.txt","r");
while(!feof($fp)){  
    $wert = fgets($fp,1024);  
    }
    fclose($fp);
    
if ($ip==$wert) {
    echo "Sie haben bereits abgestimmt!"; 


Das Ziel:
Das Script soll sich die aktuelle IP Adresse beschaffen.
Dann soll es die Datei ip.txt öffnen, sie durchgehen und die aktuelle IP mit denen in der Datei vergleichen.
Falls eine Übereinstimmung auftritt, soll es "Sie habe bereits abgestimmt" ausgeben.

Das Problem:
Das Script wird zwar scheinabar fehlerfrei ausgeführt, jedoch kommt es zu keinem Vergleich der IPs.

Wer weiss voran es liegen könnte?

[|Coding]

hi,

mach es so:

PHP-Code:

$ip=getenv('REMOTE_ADDR');
$fp = fopen('ip.txt', 'r');
while(!feof($fp)){
     $wert = fgets($fp, 1024);
     if($ip == $wert) echo 'Sie haben bereits abgestimmt!';
}
fclose($fp); 


Original geschrieben von |Coding
hi,

mach es so:

Hallo!

Danke für den Tipp, jetzt funktioniert die Sache.

Aber was genau habe ich nun falsch gemacht?
Man will ja auch für die Zukunft lernen und solche Fehler zukünfitg auch vermeiden.

[Bigdaddy]

ich denke es könnte daran liegen, dass du du mit fclose() geschlossen hast und die variablen die aus dem txt-file sind nicht mehr verglichen werden konnten...

Du kannst auch net eine karton äpfel öffnen, danach zumachen eine andere apfelsorte anschauen und danach noch sagen, ob es in der schachtel genau diese apfelsorte auch drin hat ;)

<-- keine ahnung ob das ne gute erklärung ist.... *g*

GREEZ

Original geschrieben von Bigdaddy


Du kannst auch net eine karton äpfel öffnen, danach zumachen eine andere apfelsorte anschauen und danach noch sagen, ob es in der schachtel genau diese apfelsorte auch drin hat ;)


GREEZ

Danke, sehr anschaulich erklärt. :-)

[Gweilo]

der grund ist, dass $wert immer geändert wird, und bei deinem alten script bloss die letzte ip - oder eventuell sogar gar nichts mehr (bzw n) darin gespeichert wird. Sie wird immer überschieben. Wenn du es in ein Array gespeichert hättest, hätte es funktionniert, aber in der schleife zu testen ist sowieso besser.