[vector] erreur étrange

Code:

---

1 2 3 4 5 6 7 8 9 10

private synchronized void drawTrajectories(){ for(int i=0;i<trajectories.size();i++){ try{ trajectories.get(i).draw(gbuf,startTime,currentTime); } catch (ArrayIndexOutOfBoundsException e){ // sometimes I got this exception I don't know why } } }

---

voila, j'ai donc cette méthode qui lance une exception ArrayIndexOutOfBoundsException de temps en temps (pas tout le temps).

Pourtant en théorie :
- i<trajectories.size()
- elle est synchronized

donc si vous avez des idées, ça serait pas mal, parce que je bon j'aime pas resté sur ma faim et passé à autre chose sans comprendre ;)

Salut,

je vois pas de probleme, remplace la boulce for par un Iterator pour voir. et modifie le catch:
catch(Exception e){
e.printStackTrace();
}

Si une modification est fait au vector l'Iterator lancera une exception.

bah en fait mon code roule, c'est bon, j'ai mis le try catcht pour ça, mais je comprend pas pourquoi il y a une exception, elle ne devrait pas exister à mon avis !!

mais je vais tester avec l'iterator pour voir ce que ça dit.

mais en théorie, le fait que ma méthode soit synchronized ne devrait pas éviter que le vecteur soit modifié ???

Citation:

---

Envoyé par Sp4ce

Code:

---

1 2 3

Pourtant en théorie : - i<trajectories.size() - elle est synchronized

---

---

Et les autres méthodes qui accèdent à ton vecteur sont aussi synchronisés SUR LE MÊME OBJET?

PS: ArrayList, c'est mieux que Vector :)

Citation:

---

Envoyé par ®om

Et les autres méthodes qui accèdent à ton vecteur sont aussi synchronisés SUR LE MÊME OBJET?

---

ça sous-entend que si une méthode non synchronisée (sans le mot clé synchronized donc) fait une modif sur mon objet, cette modif sera appliqué tout de suite ?

tu dis "synchronisé sur un objet", mais je ne sais que mettre le mot clé synchronized quand je déclare une méthode, je veux dire que je synchronise la méthode, pas l'objet... enfin ya peut être un truc qui m'a échappé o_O

http://rom.developpez.com/java-synchronisation/

section II. Exclusion mutuelle de sections critiques

salut, la principal fonction qui y accède c'est celle-ça

Code:

---

1 2 3 4 5 6 7

synchronized(this) { trajectories = new Vector<Trajectory>(); if(localTrajectories!=null) trajectories.addAll(localTrajectories); trajectories.addAll(app.getTrajHandler().getTrajectories()); }

---

qui se trouve dans le run() d'un thread.

donc si je voudrais synchroniser sur le même objet, il faudrait que je fasse comment ? parce que apparemment mes deux sections critiques le sont pas. (Je croyais que mettre le mot synchronized suffisait mais en fait non).

Citation:

---

Envoyé par Sp4ce

tu dis "synchronisé sur un objet", mais je ne sais que mettre le mot clé synchronized quand je déclare une méthode, je veux dire que je synchronise la méthode, pas l'objet... enfin ya peut être un truc qui m'a échappé o_O

---

Le mot synchronized permet d'obtenir un verrou sur un objet.
ex:

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

class MonProg extends JFrame{   Vector<Object> vec = new Vector<Object>; // contruteur public MonProg(){   // création d'un thread Thread T1 = new Thread(){ public void run(){ remplirVector(); // Méthode synchro T1 obtient un verrou sur l'objet MonProg }   };   T1.start();   drawTrajectories(); // MonProg tente d'obtenir le un verrou // on doit attentre que T1 libere le verrou pour continuer. // le programme se bloque ici tant que T1 possède le verrou.   }   public synchronized void remplirVector(){ //... remplire le vector } public synchronized void drawTrajectories(){ //... utiliser le vector } }

---

D'ailleurs, utilise ArrayList au lieu de Vector.

alors je vais expliquer un tout petit mieux parce que mon problème est pas aussi simple ^^

alors voila le run de mon thread qui modifie le vecteur (dsl, je vais garder Vector, ça me va très bien, a moins que ça ait un lien avec le problème :)

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

public void run(){ while(true){ (...) if(!localOnly){ synchronized(this) { trajectories = new Vector<Trajectory>(); if(localTrajectories!=null) trajectories.addAll(localTrajectories); trajectories.addAll(app.getTrajHandler().getTrajectories()); } } else { trajectories = localTrajectories; } repaint() } }

---

à coté de ça dans une autre classe interne qui s'occupe de dessiner, j'ai mon paint(Graphics g) qui invoque drawTrajectories() qui lance l'exception

Code:

---

1 2 3 4 5 6 7 8 9 10

@Override public void paint(Graphics g){ (...)   if(trajectories!=null) drawTrajectories();   (...) }

---

donc dans mon run, la partie qui modifie le vecteur Trajectories est bien verrouillé vu que j'ai mis synchronized et drawTrajectories aussi vu que je j'ai aussi mis synchronized (cf. plus haut)

Il me manque quelques informations sur la structure de ton code pour cibler le problème.

Citation:

---

Code :
@Override
public void paint(Graphics g){
(...)

if(trajectories!=null)
drawTrajectories();

(...)
}
donc dans mon run, la partie qui modifie le vecteur Trajectories est bien verrouillé vu que j'ai mis synchronized et drawTrajectories aussi vu que je j'ai aussi mis synchronized (cf. plus haut)

---

Dans cette partie, il y a un bloc synchronized ou non ? si non, faudrait en mettre un. Encore la, ca dépend de la structure du code.

Pour le Vector, il s'agit d'une collection qui date du 1.1. et ces méthodes sont syncronized, ce qui fait ralentir.

non mais tant pis je laisse tomber... j'ai pas que ça à faire (hélas) et donc je préfère m'intéresser au véritable problèmes qui me sont confié que du debuggage en profondeur... voila, je mettrais un petit bout de commentaire à coté du try catch si jamais quelqu'un qui doit bosser sur mon code se pose la question...

ça se trouve c'est tout con, mais ça me gave les thread :)

Citation:

---

Envoyé par Sp4ce

non mais tant pis je laisse tomber... j'ai pas que ça à faire (hélas) et donc je préfère m'intéresser au véritable problèmes qui me sont confié que du debuggage en profondeur... voila, je mettrais un petit bout de commentaire à coté du try catch si jamais quelqu'un qui doit bosser sur mon code se pose la question...

ça se trouve c'est tout con, mais ça me gave les thread :)

---

:aie::aie::aie: