Synchronization on non final field

[Generic1]

Hallo,

ich habe ein Problem mit dem synchronisieren und zwar habe ich unteren Sachverhalt,
NetBeans gibt mir eine Warnung aus an der makierten Zeile unten.
Die Liste kann ich nicht final deklarieren, da ich mit der Liste was machen muss im Thread (inneren Klasse).
Weiß jemand was ich machen kann, damit ich diese Warnung nicht mehr bekomme?
Besten Dank,
lg

package synchronizedtest;

import java.util.ArrayList;
import java.util.List;

public class Main {

    private List<String> list;

    public Main() {
        list = new ArrayList<String>();
        }

    public void setList(final List<String> list) { // liste kann gesetzt werden, daher kann man die Liste nicht final machen
        this.list = list;
        }

    private final class ToDoWithList extends Thread {

        @Override
        public void run() {
            synchronized(list) {  // hier bekomme ich die Warnung "Synchronization on non-final field"
                // hier wird die Liste verwendet
                }
            }
        }
}

 

[maki]

Wieso solltest du list nciht final deklarieren können?
Verhindert doch nur so Dinge wie [c]list = null[/c] nach der Initiaisierung.

 

[Generic1]

wenn ich die Liste als final deklariere, dann wirds mit der setter- Methode, die ich unter nochmal rauskopeirt habe, schwierig.

public void setList(final List<String> list) { // liste kann gesetzt werden, daher kann man die Liste nicht final machen
        this.list = list;
        }

 

[maki]

Dann musst du imho mit der Warnung leben.

 

[nrg]

edit: war quatsch ^^

 

[mvitz]

Glaube der übliche "Trick" geht in diesem Falle folgendermaßen:

package synchronizedtest;

import java.util.List;

public class Main {

    private final List<String>[] lists;

    @SuppressWarnings("unchecked")
    public Main() {
        lists = new List[1];
    }

    public void setList(final List<String> list) {
        lists[0] = list;
    }

    private final class ToDoWithList extends Thread {

        @Override
        public void run() {
            synchronized (lists[0]) {

            }
        }
    }

}

 

[maki]

@mvitz
Vielleciht steh ich auf dem Schlauch, aber das ist doch dasselbe...

@nrg
war doch ok, die Warnung ist dann weg, das Risiko das gleiche

 

[mvitz]

Dasselbe im Vergleich wozu?

Wenn du im Vergleich zum TO meinst, dann habe ich die Liste in ein Eindimensionales Array gepackt. Dadurch kann das Array final sein und man kann die Liste trotzdem neu setzen.

 

[Generic1]

@mvitz
Vielleciht steh ich auf dem Schlauch, aber das ist doch dasselbe...

Also in seinem Beispiel ist die warnung weg, der Compiler schaut nur auf die List<String> selber nicht aber
auf die Einträge ([]) -> schräg!!
ich werd mal die Warnung gekonnt ignorieren, ich weiß was ich tu und damit müsste es sich haben,
Besten Dank,

 

[maki]

Dasselbe im Vergleich wozu?

Wenn du im Vergleich zum TO meinst, dann habe ich die Liste in ein Eindimensionales Array gepackt. Dadurch kann das Array final sein und man kann die Liste trotzdem neu setzen.

.. und das erste element in der Liste kann immer noch null sein... so wie beim TS

synchronized(null) wirft eine NPE

 

[mvitz]

.. und das erste element in der Liste kann immer noch null sein... so wie beim TS

synchronized(null) wirft eine NPE

Ah ok, das wusste ich nicht ;D