Discussion :

[brunoperel]

Bonsoir,

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

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
 
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 :

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

((AutomatonComponent)comp).repaint();

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

Des idées ?
Merci.

[tchize_]

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 :

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

((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)

[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 :

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

comp.repaint();

de AutomatonFrame par

Code :

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

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() ?

[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.

[brunoperel]

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 !

[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?

[brunoperel]

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 :

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

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...

[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.

[brunoperel]

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 :

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
 
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 :

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
 
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ééééé

[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?