Discussion :

[spike2]

Bonjour,

Dans le cadre d'un projet, j'ai fais une petite application de jeu, basé sur le fait de pouvoir créer des objets de différents types.

Mon soucis se pose sur la partie graphique. J'ai un JPanel, Grille, où devront être affiché mes éléments. Ces éléments sont stockés dans la classe Modèle, dans un ArrayList.

On m'a conseillé de mettre ses éléments également dans la classe Grille, sous forme d'ArrayList, et de redéfinir la méthode paintComponent pour boucler sur les éléments de la liste, et de les afficher.

Ca donne quelque chose comme ça :

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
 
public class GrilleCentre extends JPanel{
 
    private ArrayList<Unite> listeCampsUn = new ArrayList<Unite>();
    int xcourant = 0;
    int ycourant = 10;
 
public void paintComponent(Graphics g) {
        super.paintComponent(g);       
 
        Image img;
        int x=0;
        int y=0;
 
        for(Unite u : listeCampsUn){           
            img = new ImageIcon("image/unite.jpeg").getImage();
            x = (Integer)u.getPropriete("x");
            y = (Integer)u.getPropriete("y");
 
            g2d.drawImage(img, x, y, this);
 
        }      
 
 }
 
public void ajouterUnite(Unite uni){        
 
        if (listeCampsUn.size()%5 == 0) {
             ycourant=0;
        }
        listeCampsUn.add(uni);
 
        uni.setPropriete("x", xcourant);
        uni.setPropriete("y", ycourant);            
        repaint();
        ycourant+=33;
 
    }

Le problème c'est que si j'ajoute un nouvel élément dans la liste, alors que la boucle for du paintComponent est lancé, parfois ça plante.

Il y a une autre méthode ? (Me dites pas simplement oui ^^)

Merci !

[DarkMolo]

Salut,

Tu pourrais nous indiquer le message d'erreur et la ligne où ça plante?

Ça devrait théoriquement booster le nombre de réponses.

[spike2]

Code :

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

1
2
3
4
5
Exception in thread "AWT-EventQueue-0" java.util.ConcurrentModificationException
        at java.util.AbstractList$Itr.checkForComodification(AbstractList.java:372)
        at java.util.AbstractList$Itr.next(AbstractList.java:343)
        at Graphismes.GrilleCentre.paintComponent(GrilleCentre.java:67)
        at javax.swing.JComponent.paint(JComponent.java:1017)

C'est la ligne où commence le for.

Je comprends l'erreur, c'est normal...Puisque je modifie la liste pendant que je la parcours. Donc je me demandais comment faire autrement

[DarkMolo]

Salut,

ConcurrentModificationException, c'est exactement comme tu disais, une itération sur une collection et en même temps ajout d'élément sur cette même collection, c'est ce qui cause le probléme.

Comme solution, avant d'itérer sur ta collection, tu pourrais en créer une copie, puis itérer sur la copie, ça donnerait:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
public void paintComponent(Graphics g) {
        super.paintComponent(g);       
 
        Image img;
        int x=0;
        int y=0;
        List<Unite> copieListeCampsUn = new ArrayList<Unite>();
        copieListeCampsUn.addAll(listeCampsUn);
        for(Unite u : copieListeCampsUn){ 
        //...

J'ai parcouru un peu la javadoc, j'ai vu ceci:

Javadoc dit:
java.util.concurrent.CopyOnWriteArrayList<E>
...
This array never changes during the lifetime of the iterator, so interference is impossible and the iterator is guaranteed not to throw ConcurrentModificationException.

C'est drôle, mais en fait c'est à moi de te remercier, je ne savais même pas que ce type de collection existait .

Donc en théorie il suffirait de changer la déclaration de ta liste:

Code :

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

1
2
 
private List<Unite> listeCampsUn = new CopyOnWriteArrayList<Unite>();

[bouye]

Et surtout s'arranger pour que la modification de la liste se fasse durant l'EDT, cha serait plus simple

Sinon y a aussi Collections.synchronizedList() mais apres il faut mettre des semaphores de verrou bien sur.

[spike2]

Ca a l'air de marcher DarkMolo, merci

Bouye >

Et surtout s'arranger pour que la modification de la liste se fasse durant l'EDT, cha serait plus simple

C'est à dire, comment faire ? Merci