Discussion :

[Orian]

Bonjour, presque bonsoir,

Voila j'ai une image en fond de JFrame, mais elle ne s'affiche que si je redimensionne la JFrame. C'est contrariant...

J'ai bidouillé un truc avec des repaint et un ComponentListener, mais ça ne me semble pas très propre.

J'ai ajouté l'image de façon classique :

Code :

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

1
2
3
4
5
6
 public void paint(Graphics g) {
 
 
        g.drawImage(img, 0, 18, 200, 35, null);
        g.dispose();
    }

Une idée sur la cause du problème?

Merci

[bouye]

Comment charges-tu ton image ? Si c'est avec Toolkit, tu dois utiliser le mecanisme d'ImageObserver pour etre notifie de la fin du chargement de l'image en arriere-plan de maniere a rafraichir ton interface.

[Orian]

Non je n'utilse pas le Toolkit...Mais j'ai régler le problème avec un simple repaint. Je n'explique pas pourquoi mais :

Code :

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

1
2
3
4
5
6
7
public void paint(Graphics g) {
 
 
        g.drawImage(img, 0, 18, 200, 35, null);
        repaint();
        g.dispose();
    }

marche à merveille...
Et finalement tant mieux ça marche et je cherche pas à comprendre

[bouye]

Arf, ca m'apprendra a lire le code de maniere plus detaille meme s'il ne fait que 4 lignes :

- tu ne DOIS PAS faire dispose() sur le Graphics passe en parametre la methode paint() (ou paintComponent()). Tu dois cependant faire un dispose() sur une copie/vue creee avec Graphics.create().

- tu ne dois pas appeler repaint() d'une boucle de rendu (la methode que tu as surchargee fait partie du cyle EDT de repainture/propagation des evenements), cela peut provoquer des boucles infinies et autres incoherences et bugs du programme..

[Orian]

Arf, ca m'apprendra a lire le code de maniere plus detaille meme s'il ne fait que 4 lignes :

- tu ne DOIS PAS faire dispose() sur le Graphics passe en parametre la methode paint() (ou paintComponent()). Tu dois cependant faire un dispose() sur une copie/vue creee avec Graphics.create().

- tu ne dois pas appeler repaint() d'une boucle de rendu (la methode que tu as surchargee fait partie du cyle EDT de repainture/propagation des evenements), cela peut provoquer des boucles infinies et autres incoherences et bugs du programme..

Bon je résume : je code comme un bouraing . je vais corriger tout ça et voir si ça marche...

On apprend tout les jours...

[Orian]

Voilà j'ai corrigé j'ai retiré le repaint() et le g.dispose() et remplacé le ImageObserver 'null' par 'this' et ça marche. Il n'y a pas d'objection?

- tu ne DOIS PAS faire dispose() sur le Graphics passe en parametre la methode paint() (ou paintComponent()). Tu dois cependant faire un dispose() sur une copie/vue creee avec Graphics.create().

Alors question : il vaut mieux travailler sur une copie ou sur l'original?

Je m'excuse. Coder est un loisir et je fais ça en autodidacte donc j'ai souvent du mal avec les subtilités ce qui fait que je ne code pas toujours très proprement

[bouye]

Pas de probleme.

Dans un cas aussi simple, cela ne sert a rien de faire un copie.

Une copie peut etre utile :

- pour restreindre la zone de clip.

- pour eviter d'avoir a defaire tout un tas de modifications (clip, RenderingHints, Font, Stroke, Paint, Composite, ...) et de transformation (scale(), rotate(), translate(), setTransform(), ...) pour lesquelles il aurait fallu garder des copies des valeurs d'origines. De plus compte tenu de la precision des nombres, on ne revient pas forcement a l'etat initiale quand on fait une rotation par l'angle opposee par exemple en fin de dessin.

En general, depuis pas mal de temps, mes squelettes d'operation de dessin ressemblent a ca :

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
/** {@inheritDoc}
 */
@Override protected void paintComponent(Graphics g) {
  Dimension size = getSize();
  Insets insets = getInsets();
  int width = size.width - (insets.left + insets.right);
  int height = size.height - (insets.top + insets.bottom);
  Graphics2D g2d = (Graphics2D)g.create(insets.left, insets.top, width, height);
  try {
    ...
  }
  finally {
    g2d.dispose();
  }
}