VCL not thread save

[stahli]

Der Sachverhalt ist ja bekannt.

Was müsste die VCL (oder allgemein eine GUI) denn tun/berücksichtigen, damit es nicht so wäre?

Alle Controls und Daten (Objekte, Listen) mit einer Critical Section schützen? Oder gibt es andere grundsätzliche Aspekte/Überlegungen?

Die CS-Lösung habe ich gerade mal (noneVCL) angetestet, was sehr gut funktioniert hat.

[jensw_2000]

Die komplette VCL bekommst Du bis zur Rente nicht thread safe.
Am besten ist es, die UI/VCL Zugriffe aus den Threads sinnvoll zu bündeln und die Zugriffe zu synchronisieren bzw. Teilbereiche über Critical Sections zu atomarisieren.

[Furtbichler]

Bei sehr vielen Änderungen, die in Threads vorgenommen werden, habe ich Messages verwendet. Alternativ einen Timer, der im Hauptthread die Daten periodisch aktualisiert. Kommt immer drauf an, was öfter vorkommt: Änderungen oder ein Timer (z.B. alle 20ms-200ms). Viele Änderungen => Timer, Wenig Änderungen => Messages oder Synchronize (oder Queue)

[himitsu]

SendMessage/PostMessage sind erstmal thread-save und Funktionen, welche darüber laufen (GetWindowText usw.),
da sie praktisch selber die Zugriffe in den Thread reinsynchronisieren, in welchem die angesprochende Komponente erstellt wurde (Aufruf von CreateWindow).

Meistens ist wirklich die VCL selber nicht thread-save und da müsstest du erstmal alle Methoden und Property sämtlicher VCL-Komponenten absichern. (z.B. über eine CS oder via Messages)


Und wenn du dann in 2+ Jahren damit fertig bist, dann darfst du die restlichen Lücken suchen.


Lösung: Die VCL ist nicht thradsave und an den "wenigen" Stellen in anderen Treads mußt du die zugriffe eben synchronisieren.

[Bernhard Geyer]

Die VCL ist nicht Thread-Save weil auch die Win-API hier nicht Thread-Save ist. Alle Windows-Handles darf nur im erzeugenden Thread darauf zugegriffen werden.

[Sir Rufo]

Wofür muss ein UI-Framework (bzw. UI im Allgemeinen) threadsafe sein?

Für die Kommunikation mit dem Benutzer reicht ein Thread.
Die Daten selber liegen in irgendwelchen Daten-Objekten und werden bei einer Änderung im UI dargestellt oder Daten vom UI dort hineingeschrieben.

Der Zugriff auf diese Daten-Objekte, der muss threadsafe sein (wenn multithreading).

Wenn die Rückmeldungen von den Threads zu häufig passieren, dann schaltet man einfach einen Timer dazwischen, der die eigentliche Aktualisierung des UI bremst (throttle) und diese Aktualisierung nur alle x Millisekunden zulässt.

Das menschliche Auge kann 25 Bilder pro Sekunde erfassen, so dass 40ms als throttle ein guter Ausgangswert ist. Allerdings werden auch 100ms gefühlt nicht wahrgenommen.

Und ob der Benutzer einen Mehrwert hat, wenn die Informationen über den Schirm rasen oder sich die Werte extrem schnell verändern glaube ich auch nicht, so dass also auch kein Informationsverlust entsteht.

Wenn bestimmte Werte wichtig sind, dann lässt man diese nicht durch die Anzeige den User kontrollieren, sondern durch die Anwendung und signalisiert, dass die Situation X eingetreten ist.

[stahli]

Ich möchte das Thema gern grundsätzlich diskutieren (also gern weg von der VCL im speziellen).

@Sir
Ich beschäftige mich noch mit dem Databinding (nicht dem von Emba) und betrachte die GUI als "Abbild der BL(+Daten)". Wenn sich in der BL etwas ändert sollte das in der GUI auch erkennbar sein.
Natürlich muss das nicht jede ms erfolgen.
Es sollte eine Art "natürliche Verbindung" zwischen GUI und BL geben, um die sich der Programmierer dann nicht mehr kümmern muss.

Mal ehrlich: JEDER wundert sich doch am Anfang seiner Karriere, warum ein

Delphi-Quellcode:

Label.Caption := SchleifenWert.AsString

oder ein

Delphi-Quellcode:

Progressbar.Step

im Formular nicht dargestellt wird.

Ich möchte das eben gern anders lösen.

[Sir Rufo]

Die Anzeige informieren, dass sich etwas verändert hat und die Anzeige holt dann (sobald es möglich ist) die neuen Werte und stellt diese dar.

Der UI-Thread sollte eigentlich fast immer schlafen (idle) und nur kurz aufleben um neue Informationen anzuzeigen oder Eingaben vom User entgegennehmen und diese dann weiterreichen.

Idealerweise müsste der Business-Layer (allgemeiner: alles was nicht zu UI gehört) im eigenen Thread-Kontext laufen, was aber in den meisten Fällen nicht passiert, da sich die Laufzeit der Aktionen meistens auf ein paar Millisekunden beschränkt (wenn überhaupt messbar) und ein eigener Thread-Kontext wäre dort mit Kanonen auf Spatzen schießen.

Somit werden sich nur die langwierigen Abläufe herausgepickt und diese werden in einen separaten Thread-Kontext verschoben.

Zurück zu deinem Beispiel mit der Schleife:
Die Schleife gehört in einen eigenen Thread und die Werte in ein Daten-Objekt. Das UI wird informiert, dass sich die Daten geändert haben und dann entscheidet das UI wann genau es diese neuen Informationen abruft und darstellt (ASAP oder maximal alle x Millisekunden).

Die meisten Programmier-Anfänger haben allerdings vor dem Start ihrer Karriere auch das Prinzip des UI nicht verstanden bzw. sich darüber keinen Kopf gemacht (ich möchte mich dort unbedingt eingeschlossen wissen). Daher auch diese "Fehler", denn es passiert etwas anderes als man erwartet hat. Es passiert aber nun mal genau das, was man programmiert hat. Diese Erwartungshaltung rührt aber von Unwissenheit her, und das schützt nun mal nicht vor der Strafe ;)

[sx2008]

Die Synchronize - Methode von TThread kann man in neuen Delphi Versionen auch mit einer Closure aufrufen.
Dazu ein interessanter Artikel von Uwe Rabe:
http://www.uweraabe.de/Blog/2011/01/...th-parameters/
Ich denke das ist der Weg den man gehen muss da man die VCL selbst nicht ändern kann.

[stahli]

@Sir

Früher hat man Automobile mit einer Kurbel in Gang gebracht ... bis es andere Lösungen gab. :wink:

Mir wäre eine Lösung, die das Binding und Handling zwischen GUI(s) und BL automatisiert auf jeden Fall sehr willkommen.
Dann kann man sich mehr um das Wesentliche kümmern, hat deutlich weniger Arbeit und dennoch eine flüssige grafische Schnittstelle, die sich für den User gut anfühlt (also gut funktional ist).

Mein Wusch war so etwas schon lange und so langsam etwickelt sich das auch... :)


@sx2008

Ich versuche das aus dem von Dir genannten Grund gerade mal ohne VCL.