Discussion :

[maxvador]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[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).

[®om]

Ou directement, en 1 ligne:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

private static List<String> listDetailsOfCharge = new ArrayList<String>() {{ add("BEN"); add("OUR"); add("SHA"); }};

[bouye]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Sinon une enum ca marche bien aussi.

[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

[®om]

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

[Kikito]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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;
        }
    }