Zitat von Jim Carrey:

Bevor ich mich Queue widme eine Verständnisfrage:
- muss ich auch das Ändern des Hints und das Ändern des Icons des Tray-Icons meiner Anwendung sychronisieren?
- muss ich auch einfaches "besorgen von Informationen" synchronisieren? Zum Beispiel das Besorgen der Caption eines Labels?

Ja und Ja.

Und genau das ist der Punkt den ich meine. Siehst du wie aufwändig und fummelig das jetzt schon wird? Lass den Thread die eigentliche Arbeit machen, mach deine Oberflächen-Aktualisierungen in einem Timer und du bist schon fertig.

Mein Senf dazu:

Gib dem Arbeiterthread ein Event-Property.
Definiere im Form eine Aktualisierungsprozedur.
Weise diese Aktualisierungsprozedur dem Thread-Property zu.
Lass den Thread entscheiden, ob sein Zustand eine Aktualisierung rechtfertigt. Wenn ja, dann das Property (die Aktualisierungsprozedur) mittels Synchronize aufrufen.
In der Aktualisierungsprozedur prüfst Du, ob die Form sichtbar ist und machst die Aktualisierungen oder nicht.

Damit ist erreicht, daß dem Thread die Form nicht bekannt sein muß und es gibt eine klare Zuständigkeitstrennung.

Das halte ich auch für den richtigen Weg.

Allerdings würde ich mir den Aufwand mit einem Event ersparen und das Formular dem Thread direkt bekannt machen, sofern die Funktionalität nur in diesem einen Formular benötigt wird.

Aber mit Event ist es natürlich noch klarer voneinander entkoppelt.

Zitat:

Und genau das ist der Punkt den ich meine. Siehst du wie aufwändig und fummelig das jetzt schon wird?

Wieso aufwendig? In meinem Fall sind das circa 40 Zeilen Code die dafür sorgen, dass die GUI an den richtigen Stellen aktualisiert wird. Das ist doch nicht aufwendig
Der Thread hat mehrere Kriterien:

- while not Terminated
-- schlafe 25ms
-- berechne neue ProgressBar-Position
-- ist die Position größer als die alte UND ist mindestens Zeit X vergangen
--- JA > GUI aktualisieren

Worauf ich hinaus wollte war nur:

Dein Thread wird ja wohl "echte" Arbeit tun und nicht einen Progressbar-Indicator hochsetzen. Der Thread sollte sich um ebendiese Arbeit kümmern. Immer wenn du an der Oberfläche was änderst musst du evtl. auch diesen Thread anpassen. In zwei Jahren besteht mindestens die Hälfte deines Threads aus Code für irgendwelche Feinheiten der Oberfläche während der Code für die eigentliche Arbeit immer weiter dahinter versumpft.

Genau, deswegen hab ich ja zwei Threads.
Ich wollte "Arbeit" von "GUI-Anpassung" trennen.

Vorher war es ungefähr so...

- Mach deine Arbeit Schritt 1
-- prüfe A
-- prüfe B
-- prüfe C
-- passe die GUI an
- Mach deine Arbeit Schritt 2
...
..

Und damit der Arbeiterthread nun seine Arbeit in Ruhe machen kann, hatte ich das vor ein paar Monaten getrennt.

Zitat:

In zwei Jahren besteht mindestens die Hälfte deines Threads aus Code für irgendwelche Feinheiten der Oberfläche während der Code für die eigentliche Arbeit immer weiter dahinter versumpft.

Ich verstehe solche Aussagen nie.
Warum sollte der Thread versumpfen? Wenn ich etwas an meinem Programm rändere, ändere ich auch überall das, was damit zusammenhängt. Da kann nix versumpfen

Ein Thread zwischen Arbeitsthread und Gui würde Sinn machen, wenn der Arbeitsthread irgendwelche Ergebnisse in einer Liste bereitstellt, die die Gui (oder ein anderer Thread) nach und nach abarbeiten muss.

Das ist ja aber vorliegend offenbar nicht notwendig.

Also würde ich nach den Zuständigkeiten schauen.

Der Arbeitsthread berechnet etwas und kann einen Fortschrittswert bereit stellen.

Die GUI dient der Darstellung und Bedienung durch den User.

Da der Thread ohnehin läuft und die Darstellung über Synchronize problemlos möglich ist, würde ich das dort veranlassen.

Im Formular kann der User sich - wenn er will - normal bewegen und etwas schreiben und so nebenbei ändert sich die Progressbar Stück für Stück.

Die GUI selbst muss nicht wissen, was der Thread tut oder wie weit er ist. Ein kleiner Teil der GUI-Funktionalität wird einfach vorübergehend vom Thread gesteuert.

Da braucht es m.E. keinen zweiten Thread und keinen Timer.



Anders wäre es (nur), wenn die Prozesse vollständig getrennt wären, also z.B. auf verschiedenen Rechnern im Netzwerk oder so.
Dann könnten der Arbeitsshread im Server und die GUI im Client natürlich nicht aufeinander zugreifen. Dann wäre im Client ein Timer oder Thread erforderlich, der die neuen Daten abfordert und zeichnet.
Das ist ja aber hier nicht gegeben.

Erkläre doch mal, warum Du nicht MyForm.ShowProgressValue(Value) synchronisiert aufrufen willst. Das ist doch die einfachste und sauberste Lösung (wobei es mit einem Event noch sauberer, aber etwas aufwendiger wäre).