paintComponent est appelé en boucle

Bonjour,

Je n'arrive pas à trouver la racine du problème.

J'ai une classe "MonFilm" qui instancie 2 ArrayList :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

public MonFilm(){   this.figures = new ArrayList <Figure>(); this.couleursFond = new ArrayList <CouleurFond> ();   String FondEcran; } private ArrayList <CouleurFond> couleursFond; private String FondEcran; private ArrayList <Figure> figures;

---

instanciée comme suit :

Code:

---

1 2 3

MonFilm monFilm = new MonFilm(); monFilm.AjouteCouleurFond(new CouleurFond(Color.white, 0));

---

Les deux méthodes en question sont :

Code:

---

1 2 3 4 5 6 7 8

public void AjouteCouleurFond(CouleurFond CF){ couleursFond.add(CF); }   public ArrayList <CouleurFond> getTableauCouleurs (){ return couleursFond; }

---

La classe CouleurFond est toute bête, une Color Couleur et un int Index, avec set et get.

Lorsque dans paintComponent du JPanel, j'appelle :

Code:

---

1 2	setBackground(monFilm.getTableauCouleurs().get(0).getCouleur());

---

la méthode est appelée en boucle. Sur la console, je lis :

Citation:

---

Appel paintComponent
Appel paintComponent
Appel paintComponent
Appel paintComponent
Appel paintComponent
Appel paintComponent
Appel paintComponent
Appel paintComponent

---

tant que je n'interrompt pas le programme. (j'ai mis un

Code:

---

System.out.println("Appel paintComponent");

---

en début de méthode pour contrôle.)

Je ne comprends pas ce qui se passe. Quelqu'un ayant eu cette expérience pourrait-il me guider? Merci.

Citation:

---

Envoyé par HNoury

Lorsque dans paintComponent du JPanel, j'appelle :

Code:

---

1 2	setBackground(monFilm.getTableauCouleurs().get(0).getCouleur());

---

---

Il ne faut pas faire ce genre d'appels dans paintComponent() !

Puisque tu as déjà la couleur en question sous la main, tu peux directement peindre un rectangle de cette couleur couvrant la surfrace du composant.

Merci Bouye pour ta réponse.

En effet, l'appel d'une couleur semble ne pas fonctionner dans paintComponent. Suite à ta réponse, j'ai essayé :

Code:

---

1 2 3 4

CouleurFond cf = monFilm.getTableauCouleurs().get(0); Color cl = cf.getCouleur(); setBackground(cl);

---

Les deux premières lignes ne posent pas de problème : paintComponent n'est appelé qu'une fois. Mais dès que je mets la troisième, le problème se repose.

Le problème ne viendrait donc pas de l'utilisation des tableaux comme je le pensais mais de quelque chose que j'ignore concernant la classe Color?

Il me faudrait vraiment appeler des couleurs différentes dans cette méthode, je l'ai déjà fait dans d'autres logiciels, je ne comprends pas...

Remarque :
Si je fais :

Code:

---

1 2 3 4 5

CouleurFond cf = monFilm.getTableauCouleurs().get(0); Color cl = cf.getCouleur(); g.setColor(cl); g.fillOval(10,10, 300, 300);

---

un cercle noir s'affiche et non blanc, comme quoi la couleur "ne parvient pas" à paintComponent...

Salut,


C'est le setBackground() qui pose problème. Pas le reste.
Lorsque tu fais un setBackground() tu changes les propriétés du composant, ce qui génèrera un repaint() pour prendre en compte la modification, d'où l'appel en boucle à paintComponent().

:arrow: paintComponent() doit uniquement se contenter de dessiner le composant, sans le modifier !

Citation:

---

Envoyé par HNoury

un cercle noir s'affiche et non blanc, comme quoi la couleur "ne parvient pas" à paintComponent...

---

Tu es vraiment sûr que tu lui passes la bonne couleur ???


a++

Merci adiGuba pour ta réponse.

En effet, les autres éléments de monFilm comme les entiers sont rappelables par paintCompnent mais pas les couleurs!! Je pensais que les couleurs peuvent être mises en tableaux comme les entiers ou les flottants. Est-ce que je me trompe?

Je stocke dans une classe "Color.blue" par exemple, et cette couleur n'est pas mémorisée : lorsque je l'appelle, j'ai un "null pointer exception" ou bien rien tout simplement : la couleur n'arrive pas à paintComponent!!

Mille excuse, tout ceci était la faute à une erreur dans le constructeur de CouleurFond. Tout ce que j'ai écrit est donc question sans lendemain.

Pardon!!:mouarf: