TIdTCPServer (Indy) und TClientSocket

**Schwedenbitter**

Daher musst du auch ein "Satz" mit CR/LF schicken. Also noch ein +#13#10 dranhängen.

Volltreffer! Tausend Dank.
Warum das so ist, verstehe ich trotzdem nicht so ganz. TClientSocket sendet etwas ab, was z.B. TServerSocket auch ohne #13#10 empfangen würde. Dass ich mit ReadLn(); auslese, ist korrekt. Aber das Ereignis OnExecute wird nach meinem Verständnis ausgelöst, wenn Daten kommen. TIdTCPServer weiß doch zu diesem Zeitpunkt noch gar nicht, wie ich was auslesen werde. Warum wird es ohne #13#10 nicht ausgelöst?
Wenn mir jetzt noch jemand dieses - aus meiner Sicht - merkwürdige Verhalten erklären könnte, wäre das phönomenal.

Danke aber auf jeden Fall an alle und insbes. Bambini - auf diese Lösung wäre ich nie gekommen!

Gruß, Alex

Zitat von Schwedenbitter:

Aber das Ereignis OnExecute wird nach meinem Verständnis ausgelöst, wenn Daten kommen. TIdTCPServer weiß doch zu diesem Zeitpunkt noch gar nicht, wie ich was auslesen werde. Warum wird es ohne #13#10 nicht ausgelöst?

Das OnExecute wird ja ausgelöst, nur der darin ausgeführte AContext.Connection.Socket.ReadLn(); wartet auf ein CR/LF.

PS: Das OnExecute wird aus einem Thread aus gerufen, d.h. Zugriffe auf die UI (bei dir das TMemo) sind da gefährlich. Das sollte mit TThread.Synchronize(nil, ...) geschützt werden.

**Schwedenbitter**

Zitat von Bambini:

Das OnExecute wird ja ausgelöst, nur der darin ausgeführte AContext.Connection.Socket.ReadLn(); wartet auf ein CR/LF.

Nein, leider nicht. Ich habe es eben nochmals ausprobiert, indem ich #13#10 wieder auskommentiert habe. OnExecute wird nicht abgefeuert.

Zitat von Bambini:

PS: Das OnExecute wird aus einem Thread aus gerufen, d.h. Zugriffe auf die UI (bei dir das TMemo) sind da gefährlich. Das sollte mit TThread.Synchronize(nil, ...) geschützt werden.

Wie geht denn das?
Innerhalb eines Threads ist mir das klar. Da Du das weißt, hast Du vermutlich meinen Quellcode angeschaut. Dort habe ich die das Polling des TIdTCPClient extra in einen Thread ausgelagert. Dann müsste ich ja eine eigene Klasse von TIdTCPServer reinbasteln. Das ist dann auch - gelinde gesagt - ungünstig, wenn man Objekte in die IDE integriert, ich als Laie denke, alles ist schick, und mir das dann um die Ohren fliegt ...
Danke schon mal für den Hinweis.

Zitat von Schwedenbitter:

Zitat von Bambini:

Das OnExecute wird ja ausgelöst, nur der darin ausgeführte AContext.Connection.Socket.ReadLn(); wartet auf ein CR/LF.

Nein, leider nicht. Ich habe es eben nochmals ausprobiert, indem ich #13#10 wieder auskommentiert habe. OnExecute wird nicht abgefeuert.

hm, bei mir schon.

Zitat von Schwedenbitter:

Wie geht denn das?

Dein Code sollte so aussehen:

zusammenfalten · markieren

Delphi-Quellcode:

			Procedure TForm1.IdTCPServerExecute(AContext: TIdContext);

Var

   Data:String;

Begin

   Data:= AContext.Connection.Socket.ReadLn();

   Data:= Trim(Data);

  TThread.Synchronize(nil, procedure()

     begin

        ScrollLog('[Srv]: received:');

         ScrollLog('=> ' + Data);

     end);

Zitat von Schwedenbitter:

Innerhalb eines Threads ist mir das klar. Da Du das weißt, hast Du vermutlich meinen Quellcode angeschaut. Dort habe ich die das Polling des TIdTCPClient extra in einen Thread ausgelagert.

Passt.

Zitat von Schwedenbitter:

Dann müsste ich ja eine eigene Klasse von TIdTCPServer reinbasteln. Das ist dann auch - gelinde gesagt - ungünstig, wenn man Objekte in die IDE integriert, ich als Laie denke, alles ist schick, und mir das dann um die Ohren fliegt ...
Danke schon mal für den Hinweis.

Ich will nur sagen, dass das OnExecute() from TidTCPIPServer von einem Thread aus gerufen wird. Das macht die Komponente schon ganz alleine. Jedoch muss man dann aufpassen, das deine Code dann Threadsicher ist. D.h. Zugriffe auf Globale Variablen und UI sind zu synchronisieren (s.o.)

**Schwedenbitter**

Danke nochmal für's Drüberschauen und die Erklärung.

Zitat von Bambini:

...
Dein Code sollte so aussehen: ...

Das isz jetzt zwar OT: aber wenn ich das richtig deute, muss ich dann auch VCL-Zugriffe in allen TIdTCPServer-Ereignissen entsprechend absichern? Das ist ein ganz schöner Overhead, finde ich.
Ich vermisse dann auch einen Compiler-Hinweis. Der "liest" ja den Code und sollte das wissen

Ich habe in meinem Thread das Ereignis aus dem Thread heraus threadsicher (hoffe ich) aufgerufen. Warum machen das die Indys nicht gleich so? Und was mache ich, wenn die Prozeduren größer als eine klitzekleine Ausgabe in der GUI werden? ...

Zitat von Bambini:

Ich will nur sagen, dass das OnExecute() from TidTCPIPServer von einem Thread aus gerufen wird. Das macht die Komponente schon ganz alleine. Jedoch muss man dann aufpassen, das deine Code dann Threadsicher ist. D.h. Zugriffe auf Globale Variablen und UI sind zu synchronisieren (s.o.)

Aber dass das Ereignis aus einem Thread aufgerufen wird, macht es doch nicht automatisch threadunsicher, wenn es sauber programmiert wurde. Ich dachte immer, so wäre es richtig und alle schlauen Ersteller von Komponenten würden das so (richtig) machen:

zusammenfalten · markieren

Delphi-Quellcode:

			// Völlig ungetestet zusammengeschrieben, um die Frage zu verdeutlichen!!!

type

   TfoobarEvent   = procedure (Value: Integer) of object;

   foobar = class(TThread)

   private

      FValue      : Integer;

      FOnErgebnis   : TfoobarEvent

      Procedure FireThreadSafe;

   protected

      procedure Execute; override;

   public

      property OnErgebnis: TfoobarEvent read FOnErgebnis write FOnErgebnis;

   end;

implementation

procedure foobar.FireThreadSafe;

Begin

   if Assigned(FOnErgebnis) then FOnErgebnis(FValue);

End;

procedure foobar.Execute;

begin

   while not Terminated do

   begin

      FValue:= Random(5000);

      if (FValue = 0) then Synchronize(FireThreadSafe);

   end;

end;

In diesem Fall muss ich dann innerhalb des VCL-Teils kein Syncronize mehr benutzen, oder doch?
Ich bin jetzt etwas verwirrt.

Zitat von Schwedenbitter:

Das isz jetzt zwar OT: aber wenn ich das richtig deute, muss ich dann auch VCL-Zugriffe in allen TIdTCPServer-Ereignissen entsprechend absichern? Das ist ein ganz schöner Overhead, finde ich.
Ich vermisse dann auch einen Compiler-Hinweis. Der "liest" ja den Code und sollte das wissen

Wer welchen Code zur Laufzeit aufruft, kann der Compiler nicht sehen und daher nicht helfen.

Zitat von Schwedenbitter:

Ich habe in meinem Thread das Ereignis aus dem Thread heraus threadsicher (hoffe ich) aufgerufen. Warum machen das die Indys nicht gleich so? Und was mache ich, wenn die Prozeduren größer als eine klitzekleine Ausgabe in der GUI werden? ...

Das bremst die Clients nebeneinander aus. Wenn Client1 in OnExecute() einen größeren Job zu erledigen hat, dann kann kein weiterer Client etwas machen.
Daher ist es schon sehr gut, dass die für jeden Client im eigenen Thread läuft.

Zitat von Schwedenbitter:

Zitat von Bambini:

Ich will nur sagen, dass das OnExecute() from TidTCPIPServer von einem Thread aus gerufen wird. Das macht die Komponente schon ganz alleine. Jedoch muss man dann aufpassen, das deine Code dann Threadsicher ist. D.h. Zugriffe auf Globale Variablen und UI sind zu synchronisieren (s.o.)

Aber dass das Ereignis aus einem Thread aufgerufen wird, macht es doch nicht automatisch threadunsicher, wenn es sauber programmiert wurde. Ich dachte immer, so wäre es richtig und alle schlauen Ersteller von Komponenten würden das so (richtig) machen:

zusammenfalten · markieren

Delphi-Quellcode:

			// Völlig ungetestet zusammengeschrieben, um die Frage zu verdeutlichen!!!

type

   TfoobarEvent   = procedure (Value: Integer) of object;

   foobar = class(TThread)

   private

      FValue      : Integer;

      FOnErgebnis   : TfoobarEvent

      Procedure FireThreadSafe;

   protected

      procedure Execute; override;

   public

      property OnErgebnis: TfoobarEvent read FOnErgebnis write FOnErgebnis;

   end;

implementation

procedure foobar.FireThreadSafe;

Begin

   if Assigned(FOnErgebnis) then FOnErgebnis(FValue);

End;

procedure foobar.Execute;

begin

   while not Terminated do

   begin

      FValue:= Random(5000);

      if (FValue = 0) then Synchronize(FireThreadSafe);

   end;

end;

In diesem Fall muss ich dann innerhalb des VCL-Teils kein Syncronize mehr benutzen, oder doch?
Ich bin jetzt etwas verwirrt.

Nein muss du dann nicht mehr. Einmal Synchronize reicht.

**Schwedenbitter**

Zitat von Bambini:

Wer welchen Code zur Laufzeit aufruft, kann der Compiler nicht sehen und daher nicht helfen.

Das ist richtig. Aber wenn ich z.B. einem String einen AnsiString zuweise oder umgekehrt, bekomme ich ja auch wenigstens eine Warnung - unabhängig davon, dass er das "durchcompiliert". Aber das führt auch zu weit.

Kannst Du mir bitte einen Gefallen tun:
Du schriebst weiter oben, dass bei Dir das Ereignis auch ohne #13#10 ausgelöst wird. Bei mir eben nicht. Am Quellcode kann es also (theoretisch) nicht liegen.
Macht es Dir was aus, mit ggf. per PN oder so eine von Dir compilierte exe-Datei zuzusenden?

Gruß, Alex

Zitat von Schwedenbitter:

Du schriebst weiter oben, dass bei Dir das Ereignis auch ohne #13#10 ausgelöst wird. Bei mir eben nicht. Am Quellcode kann es also (theoretisch) nicht liegen.
Macht es Dir was aus, mit ggf. per PN oder so eine von Dir compilierte exe-Datei zuzusenden?

An einer EXE wirst du es aber nicht sehen können. Das OnExecute() wird zwar gerufen, aber an dem ReadLn() kommt er nicht vorbei.
Von "Außen" sieht man das nicht. Nur wenn du einen BreakPoint in das OnExecute beim begin setzt, also VOR dem ReadLn() und deinen String ohne CRLF schickt, solltest du es auch sehen.

Zitat von Schwedenbitter:

Das ist richtig. Aber wenn ich z.B. einem String einen AnsiString zuweise oder umgekehrt, bekomme ich ja auch wenigstens eine Warnung - unabhängig davon, dass er das "durchcompiliert". Aber das führt auch zu weit.

Der "Code" ist problemlos, der parallele Zugriff ist die Kunst dabei. Und Windows Controls, wie z.B. das TMemo, nehmen es einem recht schnell übel, wenn ein anderer Thread sich mit denen unterhalten will.

**Schwedenbitter**

Zitat von Bambini:

An einer EXE wirst du es aber nicht sehen können. Das OnExecute() wird zwar gerufen, aber an dem ReadLn() kommt er nicht vorbei.
Von "Außen" sieht man das nicht. Nur wenn du einen BreakPoint in das OnExecute beim begin setzt, also VOR dem ReadLn() und deinen String ohne CRLF schickt, solltest du es auch sehen.

Entschuldige bitte. Ich weiß, dass ich langsam wohl nerve. Aber ich muss nochmal fragen.
OnExecute wird aufgerufen. Warum sehe ich dann wiederum keine Meldung; auch wenn ich die Ausgabe ganz an den Anfgang setze:

zusammenfalten · markieren

Delphi-Quellcode:

			Procedure TForm1.IdTCPServerExecute(AContext: TIdContext);

Var

   Data:String;

Begin

   ScrollLog('OnExecute();');

   Application.ProcessMessages;

   Data:= AContext.Connection.Socket.ReadLn();

   Data:= Trim(Data);   // Sonder- und Leerzeichen entfernen

   TThread.Synchronize(nil,   // threadsicherer VCL-Zugriff

      Procedure()

      Begin

         ScrollLog('[Srv]: received:');

         ScrollLog('=> ' + Data);

      End);

End;