Ajouter à ma frame fenêtre, une nouvelle classe (un panel)

**kirua99** · 15/03/2013, 08h16

Bonjour.

J'ai un problème sur une chose avec Swing

J'aimerai pouvoir appeler d'une frame "fenetre" appeler une nouvelle classe qui s'appelle "damier" et pouvoir l'ajouter.

Mais je fais ça :

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
public class Fenetre  extends JFrame{
private Damier unDamier;
...
 
public Fenetre(int[][]  Matrice_debutJeux){
unDamier = new Damier(Matrice_debutJeux);
this.getContentPane().add(unDamier, BorderLayout.CENTER);
...
}
 
// ma classe damier
 
public class Damier extends JPanel {
public Damier(int[][]  Matrice_debutJeux) {
 
..
 for (int i = 0; i <=  1 ; i++) { 
	            for (int j = 0; j <= 2; j++) { 
	            ["+j+"] =" + Matrice_debutJeux[i][j] );
	            	 lb = new JLabel();
		            	if ( Matrice_debutJeux[i][j] == 1 )
		            	{
		            		System.out.println("ok");
		            		lb.setBackground(Color.white);
 
                             } 
	        }
	}
 
//

En gros je voudrai ajouter à ma frame fenêtre, un Panel dans autre classe
mais la matrice en paramètre est correct, mais aucun affichage. Comment puis-je faire ?

Merci d'avance.

**wax78** · 15/03/2013, 09h29

Faut peut être aussi ajouter tes JLabel dans ton damier, ce que je ne vois pas dans ton code.

**kirua99** · 16/03/2013, 13h56

malheureusement si j'en ai mis , c'est pour sa que cela me semble bizarre.
Peut être que je devrai afficher tout mon code ?

Parce que il rentre bien dans ma condition et affiche bien plusieurs fois ok, mais n'affiche pas mes labels , donc est ce que au niveau de mon code cela devrai marcher ou il y aurai quelque chose qui pourrais résoudre mon problème ?

Merci d'avance

**ok.Idriss** · 16/03/2013, 17h08

Bonjour.

Montre nous ton code avec les JLabel instanciés et ajoutés dans ta classe dérivée de JPanel, parce que là comme ça, on manque un peu de détails pour te répondre

Idriss

**welcome_59** · 16/03/2013, 18h13

Pour que tes JLabel soient visibles dans ton damier, tu dois les ajouter à ton JPanel:

Code :

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

1
2
3
4
//...
lb = new JLabel("");
//...
damier.add(lb);

Je te passe la question des layout managers.

Si c'est ta fenêtre qui ne s'affiche pas

Code :

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

fenetre.setVisible(true);

**kirua99** · 16/03/2013, 18h37

Merci

Voila mes variables de la classe PanelDamier et mon constructeur PanelDamier
------------------------------------

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
 
 
JLabel lb;
	public int[] []  Matrice_debutJeu ;
	JPanel p ;
	 JButton b ;
 
 
public PanelDamier(int[][]  Matrice_debutJeux) {
		// TODO Auto-generated constructor stub
 
		System.out.println(" coucou Panel Panier ");
		 System.out.println("Matrice_debutJeux[1][0] =" + Matrice_debutJeux[1][0] );
 
		 p = new JPanel();
		  b = new JButton();
 
		  p.add(b);
 
		p.setLayout(new GridLayout (2, 3));
		System.out.println(" matrice jeux  --  matrice pions");
		 for (int i = 0; i <=  1 ; i++) { 
	            for (int j = 0; j <= 2; j++) { 
 
 
 
	            	System.out.println(" PanelPanier : Matrice_debutJeux["+i+"]["+j+"] =" + Matrice_debutJeux[i][j] );
	            	 lb = new JLabel();
		            	if ( Matrice_debutJeux[i][j] == 1 )
		            	{
		            		System.out.println("ok");
		            		lb.setBackground(Color.white);
 
		                } 
		            	if ( Matrice_debutJeux[i][j] == 2 )
		            	{
		            		lb.setBackground(Color.blue);
 
		                }
		            	else
		            	{
		            		lb.setBackground(Color.orange);
		            	}
	               p.add(lb); 
 
	            } 
	        }

Et ici , j'appel mon JPanel dans ma classe Fenetre :
-------------------------------------------------------

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
 
private PanelDamier MonDamier;
...
JFrame plateau = new JFrame();
 
		 test = new JLabel("test");
 
		plateau.getContentPane().add(MonDamier, BorderLayout.CENTER);
 
 
		 plateau.add(test, BorderLayout.SOUTH);
		 plateau.setSize(500, 400);
		 plateau.setLocationRelativeTo(null); 
		 plateau.setResizable(false);
		 //plateau.pack();
		 plateau.setVisible(true);

j’espère que cela pourra vous m'aider.

Seb

**ok.Idriss** · 16/03/2013, 18h54

Dans ton premier extrait : je ne vois pas à quel moment le JPanel p est ajouté dans une autre JComponent (JFrame, JPanel, etc). Ce JPanel n'est donc pas visible à priori.

Idriss

**kirua99** · 16/03/2013, 19h01

Le but est que dans ma classe je n'ai qu'un seul JPanel , je pense pas que je pourrai ajouter mon JPanel p à un JFrame , pour ajouter cette JFrame dans une autre JFrame.

Je ne peux pas avoir simplement un JPanel dans une classe et l'ajouter par la suite dans une JFrame dans une autre classe ?

Seb

**ok.Idriss** · 17/03/2013, 12h03

Re-bonjour.

Si c'est possible de plusieurs façon, par exemple (bien sûr d'autres solutions sont possibles) :

- Soit ta classe dérive de JPanel mais dans ce cas, les appels aux méthodes de JPanel se font sur this dans ta classe dérivée et non pas un objet appelé au sein de classe dont tu ne fait rien par la suite (p dans ton cas). Et dans l'autre classe, tu considère l'objet de ta classe instancié comme un JPanel.

- Soit tu fait de la délégation mais dans ce cas tu doit laisser un moyen de récupérer la référence de ton objet JPanel au niveau de ta classe : un getter par exemple. Ici p est public (je ne suis pas très fan) mais tu ne le récupère pas dans ta JFrame.

Bref, j'ai l'impression que tu fait un mélange de ces deux solutions qui bien évidemment ne fonctionne pas : une classe qui dérive de JPanel mais qui instancie et fait appel aux méthodes d'un attribut interne de type JPanel et n'en fait rien au niveau de ta JFrame.

Idriss

**kirua99** · 17/03/2013, 14h15

Merci Idriss.
Ton aide m'a apporté une petite amélioration mais j'ai mon JPanel qui ne ce s'affiche pas.

Je pense que c'est mon JPanel P qui ne veux pas s ajouter à ma classe PanelDamier qui hérite de JPanel.

dans la chronologie ça ferai sa :

Code :

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

1
2
3
4
5
6
7
8
9
p = new JPanel();
 b = new JButton();
this.add(b)
// mes boucles
 lb = new JLabel();
dans une condition :  p.add(mon lb de couleur ) 
// je sort de mes boucles 
this.add(b);
this.add(p);

le boutton s'affiche bien mais pas le JPanel.
Si quelqu'un pourrait me conseiller !!

Seb

**ok.Idriss** · 17/03/2013, 14h42

Juste avant ton this.add(p) est-ce que tu peux ajouter quelque chose dans ton panel comme un JButton ou un JLabel avec une valeur "toto" histoire d'être sûr que ton JPanel n'est pas ajouté.

Essayes de faire du step by step (pas à pas) avec ton EDI afin de vérifier que tu rentre bien dans ta condition lorsque tu change la couleur de ton panel. Pareil, met un point d’arrêt sur ton this.add(p) pour être sûr que tu passes dessus.

Bref, va falloir faire un peu de debug

**kirua99** · 17/03/2013, 15h03

Alors ,
J'ai remplacer mes JPanel par des JButton , et ça marche ...
Donc y aurait'il un problème avec une classe héritant d'un composant qu'on veut utiliser par la suite ? ...

**ok.Idriss** · 17/03/2013, 15h06

Tes JPanel sont peut être vide au moment ou tu les ajoutes. Essayes de forcer l'ajout de composants visibles dans tes JPanel juste avant l'ajout de ces derniers.

**kirua99** · 17/03/2013, 15h19

Alors j'ai fais des pas mal de modif en plus de que tu m'a dis , donc maintenant les JLabel fonctionnent bien mais c'est l'attribut couleur qui ne marche pas, avec une position assez bizarre faudra que je regarde ça de plus prés.

C'est

Code :

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