Seltsamer Programmfehler (Delphi schließt sich)

[jaenicke]

Zitat von blackdrake:

Delphi-Quellcode:

  z := TMySendingThread.Create(true);
  z.FreeOnTerminate := true;
  z.Resume;

Lass den Blödsinn mit suspended erzeugen und Resume, dann wird es wahrscheinlich auch keine Probleme mehr geben...

FreeOnTerminate + Resume ist eine Kombination, bei der Probleme vorprogrammiert sind. Nicht ohne Grund ist Resume mittlerweile als deprecated markiert.
Eins der möglichen Probleme: Resume ist noch nicht fertig, wenn der Thread schon fertig ist und freigegeben wird. Rumms...

[Furtbichler]

Also ich halte das nicht für Blödsinn, denn nichts anderes passiert, wenn man den Thread mit 'CreateSuspended=False' instantiiert: Er wird erst 'Suspended' erzeugt (in jedem Fall) und im AfterConstruction ggfs. per 'Resume' aufgeweckt.

Die Resume-Methode ist deprecated, weil viele Programmierer damit Threads zwischendurch anhalten bzw. wieder weiterlaufen lassen. Und DAS ist eine schlechte Idee.

Edit:... oh, es ist doch nicht geschickt, so wie es umgesetzt ist, ich bezweifle aber, das das Probleme bereitet.

Ich mache das mit dem Initialisieren immer im abgeleiteten Konstruktor:

Delphi-Quellcode:

Constructor TMyThread.Create;
Begin
  inherited Create(True);
  InitializeMyThreadStuff();
  Resume;
End;

und das ist so überflüssig (aber deklarativ).

[himitsu]

Zitat von Furtbichler:

Ich mache das mit dem Initialisieren immer im abgeleiteten Konstruktor:

Delphi-Quellcode:

Constructor TMyThread.Create;
Begin
  inherited Create(True);
  InitializeMyThreadStuff();
  Resume;
End;

Wozu das Resume?
Der Thread wird doch sowieso immer erst nach der Create-Methode gestartet. (wenn man es nicht suspended startet)

[Furtbichler]

Hi,
Eben. Ich schrieb ja, ...

Zitat von Furtbichler:

...das ist so überflüssig (aber deklarativ).

Mit "deklarativ" meine ich, das der Code beschreibt, was (sowieso im Hintergrund) geschieht.

Und, ich muss zu meiner Schande gestehen, das ich eben erst (durch Studium des Classes-Quellcodes) entdeckt habe, das meine Vorgehensweise gut gedacht aber vollkommen überflüssig ist.

Los, nun hack noch weiterauf meinem Unvermögen rum.
Es regnet sowieso.
Das Wochenende ist versaut.

Muhuuuhhhuuuu.

[himitsu]

Soll ich?

Neuerdings soll man TThread.Start und nicht TThread.Resume verwenden.



Ach ja nachträglich den Thread über diese Methoden zu Pausieren ist auch verboten.



Nja, zumindestens ist es nicht unbedingt "schlimm", wenn man einen Thread suspendet startet ... es ist nur "unschön".

[mjustin]

Zitat von Furtbichler:

Die Resume-Methode ist deprecated, weil viele Programmierer damit Threads zwischendurch anhalten bzw. wieder weiterlaufen lassen.

Sie ist auch deprecated weil es beim Aufruf von Resume eventuell keinen Thread mehr gibt. Beim ersten Start eines Threads der im Suspended-Modus erzeugt wurde ist ein Resume noch sicher - später aber nicht mehr.

Die Forumsdiskussion dazu unter https://forums.embarcadero.com/messa...messageID=3466 ist leider nicht mehr vorhanden.

Siehe auch: TThread.resume is deprecated in Delphi-2010 what should be used in place?