Hallo,

ich habe mal wieder ein Thread Problem

Damit mein Programm nicht hängt, während eingefügte Links auf ihre Gültigkeit überprüft werden, wird ein Thread gestartet, welcher diese Aufgabe übernimmt.
Seine Funktion an sich erfüllt der Thread sehr gut, allerdings lässt er sich nicht mehr stoppen sobald er am Werk ist :/

Ich versuche so den Thread zu stoppen:

markieren

Delphi-Quellcode:

LinkAdder.Terminate;
LinkAdder.WaitFor;
FreeAndNil(LinkAdder);

In der Hauptfunktion des Threads, die durch Execute() aufgerufen wird, läuft eine for-Schleife und bei jedem Link überprüfe ich, ob Terminated wurde. Das Problem ist, dass obwohl von außen Terminate aufgerufen wurde, ist Terminated im Thread, wenn ich es mir ausgeben lasse, noch False. Daher stoppt er nicht.

Weiß jemand woran das liegt und wie man es lösen könnte?

Vielleicht zeigt LinkAdder nicht auf den richtigen Thread?

markieren

Delphi-Quellcode:

while not LinkAdder.Terminated do
  LinkAdder.Terminate;
LinkAdder.WaitFor; // <-- Hier Breakpoint setzen und dann im Thread prüfen
FreeAndNil(LinkAdder);

Hi,

thread.terminate setzt nur ein Flag, ein while ist hier überflüssig.

ich würde mir gerne mal die Execute anschauen. Wahrscheinlich kommt die gar nicht zu der Abfrage des Flags.
Normalerweise hat man ja keine For Schleife im Thread sondern eine While Schleife:

markieren

Delphi-Quellcode:

   while not Terminated do
   begin
     // werkeln
   end;

Gruss

Zitat von Sir Rufo:

Vielleicht zeigt LinkAdder nicht auf den richtigen Thread?

Jetzt wo du es sagst, stimmt.
Ich habe mir mal die ThreadID ausgeben lassen:
Nach dem der Thread suspendiert erstellt wurde und .Start; aufgerufen wurde, ist sie 5012.
In der von Execute aufgerufen Methode ist sie 7700 - beim Beenden jedoch wieder 5012.

Erstellt der Thread automatisch noch einen anderen oder warum ist das so?

Hi,

das ist aber nicht normal.....
Zeig uns doch mal deine Execute Methode.

Ah ich habs.
Hatte die ganze Zeit gekuckt, ob irgendwo auf LinkAdder zugegriffen wird.
Allerdings wird in dem Teil, in dem die Links aus einer Datei geladen werden, schon ein Thread erstellt und dann eben noch einmal der "normale".

.. ist die execute Methode so geheim, dass Du sie nicht zeigen darfst?

Grüße
Klaus

Zitat von Klaus01:

.. ist die execute Methode so geheim, dass Du sie nicht zeigen darfst?

Grüße
Klaus

so geheim, er müsste uns schon erschiessen, wenn er nur drüber nachdenkt diese zu zeigen