Zugriff paralleler Threads auf selbes JTextPane

[Mujahiddin]

Guten Abend, alle zusammen, folgendes Anliegen möchte ich mit euch teilen:
Ich bin gerade im Begriff, ein Programm zu erstellen (ja, das wundert), in dem zwei Threads gleichzeitig auf ein JTextPane zugreifen und Strings anhängen via Document#insertString.
Dafür gibt es allerdings im JTextPane-Erbe eine Methode:

public void addMessage(Message msj)
{
	[...]
	int outputLength = getDocument().getLength();
	getDocument().insertString(outputLength, "© Muja - ", imageStyle); // Zeilenabstand erhöhen
	getDocument().insertString(outputLength, msj.getAuthor().getName() + ": " + text + "\n", null); // einzufügender String
	[...]
}

Nun ist das Problem, dass bei 'gleichzeitigem' Aufruf der Methode die Strings ineinander übergehen, sprich, folgendes könnte passieren:
JTextPane - Soll:
Thread-1: Halli Hallo!
Thread-2: Halli Hallo!
Ggf. ist:
Thread-Thread-2:Halli Hallo!
1:Halli Hallo!

Lösung hierfür: synchronized
public synchronized void addMessage(...){...}
Problem aber ist immer noch, dass wenn sie 'gleichzeitig' darauf zugreifen, durch diese Methode die Überlappung der Strings zwar verhindert wird, aber komischerweise wird das JTextPane urplötzlich schwarz, repaintet nicht und reagiert auch sonst nicht mehr.
Woran könnte das denn liegen?
Liebe Grüße.

 

[dku]

Hi,

ich würde evtl versuchen nicht an deiner JTextPane Instanz zu sychronisieren.

Zwei Ideen, müsstest du aber Testen:

Eine Instavariable
private Object lock = new Object();

public void addMessage(Message msj)
{
    [...]
    synchronized(lock) {
    int outputLength = getDocument().getLength();
    getDocument().insertString(outputLength, "© Muja - ", imageStyle); // Zeilenabstand erhöhen
    getDocument().insertString(outputLength, msj.getAuthor().getName() + ": " + text + "\n", null); // einzufügender String
     }
    [...]
}

oder
eine extra Klasse, welche das schreiben ins JTextPane übernimmt, synchronized ist und von beiden Threads angesprochen wird.

Viele Grüße,
David

 

[Marco13]

Alles, was den Zustand oder das aussehen einer Swing-Component verändert (oder darauf zugreift) muss vom Event-Dispatch-Thread gemacht werden: Threads and Swing

void addMessage(final String m)
{
    SwingUtilities.invokeLater(new Runnable()
    {
        public void run()
        {
            documentDingens.insertSomewhere(m);
        }
    });
}

 

[eRaaaa]

Kannst du mal ein ausführbares Beispiel posten?

AbstractDocument (Java Platform SE 7 b112)

--> Inserts some content into the document. Inserting content causes a write lock to be held while the actual changes are taking place, followed by notification to the observers on the thread that grabbed the write lock.

This method is thread safe

 

[Marco13]

OK, aber getLength und mehrere insertString-Aufrufe (und potentiell das, was bei [....] steht) werden eine synchronisation erforderlich machen....

 

[Mujahiddin]

Alles, was den Zustand oder das aussehen einer Swing-Component verändert (oder darauf zugreift) muss vom Event-Dispatch-Thread gemacht werden: Threads and Swing

void addMessage(final String m)
{
    SwingUtilities.invokeLater(new Runnable()
    {
        public void run()
        {
            documentDingens.insertSomewhere(m);
        }
    });
}

Dankesehr, das klappt problemlos!

 

[Marco13]

--> Inserts some content into the document. Inserting content causes a write lock to be held while the actual changes are taking place, followed by notification to the observers on the thread that grabbed the write lock.

This method is thread safe

Ach ja, @eRaaaa: Da wollte ich nochmal nachsehen: Ich vermute, das "thread safe" bezieht sich darauf, dass mehrere Threads diese Methode aufrufen können (und sie entweder synchronized oder reentrant ist), aber das heißt nicht notwendigerweise, dass das von jedem beliebigen Thread gemacht werden darf, wenn es sich um das Document handelt, das in einer JTextComponent liegt: Dort würde die Änderung des Documents eine Veränderung der Swing-Component bewirken, und DANN dürfte die Änderung des Documents auch nur vom EDT gemacht werden können. Zumindest sofern da nicht irgendwo sowas steht wie

void fireChanged()
{
    if (isEventDispatchThread(Thread.currentThread()) reallyDoIt();
    else refireOnEDTWithSwingUtilities();
}

Bei Gelegenheit mal ein bißchen im Code wühlen :rtfm: