Fortlaufende Nummer/ID per Inserttag

[RoGer]

Hallo zusammen,

ich greife aus aktuellem Anlass nochmal diesen Thread auf.

Bei jeder Anmeldung soll eine fortlaufende, eindeutige und verständliche Zahl vergeben werden, ähnlich einem Besucherzähler.
Dazu habe ich die Dateien counter.txt und counter.php im \templates-Verzeichnis.
Das Anmeldeformular enthält ein verstecktes Feld, mit dem per Inserttag {{file::counter.php}} die Datei counter.php aufgerufen wird/werden soll

counter.txt

3

counter.php

<?php
$file = "templates/counter.txt";
$op = fopen($file,"r");
$counter = fread($op, 4);
fclose($op);

$counter++;

$op = fopen($file, "w");
  fwrite($op, $counter);
  fclose($op);
  echo $counter;
?>

Die Datei counter.txt wird offensichtlich ausgelesen, die Zahl wird allerdings nicht erhöht und bleibt bei "3" .
Sorry, meine PHP-Kenntnisse sind wirklich minimal. Hat jemand einen Hinweis auf den Fehler oder eine andere Lösung ?

Danke
Roland

[Andreas]

Du kannst das auch einfacher haben:

<?php

$file = 'templates/counter.txt';

$counter = file_get_contents($file);

var_dump($counter);

$counter++;

file_put_contents($file, $counter);

echo $counter;

?>

Ich habe dir einen var_dump() eingefügt, damit du prüfen kannst, ob der gelesene Inhalt auch ein Integer ist. Wenn nicht, dann mach einen Integer daraus.

Das Ganze wird aber am Ende sowieso nicht funktionieren, weil du schlecht prüfen kannst, wie oft dieses Script von wem aufgerufen wird.

Besser wäre vielleicht, wenn du bei einer Anmeldung nach den tatsächlichen Anmeldungen in der DB-table nachsiehst und dort in ein Feld in Übereinstimmung mit dem Anmeldezeitpunkt eine fortlaufende Nummer vergibst.

[RoGer]

Vielen Dank für die Antwort.

Ich habe dein var_dump($counter); durch $counter = (int)$counter; ersetzt und damit funktioniert der "Zähler". Eigentlich ist es egal, wenn die Zahlenreihe lückenhaft ist, solange keine doppelt vorkommt. Und das ist wohl ausgeschlossen.

Ich hatte es auch schon einfacher - doch leider fehlerhaft. Vielleicht erkennt jemand den Fehler:

 <?php
$counterFilePath = 'templates/counter.txt';

$counter = file_exists($counterFilePath) ? file_get_contents($counterFilePath) : 0) + 1;
file_put_contents($counterFilePath, $counter);

echo $counter;

?>

An eine Lösung mit der DB habe ich auch gedacht, aber leider nirgends die Einträge von efg gefunden, um die dort automatisch vergebene ID auszulesen. Das wäre das Einfachste !

Roland

[Andreas]

$file = 'templates/counter.txt';

$counter = @file_get_contents($file);

$counter = $counter ? $counter + 1 : 1;

file_put_contents($file, $counter);

echo $counter; 

file_get_contents() erzeugt eine E_WARNING, wenn die Datei nicht gefunden wird. Das passiert nur beim ersten Mal, weil file_put_contents() die Datei erzeugt, falls sie nicht existiert. Mit dem "@" kannst du diese erste Fehlermeldung unterdrücken.

Das (int) brauchst du nicht, wenn du "+ 1" anstelle von "++" verwendest - Type Juggling http://php.net/manual/de/language.ty...e-juggling.php

edit: In deiner langen Zeile ist ein Fehler mit dem Komma, ich finde mein Code ist auch einfacher zu lesen.