Tom: Mein erstes PHP Gebastel

Beitrag lesen

Hello,

ich habe eben nur malö schnell durchgeschaut und festgestellt, dass Du den Fehler machst, den fast alle PHP-/ und/oder Programmieranfänger machen. Du trennst die Bereiche des Programms nicht voneinander:

  • Datenübernahme
  • Daten Plausibilitätskontrolle
  • Steuerflussauswertung
  • Datenbeschaffung
  • Datenverarbeitung
  • Daten wegschreiben
  • Ausgabe aufbereiten
  • ausgeben

oder kurz "EVA" für Eingabe, Verarbeitung, Ausgabe.

Und dieser Abschnitt fiel mir sofort auf:

$filename = "datafile.csv";
    $file = file($filename);
    $last_line = end($file);
    $new_line = sprintf("%04d",$last_line+1).",".$txt."\n";

$finsert = fopen($filename , "a+");
    fwrite($finsert , $new_line);
    fclose($finsert);

Da geht 'was in die Hose, enn mehrere Besucher gleichzeitig zugreifen.
Außerdem ist die Vermischung von namensbasierten und handlebasierten Dateifunktionen nicht zu empfehlen. Für sicheres Datenhandling solltest Du dich auf die handlebasierten Funktionen beschränken. Alles andere ergibt irgendwann Probleme.

Schau Dir dazu den Artikel von Christian an:
http://aktuell.de.selfhtml.org/artikel/programmiertechnik/dateisperren/index.htm

Liebe Grüße aus Syburg bei Dortmund

Tom vom Berg

--
Nur selber lernen macht schlau
http://bergpost.annerschbarrich.de