Ist die Verwendung von TTask so korrekt?

[t.roller]

Verwenden von TTask aus der Parallel Programming Library

[a.def]

Zitat von t.roller:

Verwenden von TTask aus der Parallel Programming Library

Abschnitt Eine weitere Funktionalität von TTask ist das Verhindern der Sperrung der Benutzeroberfläche, wenn Sie im Hintergrund etwas starten möchten. Im folgenden Beispiel wird gezeigt, wie Sie eine einzelne Aufgabe ausführen und starten
Danke! TTask benötigt also kein Synchronize.

P.S.:
ich glaube da ist ein Tippfehler in

Zitat:

WaitForAll informiert Sie über den Abschluss aller Aufgaben und WaitForAny über die erste Aufgabe, die abgeschossen wurde.

Ich glaube es sollte abge(l)schossen heißen.

[Uwe Raabe]

Zitat von a.def:

Danke! TTask benötigt also kein Synchronize.

Wo steht das?

TTask macht einen separaten Thread auf. Der Code, der an TTask übergeben wurde, läuft dann in diesem Thread. Das heißt aber nicht, daß Zugriffe auf die VCL nicht doch synchronisiert werden müssen - so wie in jedem anderen Thread auch.

Dein Beispiel ist irgendwie Nonsens und deswegen liefert es auch keine vernünftigen Ergebnisse. Wenn du alle 10 ms einen Zähler hochzählst und den dann in der Caption darstellst, dann musst(!) du dafür Synchronize oder Queue verwenden. Damit zwingst du den Hauptthread aber alle 10 ms diese Caption zu aktualisieren und das bremst natürlich. Solange also die Aufgabenstellung besagt, daß alle 10 ms die Caption aktualisiert werden muss, kann auch die Verwendung eines Threads den Hauptthread nicht wirklich entlasten.

Threads sind nicht dazu da, viele Zugriffe auf die VCL auszuführen, sondern längere Sequenzen, die nicht mit der VCL rumhantieren, aus dem Hauptthread zu extrahieren, damit die Anwendung weiter auf Benutzereingaben reagieren kann. Das Extrahieren erfordert manchmal schon etwas mehr Überlegung, damit das Ganze auch threadsicher wird.

Stell dir einfach vor, du müsstest eine solche Code-Sequenz als separate Kommandozeilenanwendung schreiben, bei der du Readln und Writeln nur bei Programmanfang und -ende verwenden darfst. Alles dazwischen muss ohne Bildschirmausgaben und Tastatureingaben auskommen. Dieses dazwischen könnte man dann wohl in einen Thread oder auch eine TTask verlagern, um das in einer responsiven UI-Anwendung auszuführen. Daraus folgt dann implizit, daß man in Threads nur sehr sparsam mit Statusmeldungen arbeiten sollte, da diese zwangsweise irgendwie an die UI und somit an den Hauptthread übergeben (synchronisiert) werden müssen.

[a.def]

Zitat von Uwe Raabe:

Zitat von a.def:

Danke! TTask benötigt also kein Synchronize.

Wo steht das?

Zitat:

Eine weitere Funktionalität von TTask ist das Verhindern der Sperrung der Benutzeroberfläche, wenn Sie im Hintergrund etwas starten möchten. Im folgenden Beispiel wird gezeigt, wie Sie eine einzelne Aufgabe ausführen und starten

Für mich bedeutet das, dass man es nicht benötigt. Selbst im offiziellen Emba-Beispiel wird kein Synchronize verwendet.

Mein Beispiel ist ja auch nonsense, keine Frage. Ist ja auch nur ein Beispiel zu einer dummen Frage.
Die eigentliche Arbeit meiner TTasks besteht aus ganz anderen Sachen und nicht so etwas Dummes wie "Caption alle 10ms verändern".

Zitat:

(von Emba)
Eine weitere Funktionalität von TTask ist das Verhindern der Sperrung der Benutzeroberfläche, wenn Sie im Hintergrund etwas starten möchten. Im folgenden Beispiel wird gezeigt, wie Sie eine einzelne Aufgabe ausführen und starten:
Delphi:

Delphi-Quellcode:

 
procedure TFormThreading.Button1Click(Sender: TObject);
var
 aTask: ITask;
begin
 aTask := TTask.Create (procedure ()
   begin
     sleep (3000); // 3 seconds
     ShowMessage ('Hello'); // <= kein Synchronize
   end);
 aTask.Start;
end;

[Uwe Raabe]

Zitat von a.def;1357704[QUOTE:

(von Emba)
Eine weitere Funktionalität von TTask ist das Verhindern der Sperrung der Benutzeroberfläche, wenn Sie im Hintergrund etwas starten möchten. Im folgenden Beispiel wird gezeigt, wie Sie eine einzelne Aufgabe ausführen und starten:
Delphi:

Delphi-Quellcode:

 
procedure TFormThreading.Button1Click(Sender: TObject);
var
 aTask: ITask;
begin
 aTask := TTask.Create (procedure ()
   begin
     sleep (3000); // 3 seconds
     ShowMessage ('Hello'); // <= kein Synchronize
   end);
 aTask.Start;
end;

Ich wage mal zu behaupten, daß das Beispiel dort schlichtweg falsch ist. In den Tiefen von ShowMessage wird irgendwann ein Application.ModalStarted und ein Application.ModalEnded aufgerufen. Darin ist das Verändern von FModalLevel schon mal nicht wirklich threadsicher. Weiterhin wird gegebenfalls ein OnModalBegin bzw. OnModalEnd aufgerufen (auch nicht auf thread-sichere Weise). Sind diese Events verlinkt, wird der dortige Code im Thread-Context aufgerufen, was der Event-Verlinker vermutlich gar nicht auf dem Schirm hat. Das obige Beispiel in einer hinreichend komplexen Anwendung führt zu sporadischen Fehlfunktionen, die nur sehr schwer zu analysieren sind.

Grundsätzlich würde ich mich nicht darauf verlassen, daß der Code in den Beispielen für den produktiven Einsatz geeignet ist. Auch TTask ist keine Magie, die plötzlich Thread-Programmierung zu einem Kinderspiel macht.

Um den Beispielcode kümmere ich mich noch. Wenn jemand in C++ fit ist, wäre ich für entsprechende Hinweise zu dem dort ebenfalls vorhandenen und potentiell ebenso falschen C++-Code dankbar.

[DeddyH]

Zitat von Uwe Raabe:

Grundsätzlich würde ich mich nicht darauf verlassen, daß der Code in den Beispielen für den produktiven Einsatz geeignet ist. Auch TTask ist keine Magie, die plötzlich Thread-Programmierung zu einem Kinderspiel macht.

Diese Worte sollte man sich in Stein meißeln

[t.roller]

Zitat von a.def:

Zitat von t.roller:

Verwenden von TTask aus der Parallel Programming Library

Abschnitt Eine weitere Funktionalität von TTask ist das Verhindern der Sperrung der Benutzeroberfläche, wenn Sie im Hintergrund etwas starten möchten. Im folgenden Beispiel wird gezeigt, wie Sie eine einzelne Aufgabe ausführen und starten

Beispiel:

Delphi-Quellcode:

// uses System.Threading,...
procedure TForm1.ButtonClick(Sender: TObject);
var
  lValue: Integer;
begin
    Label1.Caption := '--';
    CheckBox1.Checked:= TRUE;
    TTask.Run(procedure
      begin
          {Some calculation that takes time}
          Sleep(3000);
          lValue := Random(100);
          TThread.Synchronize(nil,
            procedure
            begin
                  CheckBox1.Checked:= FALSE;
                  Label1.Caption := lValue.ToString;
            end);
      end);
//-------------------------------------------------------------------
    Label2.Caption := '--';
    CheckBox2.Checked:= TRUE;
    TTask.Run(procedure
      begin
          {Some calculation that takes time}
          Sleep(6000);
          lValue := Random(100);
          TThread.Synchronize(nil,
            procedure
            begin
                  CheckBox2.Checked:= FALSE;
                  Label2.Caption := lValue.ToString;
            end);
      end);
//-------------------------------------------------------------------
    Label3.Caption := '--';
    CheckBox3.Checked:= TRUE;
    TTask.Run(procedure
      begin
          {Some calculation that takes time}
          Sleep(9000);
          lValue := Random(100);
          TThread.Synchronize(nil,
            procedure
            begin
                  CheckBox3.Checked:= FALSE;
                  Label3.Caption := lValue.ToString;
            end);
      end);
//-------------------------------------------------------------------
end;

[HolgerX]

Hmm..

Zitat:

Zitat:

Verwenden von TTask aus der Parallel Programming Library

Abschnitt Eine weitere Funktionalität von TTask ist das Verhindern der Sperrung der Benutzeroberfläche, wenn Sie im Hintergrund etwas starten möchten.

Da steht nur das die Benutzeroberfläche NICHT gesperrt wird!
Somit läuft etwas im Hintergrund (Thread) ohne das der MainThread und somit die Benutzereingabe gesperrt wird.
Sprich, dass wenn Du z.B. auf einem Button.Click eine laaaaannngggeee Verarbeitung machts, dann ist die Oberfläche gesperrt. Selbst Aktualisierungen werden ohne ProzessMessages nicht mehr auf z.B. Edits angezeigt.

[t.roller]

Zitat von HolgerX:

Sprich, dass wenn Du z.B. auf einem Button.Click eine laaaaannngggeee Verarbeitung machts, dann ist die Oberfläche gesperrt. Selbst Aktualisierungen werden ohne ProzessMessages nicht mehr auf z.B. Edits angezeigt.

Das ist falsch.

Füge doch meinem Beispiel in #10
das hinzu:

Delphi-Quellcode:

procedure TForm1.Button3Click(Sender: TObject);
begin
Button3.Tag:= Button3.Tag +1;
Label4.Caption:= INTTOSTR(Button3.Tag);
end;

Das bleibt bedienbar.

[Daniel]

Fettschrift hilft da auch nicht, um die eigene Position zu unterstreichen. Wirklich.
In Deinem Beispiel in Beitrag #10 wird die langlaufende Operation (im Beispiel das Sleep) ja in dem Thread ausgeführt. Die Synchronisation mit dem MainThread ist immer nur sehr kurz. Es ist klar, dass das bedienbar bleibt.
Setze das Sleep mal in das OnClick-Ereignis. Dann legt sich auch die GUI zur Ruhe nieder.