File Locking, Fehlerbehandlung

[Tom]

Hello Dennis,

ich habe gerade mal die Scripte überflogen und mir ein paar Gedanken dabei gemacht.

Im Prinzip könnte man die Lesbarkeit des Quelltextes noch verbessern ;-)

...und eine etwas bessere Aufteilung in Funktionen ist empfehlenswert. Das soll man zwar auch nicht übertragen, weil jedes Load einer Funktion auch teuer ist, aber die Fehlerbehandlung wird dadurch vereinfacht und übersichtlicher. Bei fehler wird einfach der Funktionsblock beendet mit Fehlernummer; sonst benötigt man meistens komplizierte Bedingungs-Verschachtelungen.

Und ein Denkfehler ist mir noch aufgefallen, der auch unbedingt in das wachsende Artikel-Konzept rein muss:

Thema File-Locking:
-------------------

if($fp)
{
  for($x=0;$x<5;$x++)
  {
    if(flock($fp,LOCK_SH)) break;
    usleep(8000);
  }

...

}
-------------------

Die Locking-Funktion hat eine eigene eingebaute Idle-Funktion. Wenn man flock() aufruft, bleibt die Kontrolle solange in der Funktion, bis der Lockversuch Erfolg hatte, oder die maximale Script-Laufzeit erreicht ist.

Wenn man nun den User nicht solange warten lassen will (das sind dann bei Default 30 Sekunden), baut man eben eine eigene Schleifenkonstruktion um den Lockverssuch gherum, so wie Du das getan hast. Allerdings muss man der Funktion dann sagen, dass sie nicht mehr automatisch solange warten soll, bis es passt...

--------------------------
if($fp)
{
  $lock = false;        # Kontrollschalter, ob Locking klappt
  for($x=0;$x<5;$x++)
  {
    $lock = (flock($fp,LOCK_SH + LOCK_NB));
    if ($lock) break;
    usleep(8000);
  }

if ($lock)
  {
    # ...
  }
  else
  {
    # Fehlerbehandlung
  }

fclose($fp);
}

---------------------------

Durch das Addieren des NonBlock-Bits weiß die Funktion nun, dass sie nicht warten soll, sondern false zurückgeben soll, wenn das Handle mit LOCK_EX belegt ist. Theoretisch ist es zu diesem Moment auch möglich, dass die Datei gar nicht mehr da ist. Das bedeutet, dass man alle Operationen mit dem Handle also trotzdem unter Fehlerbehandlung stellen muss.

Außerdem sit es wenig sinnvoll, normal weiter zu arbeiten, wenn das Locking nicht geklappt hat. Also muss man sich den Erfolg merken und nochmals abprüfen.

Im Prinzip sind "echte" Funktionen für den Computeralltag ungeeignet, denn jede Funktion müsste mindestens zwei Rückgabewerte haben:

1. Funktionsergebnis
2. Fehlernummer

Dafür muss also für größere Projekte ein einheitliches Konzept her. Ich habe mir angewöhnt (bzw. bin noch dabei), das Arbeitsergebnis der Funktion über einen Call-by-Reference zurückzugeben und die Fehlernummer als herkömmliches Funktionsergebnis. Das hat bei den klassischen Programmiersprachen auch den Vorteil, dass man variante Records als Arbeitsergebnis nutzen kann, und so die Haltbarkeit des Codes (Abwärtskompatibilität) erhöht.

Soviel zum Thema Locking und Fehlerbehandlung.
Ich bitte höflichst um Korrektur durch unsere Regulars... ;-)

Liebe Grüße aus http://www.braunschweig.de

Tom