Ne pas appeler le paintComponent de la classe mère

Version imprimable

Bonsoir,

J'ai, pour simplifier, 4 classes. Voici les codes qui m'intéressent dedans :

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
32
33
34
35
36
37
38
 
public class GraphComponent extends JComponent {
  public GraphComponent() {
  }
  protected void paintComponent(Graphics g) {
     ...
  }
}
 
public class GraphFrame extends JFrame {
  GraphComponent component= new GraphComponent();
  private FramesController controller;
  public GraphFrame(FramesController controller) {
		this.controller = controller;
 
		component.setForeground(Color.BLACK);
		component.setBackground(Color.WHITE);
		component.setOpaque(true);
  }
}
 
public class AutomatonComponent extends GraphComponent{
  public AutomatonComponent() {
		super();
  }
 
  protected void paintComponent(Graphics g) {
     ...
  }
}
 
public class AutomatonFrame extends GraphFrame{
  public AutomatonComponent comp=new AutomatonComponent();
  public AutomatonFrame(FramesController controller) {
		super(controller);
                comp.repaint();
  }
}

Mon problème est le suivant : quand j'arrive à l'instruction comp.repaint() de AutomatonFrame, c'est le paintComponent de GraphComponent qui est appelé, et non celui de AutomatonComponent...
J'ai essayé de faire un cast :

Code:

((AutomatonComponent)comp).repaint();

Mais du coup ça ne m'affiche plus rien !

Des idées ?
Merci.

06/12/2008, 23h38
tchize_

Citation:

Envoyé par brunoperel

Mon problème est le suivant : quand j'arrive à l'instruction comp.repaint() de AutomatonFrame, c'est le paintComponent de GraphComponent qui est appelé, et non celui de AutomatonComponent...
J'ai essayé de faire un cast :

Code:

((AutomatonComponent)comp).repaint();

Mais du coup ça ne m'affiche plus rien !

Fait un nettoyage de tes classe compilés et recompile bien tout. Y a aucune raison pour que le paintComponent de AutomatonComponent ne soit pas appelée, comme il n'y a aucune raison pour qu'il y aie une différence entre comp.repaint() et ((AutomatonComponent)comp).repaint() puisque comp est déclaré comme un AutomatonComponent (le typecasting sert à rien)
07/12/2008, 11h16
brunoperel

Bonjour, et merci pour ta réponse,

J'ai bien nettoyé tout le projet comme tu me l'as conseillé, mais ça n'a pas l'air de résoudre mon problème.

J'ai essayé, à tout hasard, de remplacer tous les

Code:

comp.repaint();

de AutomatonFrame par

Code:

comp.paintComponent(getGraphics());

Et là, ça marche ! Enfin, il y a plein de problèmes de positionnement et tout, mais en tout cas le paintComponent de AutomatonComponent est bien appelé (j'ai vérigier avec le debugger).
Y a-t-il une différence entre appeler paintComponent et appeler repaint() ?
07/12/2008, 18h15
tchize_

Quelle est la taille de GraphComponent? Dans la doc java, il est stipulé que, lors d'un appel à paint(Graphics g), si le composant a une largeur ou une hauteur de 0, aucune action n'est effectuée.
07/12/2008, 19h11
brunoperel

Citation:

Envoyé par tchize_

Quelle est la taille de GraphComponent? Dans la doc java, il est stipulé que, lors d'un appel à paint(Graphics g), si le composant a une largeur ou une hauteur de 0, aucune action n'est effectuée.

Mais justement, maintenant je me rends compte que le paintComponent de AutomatonComponent est appelé si j'appelle directement cette méthode, alors que si j'essaie de l'appeler via repaint(), ça ne marche pas !
07/12/2008, 19h40
tchize_

repaint() appelle paint() qui, éventuellement, appelle paintComponent(). Donc si ton composant a une taille nulle, paintComponent() ne sera pas appelé. Comme tu l'a dit, si tu l'appelle directement, sa marche, mais pas si tu passe par paint. Montre nous comment ton composant est positionné et dimensionné. Surcharge-tu les méthode getMinimumSize() et getPreferredSize() qui servent à dimensionner ton composant?

Oui, dans GraphFrame (dont hérite AutomatonFrame), ce setPreferredSize est déclaré. En fait, pour expliquer un peu mieux ma situation, avant je ne possédais que GraphFrame/Editor/Component, mais comme je souhaitais séparer ce que j'avais comme code source au départ, j'ai créé AutomatonFrame/Editor/Component, et c'est là que ça n'a plus marché du tout...
Code:

1 2 3 4 5 6 7 8 9 10 11 12 public class GraphFrame extends JFrame { GraphComponent component= new GraphComponent(); private FramesController controller; public GraphFrame(FramesController controller) { this.controller = controller; component.setForeground(Color.BLACK); component.setBackground(Color.WHITE); component.setOpaque(true); component.setPreferredSize(new Dimension(1000, 1000)); } }
Avant, quand je n'avais pas encore les classes Automaton, les repaint() marchaient sans problème...:(

08/12/2008, 00h21
tchize_

il manquerait pas un add() du composant par hasard? Si le composant n'est pas placé dans un conteneur, il ne risque pas de se dessiner.

Citation:

Envoyé par tchize_

il manquerait pas un add() du composant par hasard? Si le composant n'est pas placé dans un conteneur, il ne risque pas de se dessiner.

Non, il est dans GraphFrame en tout cas !
Je sais pas si c'est "raisonnable" de poster tout le code ici : il est assez long pour un message de forum (1058 lignes, je viens de calculer)...

Voici GraphFrame, dans une version épurée...

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
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
 
public class GraphFrame extends JFrame {
	GraphComponent component= new GraphComponent();
	private FramesController controller;
 
	protected JMenuBar menuBar = new JMenuBar();
	protected JMenu menu;
 
	public GraphFrame(FramesController controller) {
		this.controller = controller;
 
		component.setForeground(Color.BLACK);
		component.setBackground(Color.WHITE);
		component.setOpaque(true);
		component.setPreferredSize(new Dimension(1000, 1000));
		JScrollPane scrollPane = new JScrollPane(component);
 
		setJMenuBar(menuBar);
		menu = new JMenu(GraphEditor.MENU_FILE);
		menuBar.add(menu);
 
		createMenuItem(menu, GraphEditor.MENU_ITEM_NEW, new ActionListener() {
			public void actionPerformed(ActionEvent ev) {
				GraphFrame.this.controller.createFrame();
			}
		});
		createMenuItem(menu, GraphEditor.MENU_ITEM_CLOSE, new ActionListener() {
			public void actionPerformed(ActionEvent ev) {
				GraphFrame.this.controller.deleteFrame(GraphFrame.this);
			}
		});
		createMenuSeparator(menu);
		createMenuItem(menu, GraphEditor.MENU_ITEM_QUIT, new ActionListener() {
			public void actionPerformed(ActionEvent ev) {
				GraphFrame.this.controller.quit();
			}
		});
 
		addWindowListener(new WindowAdapter() {
			public void windowClosing(WindowEvent e) {
				GraphFrame.this.controller.deleteFrame(GraphFrame.this);
			}
		});
 
		JToolBar toolbar = new JToolBar();
		toolbar.setLayout(new GridLayout(0, 1));
		addShapeButton(toolbar, new Ellipse2D.Double(0, 0, 10, 10), "Small Circle");
addShapeButton(toolbar, new Ellipse2D.Double(0, 0, 20, 20), "<html>Big Circle<br>(Final state)</html>");
		addShapeButton(toolbar, new Rectangle2D.Double(0, 0, 10, 10), "<html>Small Square<br>(Intial&final<br>state)</html>");
		addShapeButton(toolbar, new Rectangle2D.Double(0, 0, 20, 20), "<html>Big Square<br>(Initial state)</html>");
 
		Container contentPane = getContentPane();
		contentPane.add(toolbar, BorderLayout.WEST);
		contentPane.add(scrollPane, BorderLayout.CENTER);
		//menu = new JMenu(AutomatonEditor.MENU_EDITOR);
		//menuBar.add(menu);
	}
 
	private void addShapeButton(JToolBar toolbar, final RectangularShape sample, String name) {
		JButton button = new JButton(name);
		button.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				component.setVertexSample(sample);
			}
		});
		toolbar.add(button);
	}
 
	protected void createMenuItem(JMenu menu, String name, ActionListener action) {
		JMenuItem menuItem = new JMenuItem(name);
		menuItem.addActionListener(action);
		menu.add(menuItem);
	}
 
	private void createMenuSeparator(JMenu menu) {
		JSeparator separator = new JSeparator();
		separator.setForeground(Color.lightGray);
		menu.add(separator);
	}
}

Et voici la version allégée pour le forum de AutomatonFrame :

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
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
 
public class AutomatonFrame extends GraphFrame{
	JMenu menu_editor;
	public AutomatonComponent comp=new AutomatonComponent();
 
 
	public AutomatonFrame(FramesController controller) {
		super(controller);
 
		menu_editor= new JMenu(AutomatonEditor.MENU_EDITOR);
 
		menuBar.add(menu_editor);
		createMenuItem(menu, AutomatonEditor.MENU_ITEM_OPEN, new ActionListener() {
			public void actionPerformed(ActionEvent ev) {
				final JFileChooser jfc = new JFileChooser();
				FileFilter filter = new FileNameExtensionFilter("GraphEditor file (*.ged, *.dat)", "ged", "dat");
				jfc.addChoosableFileFilter(filter);
				int open_file_confirmation = jfc.showOpenDialog(comp);
				if (open_file_confirmation == JFileChooser.APPROVE_OPTION) {
		            File file = jfc.getSelectedFile();
		            BufferedReader br=null;
		            String line;
			    	RectangularShape rs=null;
			    	System.out.println("Opening "+file.getName()+"...");
			    	try {
		            	br = new BufferedReader (new FileReader(file.toString()));
			        }
			        catch(FileNotFoundException exc) {
			        	System.out.println("Open error");
			        }
			        try {
				    	comp.vertices.removeAll(comp.vertices);
				    	comp.edges.removeAll(comp.edges);
				    	GraphComponent.n=0;
				    	comp.removeAll();
				    	comp.repaint();
 
			        	Edge current_edge=null;
			        	while ((line = br.readLine()) != null) {
			        		/* Lecture du fichier */
						comp.paintComponent(getGraphics());
						comp.repaint();
				        br.close();
				        System.out.println("The file "+file.getName()+" has been opened.");
					} 
			        catch (IOException e) {
						e.printStackTrace();
					}
 
 
		        } else {
		           System.out.println("File opening has been canceled");
		        }
 
			}
 
		});
 
 
	}
 
}

tchize_, des suggestions ? Je suis désespérééééé :cry:

09/12/2008, 20h36
tchize_

peux-tu définir un border() à ton composant (un gros truc rouge vif par exemple) pour voir quelle taille il fait à l'exécution?