Habe da im Moment eine Frage, bei der ich nicht so recht weiß, ob das Thread-Safe ist oder nicht. Ich habe eine eigens erstellte Thread-sichere Klasse TApLogThreadedBuffer<T: class>, von der eine Instanz in einer anderen Klasse gekapselt wird.

markieren

Delphi-Quellcode:

TApLogErrorHandler = class(TApThreadedObject)
  FItems : TApLogThreadedBuffer<TApLogErrorMessage>;
  ...

Nun gibts da auch die folgende Methode:

markieren

Delphi-Quellcode:

function TApLogErrorHandler.GetErrorCount(): Integer;
begin
  Lock();
  try
    Result := FItems.Count;
  finally
    Unlock();
  end;
end;

Nachdem hinter dem Aufruf von FItems.Count ja eh schon eine Thread-sichere Getter-Methode steckt könnte ich doch an dieser Stelle auf Lock() und Unlock() verzichten, oder?

PS: Lock() und Unlock() in der gezeigten Methode ruft eine andere CriticalSection auf, als die der FItems-Instanz. Wollte das nur noch geklärt haben

-- Edit: Noch ein kleiner Nachtrag: Trifft das gleiche dann auch hier zu? (hier nun ohne Lock() und Unlock())

zusammenfalten · markieren

Delphi-Quellcode:

function TApLogErrorHandler.GetItem(Index: Integer): TApLogErrorMessage;
begin
  Result := FItems[Index];
end;

function TApLogErrorHandler.GetMaxBufferElements(): Integer;
begin
  Result := FItems.MaxElements;
end;

procedure TApLogErrorHandler.SetMaxBufferElements(Value: Integer);
begin
  FItems.MaxElements := Value;
end;

Das kommt darauf an ob FItems verändert werden kann.
Somit ist das Lock nicht für den Zugriff auf Count (das sollte sich dann selber locken) sondern für den Zugriff auf FItems

FItems selbst ist komplett gekapselt und wird daher nicht verändert, d.h. es wird im Konstruktor erzeugt und im Destruktor wieder freigegeben.

Dann sollte das auch ohne Lock funktionieren.

Ein Problem könnte theoretisch noch dann auftauchen, wenn das Objekt freigegeben wird und von einem anderen Thread darauf zugegriffen wird.
Ist aber auch ein sehr theoretischer Fall

An der Stelle achte ich immer brav darauf, dass ich es nur an einer Stelle und wirklich ganz am Schluss freigebe Aber Danke für deinen Hinweis! Bzgl. Threads bin ich bisher in noch keine all zu großen Probleme rein gelaufen, vor allem nicht in solche, daher kommt die dezente Unsicherheit im Moment