sporadisches Deadlock und noch ein Schmankerl ;)

[ugh_bough_]

Hallihallo.
Ich habe ein kleines Problem, bei dem ich nicht weiß, woher es resultiert.

1. Deadlock:
Pseudocode:

class MyPanel
extends JPanel
{
    //
    private synchronized void doIt(arg)
    {
        ... // zugriff auf die methoden eines objekts, welches collections verwaltet
            // keine änderung in irgendwelchen swingObjekten
    }

    private Thread
    {
        public void run()
        {
            ...
            doIt(...);    // vom programm aufgerufen
            ...
        }
    }

    void aButtonWasClicked(ActionEvent e)
    {
        doIt(...);    // vom benutzer aufgerufen
    }
}

Grundgedanke:
Der Thread wertet Daten aus, die dann in das in doIt() erwähnte Objekt übergeben werden, welches sie in verschiedenen collections sammelt. doIt() wird also aus dem anonymen Thread und vom Benutzer aus dem Swing-Thread aus aufgerufen, der auch Daten auflisten will.

Problem:
Ab und zu kommt es zu einem Deadlock, den ich mir nicht erklären kann. In den meisten Fällen funktioniert es. Könnte mir auch nicht erklären, daß es irgendwie an Swing läge.

Lösung:
Kann ich einfach darauf verzichten, doIt() synchronized zu machen und statt dessen die Collections in dem benutzen Objekt (nicht im JPanel) synchronized machen? Ich habe diese Lösung sein lassen, da
1. die Daten im die collections beherbergenden Objekt noch in Schleifen verarbeitet werden und
2. ich mir nicht sicher war, ob diese Synchronität sich bis auf die oben gezeigt Ebene (also die Trennung vom Automatik-Thread und dem Benutzer-Thred) durchschlägt.

Danke für Hilfe.

PS: Java 5, falls es doch wichtig ist.





Nun zum Schmankerl:
Ist eigentlich nur eine Interessefrage.

HashSet<Object> set = new HashSet<Object>(); // irgendein Inhalt. Für die Frage irrelevant
... // set füllen
// nun einen Iterator holen und für jedes Objekt o im Iterator
    set.remove(o); // knackpunkt

Kann man die einträge der Collection (HashSet hab ich nur als Bsp benutzt) löschen, wärend man mit nem Iterator drüberläuft? Sollte doch eigentlich gehen, oder? Der It macht doch Kopien der Referenzen der in der Collection enthaltenen Objekte, oder nicht?

 

[Wildcard]

Kann ich einfach darauf verzichten, doIt() synchronized zu machen und statt dessen die Collections in dem benutzen Objekt (nicht im JPanel) synchronized machen?

Gibt doch schon synchronisierte Collections

Kann man die einträge der Collection (HashSet hab ich nur als Bsp benutzt) löschen, wärend man mit nem Iterator drüberläuft? Sollte doch eigentlich gehen, oder? Der It macht doch Kopien der Referenzen der in der Collection enthaltenen Objekte, oder nicht?

Nur über den Iterator löschen!!!

 

[ugh_bough_]

1. Das meinte ich. Habe nur gedacht, daß es zu Problemen führen könnte, denn doIt() könnte ja mitunter gleichzeitig mehrmals laufen und ebenso die Methoden des Objekts, dass die Daten weiter auswertet und dann in die Collections legt.
Du meinst also, dass es ok ist und keinerlei Probleme mit sich bringt.
(Mist, muß mich mal mehr mit Thread beschäftigen. Erst recht mit java.util.concurrent

2. War das ein Befehlt? Die Methode scheint also ok zu sein.

 

[Wildcard]

Das meinte ich. Habe nur gedacht, daß es zu Problemen führen könnte, denn doIt() könnte ja mitunter gleichzeitig mehrmals laufen und ebenso die Methoden des Objekts, dass die Daten weiter auswertet und dann in die Collections legt.

deshalb ja synchronized

Die Methode scheint also ok zu sein.

Als Beispiel:

            List a = new Vector();
            Iterator it = a.iterator();
            while(it.hasNext())
            {
                it.next();
                it.remove();
                //NIE a.remove()
            }

klar was ich meine?

 

[ugh_bough_]

ja ok klar aber wieso über den it löschen. lass mich raten, weil er dann die referenzen aus sich selbst auch rauslöscht?

 

[ugh_bough_]

frage 1 hatte nichts zu tun mit frage 2

obwohl mir das mit der ersten lösung immer noch irgendwie suspekt ist. ich hab da kein VERTRAUEN

 

[Wildcard]

Nein, weil er dann eine Exception wirft, da die Collection weniger Element enthält als der Iterator erwartet.

 

[Gast]

i see. danke. haken.

 

[Illuvatar]

meld dich halt mal an du bist - glaube ich - schon länger als gast hier als ich angemeldet bin ???:L

 

[Wildcard]

meld dich halt mal an du bist - glaube ich - schon länger als gast hier als ich angemeldet bin

Ist er doch. Nur nicht eingeloggt. Es sei denn, den seltsamen Namen gibts 2mal :?