TMessageManager/SendMessage

[TigerLilly]

Ich habe in meinem Formular dieses, um Messages zu empfangen + als Status ausgeben zu können:

Delphi-Quellcode:

 MessageManager := TMessageManager.DefaultManager;
  SubscriptionId := MessageManager.SubscribeToMessage(TMessage<UnicodeString>,
    procedure(const Sender: TObject; const M: TMessage)
    begin
      lblStatus.Text := (M as TMessage<UnicodeString>).Value;
    end);

Eine Methode, die länger braucht, wird so aufgerufen:

Delphi-Quellcode:

    TTask.Run(procedure()
    begin
      LoadXML(dlgOpenXML.FileName);
    end);

Im LoadXML wird so der Status versandt:
Die Nachricht wird so versandt:
    TMessageManager.DefaultManager.SendMessage(nil, TMessage<UnicodeString>.Create(xNode.Text), True);
Das führt aber nach ein paar 100 Nachrichten zu einer Zugriffsverletzung. Übersehe ich da etwas?

[TurboMagic]

1. Zugriffsverletzung an welcher Stelle?

2. Soweit mir bekannt arbeitet dieses Messagjng synchron,
wenn du also im Empfangsevent der Nachricht mit GUI
Controls interagierst und die Message von einem Thread
verschickt wurde muss es Mal knallen.

[TigerLilly]

Die Zugriffsverletzung entsteht irgendwo in der RTL bei den Threads. Ohne Sender Message läuft das ohne Probleme.

In einem anderen Thread
https://www.delphipraxis.net/202390-...alisieren.html

wurde dieses Vorgehen vorgeschlagen.

[TiGü]

Delphi-Quellcode:

 MessageManager := TMessageManager.DefaultManager;
  SubscriptionId := MessageManager.SubscribeToMessage(TMessage<UnicodeString>,
    procedure(const Sender: TObject; const M: TMessage)
    begin
      if TThread.Current.ThreadID <> MainThreadID then
        raise EProgrammerNotFound.Create('Ich komme aus dem Thread-Kontext des Tasks, also muss ich den Zugriff auf das Label lblStatus im Mainthread synchronisieren!');

      lblStatus.Text := (M as TMessage<UnicodeString>).Value;
    end);

[TigerLilly]

Das heisst, das Problem ist, dass ich hier auch auf Messages reagiere, die gar nicht von mit selber komen?

Ich müsste also TMessage<TnurVonMirString> registrieren und alles ist gut?

[TiGü]

Nein, dass ist die völlig falsche Schlussfolgerung.

Du hast einen Workerthread, in dessen Kontext die Task ausgeführt wird.
Innerhalb dieses Kontextes greifst du auf ein VCL-TLabel zu.
Auf VCL-Komponenten darf nur innerhalb des Mainthreads zugegriffen werden.
Das weißt du, dass wird hier auch an jeder Ecke oft erklärt.
In dem von dir verlinkten Thread hast du die richtige Vorgehensweise mit TThread.Synchronize schon drin, einfach wieder mit übernehmen.

[TigerLilly]

FMX, nicht VCL.

Ich greife ja nicht im gliechen Kontext auf den Label zu. Ich habe einen Task, der tut was und versendet Messages.
Mein MainThread ist derweil im Leerlauf und arbeitet die messages ab. Synchronize braucht es da nicht.

PS: Eigentlich wollte ich auf diesen Thread verlinken:
https://www.delphipraxis.net/202388-...alisieren.html

[TiGü]

Zitat von TigerLilly:

FMX, nicht VCL.

Ich greife ja nicht im gliechen Kontext auf den Label zu. Ich habe einen Task, der tut was und versendet Messages.
Mein MainThread ist derweil im Leerlauf und arbeitet die messages ab. Synchronize braucht es da nicht.

PS: Eigentlich wollte ich auf diesen Thread verlinken:
https://www.delphipraxis.net/202388-...alisieren.html

FMX oder VCL spielt keine Rolle, für beide gelten die gleichen Regeln.

Wenn dem so ist, wie du glaubst und behauptest, dann könntest du ja einfach meine If-Abfrage mit der EProgrammerNotFound-Exception einfügen.
Wenn die Exception kommt, dann hast du einen Denkfehler und musst das einfach umsetzen, was dir schon empfohlen wurde.
Wenn die Exception nicht kommt, dann enthältst du uns Informationen vor, die dir zwar klar sein mögen, aber uns aufgrund des Nichtvorhandenseins des Quelltextes der Funktion LoadXML() nicht ersichtlich sind.

PS: Im neuen verlinkten Thread wird von Mavarik in der Methode TForm319.UpdateUI auch mithilfe von TThread.Queue() aus dem Thread-Kontexts des Tasks heraus in den Mainthread synchronisiert.

[TiGü]

Um nochmal sicher zu gehen - weil ich hier eine Wissenslücke vermute - wenn du TTask.Run() schreibst, machst du einen Thread auf (bzw. benutzt einen vorhandenen Workerthread) der UNGLEICH des Mainthreads ist.

Wenn du darin Dinge tust, wie zum Beispiel den TMessageManager zu benutzen, der dann für die jeweilige Message die angemeldete Callback/Event-Methode aufruft und in der auf visuelle Komponenten zugegriffen werden, dann ist das falsch!

Der TMessageManager ist kein abgekoppeltes Gelumpe wie die Windows PostMessage-Funktion.
Das ist ein ganz schnödes Observer-Pattern.
Du musst als Benutzer/Client dafür sorgen, dass alles in seinen richtigen Threadkontext passiert.

[Rollo62]

Delphi-Quellcode:

  // in TTask.Run
  // 

  ...
  LTxt := xNode.Text;

  TThread.Queue(
    nil,
    procedure
    begin
         TMessageManager.DefaultManager.SendMessage(nil, TMessage<UnicodeString>.Create(LTxt) );
    end);

   ...