Threads - Swing - Synchronisation nötig?

[Kirro]

Hallo zusammen,

ich schreibe gerade an einem JApplet! Bei der Initialisierung habe ich folgendes:
[Java=4]SwingUtilities.invokeAndWait(new Runnable() {
public void run() {
createGUI();
}
});[/code]

in der createGUI werden Panels erstellt und hinzugefügt usw.

Nun zur 1.Frage:
im createGUI starte ich jetzt ein weiteren Thread, darf man das, weil es ja im invokeAndWait passiert? Sonst könnte ich von der Logik her den Thread auch vorher starten aber nicht nachher!
Ach als Info: Ich habe das invokeAndWait so verstanden, dass es für die Threadsicherheit von Swing benötigt wird!?!

Jetzt kommen wir zu der Synchronisation:
Im Thread wird eine List, die er einem Webservice bekommt, gesetzt!

liste = listeVomWebservice

, wenn er das gemacht hat Sleept er für 5 Sekunden, vor dem schlafen führt er noch ein notfiy() aus, die die Observer(Panels) benachrichtigt!
Die Panels lesen die List schreiben aber aber nicht in sie hinein!(das wiederholt sich dann die ganze zeit)

2.Frage:
Muss man hier die Abschnitte synchronisieren, also einmal wo er die Liste setzt und einmal in den Panels wo er sie ließt?

3.Frage:
In den Panels wird nach dem notify, die GUI upgedatet, z.B. hintergrundfarbe gesetzt oder vom Label die texte verändert, aber sowas soll ja nur in den Event-Methoden (Event dispatcher thread) geschehen, habe ich irgendwo mal gelesen! Oder ist hierfür die Funktion invokeAndWait zuständig, also das man es nicht in einer event methode machen muss?

Ich hoffe ihr versteht alles ...

Vielen Dank im Voraus!

MfG
GastyGast

 

[despikyxd]

wie ich bereits wo anders erwähnte : NOT THREADSAFE hat überhaupt nichts mit SYNCHRONIZED zu tun ...
es beschreibt lediglich den umstand das ein fehler im EDT möglicherweise nicht bemerkt wird da das exception-handling im EDT selbst erfolgt und man nur über umwege darauf einfluss nehmen kann > also bitte erstmal googlen was NOT THREADSAFE eigentlich bedeutet

 

[Kirro]

habe ich das denn irgendwo was anderes behauptet, in Bezug auf "nicht threadsicher" und "synchronized"? Ich sehe in meinem Text zumindestens nichts!

Die Fragen sind alle voneinander unabhängig zu betrachten(außer 1 und 3)... vielleicht schlecht sortiert ...sry!

das threadunsicher bezieht sich auf frage 1 und 3

und synchronized bezieht sich auf die 2. frage, wobei hier eigentlich die frage nur ist ob man es nötig hat, wenn eine thread in eine variable schreibt und die andere nur liest?

ich hoffe jetzt ist es bisschen deutlicher...

danke schonmal

 

[Marco13]

Die Fragen sind etwas unpräzise, aber ein Versuch:
im createGUI starte ich jetzt ein weiteren Thread, darf man das, weil es ja im invokeAndWait passiert?
Ja, das darf man.

2.Frage:
Muss man hier die Abschnitte synchronisieren, also einmal wo er die Liste setzt und einmal in den Panels wo er sie ließt?

Zu unpräzise, Beispielcode könnte helfen.


3.Frage:
In den Panels wird nach dem notify, die GUI upgedatet, z.B. hintergrundfarbe gesetzt oder vom Label die texte verändert, aber sowas soll ja nur in den Event-Methoden (Event dispatcher thread) geschehen, habe ich irgendwo mal gelesen! Oder ist hierfür die Funktion invokeAndWait zuständig, also das man es nicht in einer event methode machen muss?


Wenn das notify von dem neuen Thread aufgerufen wird, muss man das Setzen der Hintergrundfarbe wieder auf den EDT legen. Das geht übrigens oft mit invokeLater, aber invokeAndWait braucht man SEHR selten.

 

[Kirro]

Danke erstmal für die

zur 2.Frage
run() vom thread ( es fehlen noch try catch blöcke und while-schleifen, aber diese sind eher unwichtig):
die ersten 2 Zeilen sind bzw. nur die 2. zeile ist der schreibzugriff
[Java=7] DataArray o = App.webservice.getData();
App.datas = o.getItem();
setChanged();
notifyObservers();
current= new Date();
this.update.setText(formatter.format(current));
Thread.sleep(5000);
[/Java]

bei notifyObservers werden nun die Daten gelesen und auch ausgegeben per JLabels etc.
und direkt zur 3.Frage das Setzen von JLabels wird dann auch über das update() geregelt

also bsp:[JAVA=6]@Override
public void update(Observable arg0, Object arg1) {
// TODO Auto-generated method stub
repaint();
updateUserInterface();
}[/code]
in der updateUserInferface() wird das label.setText etc. ausgeführt

ich hoffe das ist präzise genug

 

[Marco13]

Naja ... ganz allgemein: Alle Änderungen an GUI-Components dürfen nur vom EDT aus gemacht werden. Sowas wie

void runMethodOfSomeThread()
{
    String text = createText();
    label.setText(text);
}

darf man NICHT machen. Auch wenn diese Änderung "indirekt" über das notify von Observern o.ä. gemacht wird. Solche Änderungen müssen auf dem EDT gemacht werden, z.B. als

void runMethodOfSomeThread()
{
    String text = createText();
    setTextOnEDT(text);
}

private void setTextOnEDT(final String text)
{
    SwingUtilities.invokeLater(new Runnable()
    {
        public void run()
        {
            label.setText(text);
        }
    };
}

 

[Kirro]

ja genau das wollte ich wissen

eine Frage noch: Wenn es in einem ActionListener z.b. MousePressed oder so ist muss aber kein invokeLater hinzugefügt werden, da es ja schon im EDT ist oder?


Nun zum Synchronisieren:

Es ist doch eigentlich nicht nötig in dem Fall zu synchronisieren oder, weil es schreiben ja nicht beide Abschnitte in die Variable?

 

[Marco13]

Synchronisieren hat nicht notwendigerweise was mit der Vermeidung von gleichzeitigem Schreiben zu tun, auch wenn das ein prominentes Beispiel ist. Synchronisation verwendet man ganz allgemein zur Absicherung eines "kritischen Abschnitts". Nochmal ganz spezifisch:
Bei
App.datas = o.getItem();
wird in "datas" irgendeine Liste oder so gespeichert, und innerhalb von
updateUserInferface()
wird auf diese Liste zugegriffen und Ding daraus in Labels geschrieben?

Soweit ich das sehe, KÖNNTE(!) es sein, dass dort z.B. für einen kurzen Moment ein inkonsistenter Zustand angezeigt wird: Wenn "updateUserInterface" vom EDT gemacht wird, und dort z.B. sowas wie

label0.setText(App.datas.get(0));
label1.setText(App.datas.get(1));
label2.setText(App.datas.get(2));

kann theoretisch zwischen der 1. und der 2. Zeile ein anderer Thread das "App.datas" ändern - aber wenn diese Änderung wieder ein update bewirkt, dann wird der Zustand sofort korrigiert. Ob so eine Änderung aber "kritisch" ist (und schwerwiegendere Folgen haben kann als einen für 5 Millisekunden angezeigten "inkonsitenten Zustand") musst du dir überlegen. In diesem Fall müßte man dort synchronisieren.

 

[Kirro]

ok für diese Antwort bin ich super dankbar!

Dann habe ich es vermutlich verstanden!