initialisation static d'une liste

Version imprimable

Bonjour,

Je voudrais faire une methode static qui renvoie une liste de string sans avoir à la recréer à chaque fois. Celle liste est fixe et est connue au début.

J'avais pensé au code suivant mais je suis pas sur que ca marche à cause du multithread.
Code:

1 2 3 4 5 6 7 8 9 10 11 private static List listDetailsOfCharge = null; private static List getDetailsOfChargeList() { if(listDetailsOfCharge==null){ listDetailsOfCharge = new ArrayList(); listDetailsOfCharge.add("BEN"); listDetailsOfCharge.add("OUR"); listDetailsOfCharge.add("SHA"); } return listDetailsOfCharge; }
Ensuite j'ai pensé à ca:
Code:

1 2 3 4 5 6 7 8 9 10 private static List listDetailsOfCharge = new ArrayList(); static { listDetailsOfCharge.add("BEN"); listDetailsOfCharge.add("OUR"); listDetailsOfCharge.add("SHA"); } private static List getDetailsOfChargeList() { return listDetailsOfCharge; }
Sans utiliser de synchronise comment je fais?

Merci pour votre aide.

12/10/2006, 18h51
le y@m's

Dans ton premier code, la méthode doit être décarée synchronized pour être utilisée dan un contexte multithread.
Mais dans ton cas le second code est préférable (plus performant car pas besoin de synchronized).
12/10/2006, 20h42
®om

Ou directement, en 1 ligne:

Code:

private static List<String> listDetailsOfCharge = new ArrayList<String>() {{ add("BEN"); add("OUR"); add("SHA"); }};
13/10/2006, 00h45
bouye

Code:

private static List<String> CHARGE_DETAILS_LIST = Collections.unmodifiableList(Arrays.asList(new String[] {"BEN", "OUR", "SHA"}));

Sinon une enum ca marche bien aussi.
13/10/2006, 09h14
maxvador

Merci pour vos réponses, je vais donc garder mon 2eme code.
J'aurai du préciser que j'etais pas en JDK1.5, mais en 1.4
13/10/2006, 09h15
®om

Je pense que si tu utilises asList, tu n'as pas besoin de unmodifiableList (c'est déjà une unmodifiableList il me semble)...

Citation:

Envoyé par ®om

Je pense que si tu utilises asList, tu n'as pas besoin de unmodifiableList (c'est déjà une unmodifiableList il me semble)...

La taille de la liste est fixe mais on peut en changer les éléments :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
 /**
     * @serial include
     */
    private static class ArrayList<E> extends AbstractList<E>
	implements RandomAccess, java.io.Serializable
    {
        private static final long serialVersionUID = -2764017481108945198L;
	private Object[] a;

	ArrayList(E[] array) {
            if (array==null)
                throw new NullPointerException();
	    a = array;
	}

	public int size() {
	    return a.length;
	}

	public Object[] toArray() {
	    return (Object[])a.clone();
	}

	public E get(int index) {
	    return (E)a[index];
	}

	public E set(int index, E element) {
	    Object oldValue = a[index];
	    a[index] = element;
	    return (E)oldValue;
	}

        public int indexOf(Object o) {
            if (o==null) {
                for (int i=0; i<a.length; i++)
                    if (a[i]==null)
                        return i;
            } else {
                for (int i=0; i<a.length; i++)
                    if (o.equals(a[i]))
                        return i;
            }
            return -1;
        }

        public boolean contains(Object o) {
            return indexOf(o) != -1;
        }
    }