AGB  ·  Datenschutz  ·  Impressum  







Anmelden
Nützliche Links
Registrieren
Thema durchsuchen
Ansicht
Themen-Optionen

Anfängerfragen zu Threads (TThread)

Ein Thema von Dalai · begonnen am 3. Nov 2017 · letzter Beitrag vom 5. Nov 2017
 
Aviator

Registriert seit: 3. Jun 2010
1.611 Beiträge
 
Delphi 10.3 Rio
 
#13

AW: Anfängerfragen zu Threads (TThread)

  Alt 5. Nov 2017, 11:09
Hallo Dalai,

das sieht doch soweit ganz gut aus. Eine Sache die mir noch aufgefallen ist wäre folgende (das ist aber schon meckern auf ganz hohem Niveau):

Du durchläufst deine Execute Methode und feuerst zum Schluss das OnPingDone Event. Du rufst aber zuerst Synchronize auf und prüfst dann, ob das Event überhaupt abonniert wurde. Somit wartet der Thread zuerst, ob er sich mit dem Main Thread Synchronisieren kann. Und das kann Zeit kosten. Ich würde es umgekehrt machen. Das spart zwar nur ein paar Nano-/Milisekunden, aber wenn du dem Thread jetzt die Aufgaben geben würdest in einer Schleife mehrere Hosts abzufrühstücken, dann würde sich das u.U. bemerkbar machen.

Also könntest du aus deinem Code

Delphi-Quellcode:
procedure TPingThread.Execute;
begin
  if Assigned(FClient) then
    ReturnValue:= Integer(FClient.IsAlive);
  if NOT Terminated then
    Synchronize(CallOnPingDone);
end;

procedure TPingThread.CallOnPingDone;
begin
  if Assigned(FOnPingDone) then
    FOnPingDone(Self);
end;
folgenden Code machen:

Delphi-Quellcode:
procedure TPingThread.CallOnPingDone;
begin
  if Assigned(FOnPingDone) then
    FOnPingDone(Self);
end;

procedure TPingThread.Execute;
begin
  if Assigned(FClient) then
    ReturnValue:= Integer(FClient.IsAlive);
  if NOT Terminated then begin
    if Assigned(FOnPingDone) then begin
      Synchronize(CallOnPingDone);
    end;
  end;
end;


Oder gleich mit einer anonymen Methode das Synchronize aufrufen. Dann sparst du dir auch noch die Deklaration der CallOnPingDone Methode.

Delphi-Quellcode:
procedure TPingThread.Execute;
begin
  if Assigned(FClient) then
    ReturnValue:= Integer(FClient.IsAlive);
  if NOT Terminated then begin
    if Assigned(FOnPingDone) then begin
      Synchronize
      (
        procedure
        begin
          FOnPingDone(Self);
        end;
      );
    end;
  end;
end;

EDIT: Die Methode sind jetzt nur im Editor runtergetippt, sollten aber hoffentlich fehlerfrei sein. So viel habe ich jetzt ja auch nicht geändert.

Geändert von Aviator ( 5. Nov 2017 um 11:10 Uhr) Grund: Bemerkung hinzugefügt
  Mit Zitat antworten Zitat
 


Forumregeln

Es ist dir nicht erlaubt, neue Themen zu verfassen.
Es ist dir nicht erlaubt, auf Beiträge zu antworten.
Es ist dir nicht erlaubt, Anhänge hochzuladen.
Es ist dir nicht erlaubt, deine Beiträge zu bearbeiten.

BB-Code ist an.
Smileys sind an.
[IMG] Code ist an.
HTML-Code ist aus.
Trackbacks are an
Pingbacks are an
Refbacks are aus

Gehe zu:

Impressum · AGB · Datenschutz · Nach oben
Alle Zeitangaben in WEZ +1. Es ist jetzt 10:21 Uhr.
Powered by vBulletin® Copyright ©2000 - 2025, Jelsoft Enterprises Ltd.
LinkBacks Enabled by vBSEO © 2011, Crawlability, Inc.
Delphi-PRAXiS (c) 2002 - 2023 by Daniel R. Wolf, 2024-2025 by Thomas Breitkreuz