Ich habe noch nicht so viel mit Threads gemacht, deshalb brauche ich eure Meinung.
Ich habe eine Klasse die verschiedene Methoden hat, die länger dauern "können".
Die Klasse wurde bisher nie innerhalb eines Threads verwendet.
Im DoNotify kann alles mögliche aufgerufen werden. Auch Bildschirmaktualisierungen.
Der Code innerhalb der aufrufenden Methoden (LongTimePart1 etc.) ist threadsicher. Aber durch den Aufruf
von DoNotify kann ich die ganze Methode nicht als threadsicher ansehen. Daher sehe ich die ganze Klasse nicht als threadsicher an.
Nun möchte ich die Procedure LongTimePart1 innerhalb eine Threads aufrufen. Da LongTimePart1 nicht threadsicher ist, müsste ich
diese innerhalb eines Thread.Synchronize aufrufen. Dann würde aber auch "Mach Viel" im Hauptthread laufen.
Nun mein Gedankengang: Kann ich nicht einfach DoNotify threadsicher machen, in dem ich prophylaktisch dort Thread.Synchronize verwende?
Dann wäre die ganze Klasse threadsicher.
Nun kann es aber sein, dass die Procedure LongTimePart1 im Hauptthread aufgerufen wird. Nun weis ich nicht, ob es Probleme gibt, wenn Thread.Synchronize
nicht innerhalb eines Threads aufgerufen wird sondern im Hauptthread.
Wie ist eure Meinung?
Delphi-Quellcode:
// Dieser Code ist ein stark verkürzter Beispielcode
type
TWorkerTest = class(TObject)
private
FOnNotify: TNotifyEvent;
public
property OnNotify: TNotifyEvent read FOnNotify write FOnNotify;
Procedure DoNotify;
public
Procedure LongTimePart1;
Procedure LongTimePart2;
Procedure LongTimePart3;
end;
implementation
{ TWorkerTest }
procedure TWorkerTest.DoNotify;
begin
//******************************************
// bestehende Version - Nicht threadsicher
//
// if assigned(FOnNotify) then
// FOnNotify(self);
//******************************************
// neue Version - hoffentlich threadsicher
// Die Methoden, welche OnNotify verwenden,
// "können" aus einem Thread aufgerufen sein. Müsse aber nicht.
TThread.Synchronize(nil,
procedure
begin
if assigned(FOnNotify) then
FOnNotify(Self);
end);
end;
procedure TWorkerTest.LongTimePart1;
begin
// Mach viel. Threadsicherer Code.
DoNotify;
end;
procedure TWorkerTest.LongTimePart2;
begin
// Mach viel. Threadsicherer Code.
DoNotify;
end;
procedure TWorkerTest.LongTimePart3;
begin
// Mach viel. Threadsicherer Code.
DoNotify;
end;