Codefehler ?

[DANP]

Hallo,
ich habe mir einen kleinen Code zusammengeschrieben welcher:
-Einen String empfängt
--Prüft ob der String in xxxx.txt steht
---Falls Ja, ein echo zurück gibt
---Falls Nein, den String in die xxxx.txt schreibt

<?php
$text = $_GET["hwid"];
 $datei = fopen("hwid.txt", "a+"); 
$file = file('hwid.txt'); 
   if (!$datei) {
      echo "Die Datei konnte zum Schreiben nicht geöffnet werden.";
   } else {
       if (searchhwid())
       {
           $correct = "HWID wurde gefunden.";
           echo $correct;
       }
       else
       {
           fwrite($datei, "$text\n");
           fclose($datei);
           echo "Datei erfolgreich beschrieben.";
       }
   }
   
function searchhwid()
{ 
    foreach ($file as $line) { 
        $fields = explode(' ', $line); 
        if($fields[2] == $text)
        {
            return true;
        }
        else {
            return false;
        }
        
    }
}

?>

Leider funktioniert der Code nicht so ganz. Er schreibt jedes mal den String neu in die Textdatei (sogar doppelt). Sie schaut dann beim Aufrufen so aus: (2 mal aufgerufen)

text1
text1
text2
text2
...

Bitte um Hilfe

[Comu]

Umständlich gemacht. Erstmal nen paar Schönheits / Vebesserungsfehler:

       else
       {
           fwrite($datei, "$text\n");
           fclose($datei);
           echo "Datei erfolgreich beschrieben.";
       } 

Ist völlig unlogisch. Was passier denn, wenn das fwrite() failed? Von daher:

else
{
  if(fwrite($datei,$text.'\n');
  {
     fclose($datei);
     echo "...";
   }
} 

Wie du oben noch siehst, stimmt da auch was mit der Variable und dem Text nicht.

Eventuell behebt das eins deiner Probleme schon, und das zweite sieht man hier (glaub ich jedenfalls):

        $fields = explode(' ', $line); 

\n erzeugt einen Zeilenumbruch. Deshalb verstehe ich nicht, warum du den Text dann mit ' ' teilst, und nicht mit \n\r . (Ausser du hast und nen Teil verschwiegen )

[Lyra]

\r\n nicht \n\r !

[DANP]

<?php
$text = $_GET["hwid"];
$datei = fopen("hwid.txt", "a+");
$file = file('hwid.txt'); 

function searchhwid($needle)
{
    $content = file("hwid.txt"); //Der ganze Inhalt in das Array!
    if(strstr($content, $needle) == FALSE)
    {
        return false;
    }
    else
    {
        return true;
    }
}  

if (!(searchhwid($text)))
    {
        fwrite($datei, "$text\n");
        fclose($datei);
        echo "Datei erfolgreich beschrieben.";
    }
else
    {
         echo "HWID wurde gefunden.";
    }


?>

Habe nun die Funktion ersetzt (bzw ersetzen lassen), allerdings zeigt er mir dennoch immer Datei erfolgreich beschrieben an, und nie "Hwid gefunden.." obwohl die hwid eingetragen ist...

[Lyra]

function searchhwid($needle)
{
    return strstr(file("hwid.txt"), $needle);
}

[DANP]

Habe ich ersetzt, allerdings gibt er mir immernoch nur "Datei erfolgreich beschrieben." aus und schreibt den get nochmal in die Datei

[Sky.NET]

eine allgemeine Frage: warum speicherst du das in einer Textdateio und nicht in einer Datenbank?

[DANP]

Da die Version mit der Datenbank bereits Funktioniert .

Aber möchte wissen woran das liegt das das Script nicht funktioniert ...

[Lyra]

function searchhwid($needle)
{
    return strpos(file_get_contents("hwid.txt"), $needle);
} 

[DANP]

Leider Funktioniert es immernoch nicht. Würde es vllt einen Unterschied machen, wenn man die Textdatei außerhalb der FUnktion in das Array lädt?