Thread-Sicherheit durch Thread-Safe-Liste gegeben?

[s.h.a.r.k]

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.

Delphi-Quellcode:

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

Nun gibts da auch die folgende Methode:

Delphi-Quellcode:

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

Nachdem hinter dem Aufruf von

Delphi-Quellcode:

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())

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;

[Sir Rufo]

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 ;)

[s.h.a.r.k]

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

[Sir Rufo]

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 :mrgreen:

[s.h.a.r.k]

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 :mrgreen: