paintComponent appelé 2 fois.

**pjmorce** · 12/05/2014, 08h58

Bonjour

J'ai un programme AWT avec lequel j'essaye de faire en sorte de dessiné un stackbarchart.
Cependant, il a un problème: la méthode paintComponent est appelée 2 fois.

J'ai simplifié le tout afin de pouvoir identifier le problème au minimum, cependant sans succès. Le code suivant est simple et affiche simplement une fenêtre vide. Cependant, dans ce cas là le problème existe aussi (la méthode en cause est aussi appelée et exécutée 2 fois).

Savez vous pourquoi? comment puis-je résoudre le problème?

merci et bonne journée

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
package tstswing;
import java.awt.Graphics;
import java.awt.event.WindowAdapter;
import java.awt.event.WindowEvent;
import java.awt.event.WindowListener;
 
import javax.swing.JFrame;
import javax.swing.JPanel;
 
public class TestPanel extends JPanel {
 
	private static final long serialVersionUID = 1L;
 
	public TestPanel(){}
 
	public void paintComponent(Graphics g) {
		System.out.println("dddddddddddd");
	}
 
	public static void main(final String[] args) {
		final JFrame f = new JFrame();
 
		f.setSize(600, 400);
	    f.getContentPane().add(new TestPanel());
 
	    WindowListener wndCloser = new WindowAdapter() {
	      public void windowClosing(WindowEvent e) {
	        System.exit(0);
	      }
	    };
	    f.addWindowListener(wndCloser);
	    f.setVisible(true);
	}
 
}

**joel.drigo** · 12/05/2014, 11h11

Salut,

Les méthodes paint() des composants graphiques, ou les méthodes appelées indirectement par paint(), comme paintComponent() par exemple, sont appelées chaque fois que SWING l'estime nécessaire, ou consécutivement à une demande volontaire (lorsque le programmeur sait qu'il faut rafraîchir alors que le changement effectué ne peut être connu de SWING sans qu'on lui "dise"), comme par l'appel d'un repaint(). Ceci est tout à fait normal, le but de cette méthode étant de rafraîchir l'affichage.

PS : néanmoins, quand je teste ton programme (sans rien faire d'autre qu'une exécution), je n'ai qu'un seul passage dans la méthode paintCompontent. Tu peux avoir d'autres appels si tu redimensionnes la fenêtre par exemple.

**pjmorce** · 12/05/2014, 11h46

Merci pour la réponse.

Le fait que le code que j'ai envoyé sois exécuté 2 fois sur mon PC contrairement à ton cas, peut être lié à quoi? (prendre en compte que j'exécute seulement le code sous eclipse et le "dddddddddddd" apparait 2 fois sur ma console (je ne touche pas à la fenêtre qui est affichée. Pas de resize, rien...)

Dans ma "vrai" version de l'appli, j'ai du code ou je construit les composants de ma fenêtre. Actuellement, c'est comme c'est exécuté 2 fois, ça me pose des problèmes. Est-ce le meilleur moyen de le faire? devrais-je mettre le code dans une autre méthode?

Merci bien.

Envoyé par joel.drigo

Salut,

Les méthodes paint() des composants graphiques, ou les méthodes appelées indirectement par paint(), comme paintComponent() par exemple, sont appelées chaque fois que SWING l'estime nécessaire, ou consécutivement à une demande volontaire (lorsque le programmeur sait qu'il faut rafraîchir alors que le changement effectué ne peut être connu de SWING sans qu'on lui "dise"), comme par l'appel d'un repaint(). Ceci est tout à fait normal, le but de cette méthode étant de rafraîchir l'affichage.

PS : néanmoins, quand je teste ton programme (sans rien faire d'autre qu'une exécution), je n'ai qu'un seul passage dans la méthode paintCompontent. Tu peux avoir d'autres appels si tu redimensionnes la fenêtre par exemple.

**joel.drigo** · 12/05/2014, 11h56

En test (uniquement en test), tu peux faire (new Throwable()).printStackTrace(); au lieu de System.out.println("dddddddddddd");, pour voir comment est appelée cette méthode. Malheureusement, en cas d'appel par SWING, tu auras une stacktrace du genre :

java.lang.Throwable
	at tstswing.TestPanel.paintComponent(TestPanel.java:17)
	at javax.swing.JComponent.paint(JComponent.java:1054)
...
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:138)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:91)

qui ne te permettra pas donc de connaître exactement la raison de l'invocation de la méthode, mais te garantira au moins que c'est bien un appel SWING, et non un appel direct à paint/paintComponent, ou un appel indirect, par exemple dans un actionPerformed, ou autre évènement.

Dans le cas précis de ton programme (celui posté), je ne vois pas pourquoi il y a 2 appels : je teste également sous Kepler, en java 1.7, et je n'ai qu'un appel, celui dû à l'affichage initial (l'appel de f.setVisible(true);), donc normal.

**joel.drigo** · 12/05/2014, 11h59

Envoyé par pjmorce

Dans ma "vrai" version de l'appli, j'ai du code ou je construit les composants de ma fenêtre. Actuellement, c'est comme c'est exécuté 2 fois, ça me pose des problèmes. Est-ce le meilleur moyen de le faire? devrais-je mettre le code dans une autre méthode?

Si ça te pose des problèmes, je suppose que c'est soit dû à des problèmes de performances, de blocage de ressources, ou de duplication d'instances éventuellement : dans ces cas, oui, la meilleure façon de procéder est de créer les éléments en dehors du thread graphique, donc en dehors de toutes méthodes d'affichage, ou de réaction à un évènement. Ceci inclus également les méthodes appelées indirectement, comme celles appelées dans des constructeurs de composants graphiques.

Avec plus de détail sur ton code, on pourrait te donner des réponses plus précises.

**pjmorce** · 12/05/2014, 12h28

Pas de problèmes.

Je suis en phase initiale de développement d'une classe qui serait utilisée pour gérer un stackedbarchart.

Pour l'instant je encore très au début. Le résultat en gros est celui que j'attendais jusqu'à présent, cependant, il y a toujours 2 exécutions du code, bien que je ne fasse rien (que je sache) pour.

Le code le voici:

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
 
 
package chart;
 
import java.awt.Color;
import java.awt.Dimension;
import java.awt.Font;
import java.awt.FontMetrics;
import java.awt.Graphics;
import java.awt.event.WindowAdapter;
import java.awt.event.WindowEvent;
import java.awt.event.WindowListener;
import java.util.ArrayList;
import java.util.List;
 
import javax.swing.JFrame;
import javax.swing.JPanel;
 
import objects.CSVCellInfo;
 
public class BarChartPanel extends JPanel {
 
	private List<List<CSVCellInfo>> values;
	private String[] filenames;
	private String title;
 
	/**
         * 
         */
	private static final long serialVersionUID = 1L;
 
	public BarChartPanel(final List<List<CSVCellInfo>> values, final String[] filenames, final String title) {
		this.values = values;
		this.filenames = filenames;
		this.title = title;
	}
 
	public void paintComponent(Graphics g) {
		super.paintComponent(g);
System.out.println("a");		
		if(values == null || values.size() == 0)
			return;
 
		//final double minValue = 1;
	    final double maxValue = 1;
 
	    final Dimension d = getSize();
	    final int clientWidth = d.width;
	    final int clientHeight = d.height;
	    final int barWidth = clientWidth / values.size();
 
	    final Font titleFont = new Font("SansSerif", Font.BOLD, 20);
	    final FontMetrics titleFontMetrics = g.getFontMetrics(titleFont);
	    final Font labelFont = new Font("SansSerif", Font.PLAIN, 10);
	    final FontMetrics labelFontMetrics = g.getFontMetrics(labelFont);
 
	    final int titleWidth = titleFontMetrics.stringWidth(title);
	    int y = titleFontMetrics.getAscent();
	    int x = (clientWidth - titleWidth) / 2;
	    g.setFont(titleFont);
	    g.drawString(this.title, x, y);
 
	    final int top = titleFontMetrics.getHeight();
	    final int bottom = labelFontMetrics.getHeight();
	    /*if (maxValue == minValue)
	      return;*/
	    //final double scale = (clientHeight - top - bottom) / (maxValue - minValue);
 
	    final List<CSVCellInfo> tcells = values.get(0);
	    int num = 3;
	    if(tcells.size()>1)
	    	num = tcells.size()+1;
//System.out.println("=>" + tcells.size());
	    final double scale = (clientHeight - top - bottom)/num;
//System.out.println("scale: " + scale);
	    y = clientHeight - labelFontMetrics.getDescent();
	    g.setFont(labelFont);
 
 
	    for (int i = 0; i < values.size(); i++) {
	    	int valueX = i * barWidth + 1;
	    	int valueY = top;
//System.out.println("valueY: " + valueY);
	    	int height = (int) (maxValue * scale);
	    	valueY += (int) (maxValue * scale) ;
	    	valueY-=y / scale;
	    	//System.out.println("Y: " + y / scale);
 
	    	final List<CSVCellInfo> cells = values.get(i);
	    	for(final CSVCellInfo cellInfo : cells) {
//System.out.println("X: " + valueX + "; Y: " + valueY + "; barWidth: " + barWidth + "; height: " + height);
	    		final Color color = CSVCellInfo.getColor(cellInfo);
	    		g.setColor(color);
	    		g.fillRect(valueX, valueY, barWidth - 2, height);
	    		g.setColor(Color.black);
 
	    		int labelWidth = labelFontMetrics.stringWidth(cellInfo.getValue());
	    		int xpos = i * barWidth + (barWidth - labelWidth) / 2;
	    		g.drawString(cellInfo.getValue(), xpos, valueY+(height/2));
	    		g.drawRect(valueX, valueY, barWidth - 2, height);
	    	    valueY += (int) (maxValue * scale);
	    	}
	    	int labelWidth = labelFontMetrics.stringWidth(this.filenames[i]);
	    	x = i * barWidth + (barWidth - labelWidth) / 2;
//System.out.println("names[i]: " + this.filenames[i] + "; x: " + x + "; y: " + y);
	    	g.drawString(this.filenames[i], x, y);
	    }
	}
 
	public static void main(String[] argv) {
	    JFrame f = new JFrame();
	    f.setSize(600, 400);
	    //double[] values = new double[3];
	    final List<List<CSVCellInfo>> values = new ArrayList<List<CSVCellInfo>>();
	    String[] filenames = new String[3];
 
	    final List<CSVCellInfo> valuesFil1 = new ArrayList<CSVCellInfo>();
	    final CSVCellInfo f1CSVCI = new CSVCellInfo("Name", 3);
	    final CSVCellInfo f2CSVCI = new CSVCellInfo("Id", 0);
	    final CSVCellInfo f3CSVCI = new CSVCellInfo("Born date", 1);
	    valuesFil1.add(f1CSVCI);
	    valuesFil1.add(f2CSVCI);
	    valuesFil1.add(f3CSVCI);
	    values.add(valuesFil1);
	    filenames[0] = "File 1";
 
	    final List<CSVCellInfo> valuesFil2 = new ArrayList<CSVCellInfo>();
	    final CSVCellInfo f1CSVCI2 = new CSVCellInfo("Id", 0);
	    final CSVCellInfo f2CSVCI2 = new CSVCellInfo("Name", 3);
	    final CSVCellInfo f3CSVCI2 = new CSVCellInfo("Born date", 1);
	    valuesFil2.add(f1CSVCI2);
	    valuesFil2.add(f2CSVCI2);
	    valuesFil2.add(f3CSVCI2);
	    values.add(valuesFil2);
	    filenames[1] = "File 2";
 
	    final List<CSVCellInfo> valuesFil3 = new ArrayList<CSVCellInfo>();
	    final CSVCellInfo f1CSVCI3 = new CSVCellInfo("Born date", 1);
	    final CSVCellInfo f2CSVCI3 = new CSVCellInfo("Name", 3);
	    final CSVCellInfo f3CSVCI3 = new CSVCellInfo("Id", 0);	    
	    valuesFil3.add(f1CSVCI3);
	    valuesFil3.add(f2CSVCI3);
	    valuesFil3.add(f3CSVCI3);
	    values.add(valuesFil3);
	    filenames[2] = "File 3";
 
	    f.getContentPane().add(new BarChartPanel(values, filenames, "Test"));
 
	    WindowListener wndCloser = new WindowAdapter() {
	      public void windowClosing(WindowEvent e) {
	        System.exit(0);
	      }
	    };
	    f.addWindowListener(wndCloser);
	    f.setVisible(true);
	  }
 
}

Merci.

**joel.drigo** · 12/05/2014, 13h49

Toujours qu'un appel en ce qui me concerne. Et c'est quoi, le code de CSVCellInfo, des fois que ça vienne de là ?

**pjmorce** · 12/05/2014, 15h24

Non, CSVCellInfo est juste une classe pour aider, n'ayant rien de Swing...

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
 
 
package objects;
 
import java.awt.Color;
 
public class CSVCellInfo {
 
	private String header;
	private String value;
 
	// case -1: Value empty.
	// case 0: It is a numeric.
	// case 1: It is a date.
	// case 2: It is a percentage.
	// case 3: It is a string.
	private int type;
 
	public CSVCellInfo(final String header, final String value, final int type) {
		this.header = header;
		this.value = value;
		this.type = type;
	}
 
	public CSVCellInfo(final String value, final int type) {
		this.value = value;
		this.type = type;
	}
 
	public String getValue() {
		return value;
	}
 
	public void setValue(final String value) {
		this.value = value;
	}
 
	public int getType() {
		return type;
	}
 
	public void setType(final int type) {
		this.type = type;
	}
 
	public String getHeader() {
		return header;
	}
 
	public void setHeader(String header) {
		this.header = header;
	}
 
	public static Color getColor(final CSVCellInfo cellInfo) {
		final int type = cellInfo.getType();
		switch(type) {
			case -1:
				// Unknown
				return Color.gray;
			case 0:
				// Numeric
				return Color.yellow;
			case 1:
				// Date
				return Color.green;
			case 2:
				// Percentage
				return Color.orange;
			case 3:
				// String
				return Color.blue;
		}
		return Color.gray;
	}
}

**pjmorce** · 12/05/2014, 15h25

Tu as exécuté le code sous quel genre d'environnement?
Windows? linux?
Sous eclipse?
Quelle version Java?

Merci

Envoyé par pjmorce

Non, CSVCellInfo est juste une classe pour aider, n'ayant rien de Swing...

**joel.drigo** · 12/05/2014, 16h01

Windows 8.1/Eclipse Kepler SR1/Java 1.7.u51

Normalement, ça ne devrait pas dépendre de l’environnement, mais c'est vrai que parfois il y a quelques différences dans les implémentations. M'enfin, le nombre d'appel de paint() m'étonnerait qu'il soit différent sous linux, windows ou mac.

**pjmorce** · 13/05/2014, 11h57

Bon. Je ne sais pas.

Je viens d'installer Eclipse Kepler. J'ai Java 7. La seule différence est Windows (Windows 7 Professionel).

Je ne vois pas d'où ça peut venir...

Envoyé par joel.drigo

Windows 8.1/Eclipse Kepler SR1/Java 1.7.u51

Normalement, ça ne devrait pas dépendre de l’environnement, mais c'est vrai que parfois il y a quelques différences dans les implémentations. M'enfin, le nombre d'appel de paint() m'étonnerait qu'il soit différent sous linux, windows ou mac.

**joel.drigo** · 13/05/2014, 12h26

Là, je ne vois pas pourquoi tu aurais 2 exécutions de paintComponent(). Ça ne dépend évidemment pas de Eclipse. Et je ne pense pas qu'il y ait une différence sous Windows 7 en particulier. Tu as essayé l'affichage de la stacktrace ?

D'un autre côté, ça n'a pas une grande importance vu ton code : il n'est pas très consommateur. Tu pourrais créer tes polices en constante de la classe éventuellement. Ou encore stocker la couleur directement, plutôt que l'évaluer avec getColor(final CSVCellInfo cellInfo) (m'enfin on parle de gagner des chouièmes de seconde). Pour le reste, je ne pense pas que l'ajout d'un "cache" soit nécessaire, ou autre (genre utilisé un textlayout par exemple). Bien sûr, si le code devait évoluer, et faire (beaucoup) plus, ou si ton composant est utilisé de très nombreuses fois, et surtout si tu as besoin de rafraîchir souvent l'affichage. il faudra peut être travailler son optimisation pour éviter de bloquer trop longtemps l'UI. Tu verras à l'usage si l'affichage manque de fluidité, ou si le premier affichage est visiblement trop différé.

**tchize_** · 13/05/2014, 14h24

Vous vous triturez la nenette pour rien, il peut y avoir un tas de raisons légitimes à ce double appel. Par exemple, si son environnement est plus lent ou plus rapide que le votre, l'affichage pourrais commencer avant la fin de l'initialisation de certains éléments, et swing en conclu qu'il faut rafraichire. Ou la fenetre apparaitrait, à cause de son window manager, d'abord masquée puis passerais à l'avant plan, ... etc.

**joel.drigo** · 13/05/2014, 14h33

Oui, je suis d'accord @Tchize_, de toute manière, le paint() sera appelé quand SWING le "demandera", au moins. Ce que j'ai dit dès le départ. Mais comme ça semblait embêter @pjmorce, ma foi, et qu'il veut savoir pourquoi et éventuellement l'empêcher, il fait ce qu'il veut.
Maintenant, je ne vois pas pourquoi on aurait un appel avant le setVisible(true) de la frame, quelque soit la vitesse et la réactivité de l'environnement. Je ne cherche pas à le savoir non plus

Le principe c'est juste qu'il faut s'arranger pour que le code "dans paint()" se fasse le plus vite possible pour ne pas trop "figer" l'UI et surtout éviter d'y faire des choses redondantes, parce que toute manière, il sera possiblement appelé un nombre non déterminable de fois.

paintComponent appelé 2 fois.

AWT/Swing Java

Discussions similaires

Partager

Partager