Problème avec GridBagLayout

**JeremGamer** · 29/09/2013, 11h31

Bonjour,

J'ai un problème avec un GridLayoutBagLayout: je pense avoir fait ça correctement (dites-moi si c'est faux^^) mais la seconde ligne de ma grille ne s'affiche pas... Je pense que c'est dû au .setPreferredSize() de mes composants mais je n'en suis pas sûr.

Voici mon code: (j'ai supprimé les import et package pour alléger)

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
 
public class TodayArticles extends JPanel {
 
	public JPanel content = new JPanel();
 
 
	public TodayArticles() {
 
		this.setPreferredSize(getMaximumSize());
		content.setLayout(new GridBagLayout());
 
		JPanel headline = new JPanel();	
		headline.setPreferredSize(getPreferredSize());
		headline.setBackground(Color.orange);
 
		JPanel vertical = new JPanel();	
		vertical.setPreferredSize(getPreferredSize());
		vertical.setBackground(Color.cyan);
 
		JPanel botleft = new JPanel();
		botleft.setPreferredSize(getPreferredSize());
		botleft.setBackground(Color.green);
 
		JPanel botright = new JPanel();
		botright.setPreferredSize(getPreferredSize());
		botright.setBackground(Color.BLUE);
 
		GridBagConstraints gbc = new GridBagConstraints();
 
		gbc.gridx = gbc.gridy = 0;
		gbc.gridheight = gbc.gridwidth = 1;
		gbc.anchor = GridBagConstraints.BASELINE_LEADING;
 
		gbc.insets = new Insets(5, 5, 5, 5);
		content.add(headline, gbc);
 
		gbc.gridx = 1;
		gbc.gridwidth = GridBagConstraints.REMAINDER;
		gbc.anchor = GridBagConstraints.BASELINE_LEADING;
		gbc.insets = new Insets(5, 5, 5, 5);
		content.add(vertical, gbc);
 
		gbc.gridx = 0;
		gbc.gridy = 1;
		gbc.anchor = GridBagConstraints.BASELINE_LEADING;
		gbc.insets = new Insets(5, 5, 5, 5);
		content.add(botleft, gbc);
 
		gbc.gridx = 1;
		gbc.gridwidth = gbc.gridheight = GridBagConstraints.REMAINDER;
		gbc.anchor = GridBagConstraints.BASELINE_LEADING;
		gbc.insets = new Insets(5, 5, 5, 5);
		content.add(botright, gbc);
 
 
 
		this.add(content);
	}
 
}

Merci d'avance!

**joel.drigo** · 29/09/2013, 12h06

Salut,

essayes justement de mettre new Dimension(10,10) au lieu de getPreferedSize() et tu verras immédiatement que c'est bien ça : le preferedSize de TodayArticles étant égal au maximumSize (ce que tu forces au début de la construction).
Comme tu as n'a pas mis de maximumSize, ça prend la maximumSize par défaut, soit new Dimension(Short.MAX_VALUE, Short.MAX_VALUE) !

**JeremGamer** · 29/09/2013, 12h24

Le problème est là, j'ai testé pour voir si le ça venait du getPreferredSize() en mettant new Dimension à la place, et en effet ça fonctionne. MAIS, je voudrais que les composants headline, vertical, botleft et botright soit redimensionnés proportionnellement à la taille de la fenêtre. C'est pour ça que je pensais que le getPreferredSize() serait plus approprié...

**joel.drigo** · 29/09/2013, 13h18

pourquoi ne pas utiliser un GridLayout dans ce cas ?

Sinon modifie ton constructeur comme suit :

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
	public TodayArticles() {
 
		//this.setPreferredSize(getMaximumSize());
		content.setLayout(new GridBagLayout());
 
		JPanel headline = new JPanel();	
		//headline.setPreferredSize(getDPreferredSize());
		headline.setBackground(Color.orange);
 
		JPanel vertical = new JPanel();	
		//vertical.setPreferredSize(getDPreferredSize());
		vertical.setBackground(Color.cyan);
 
		JPanel botleft = new JPanel();
		//botleft.setPreferredSize(getDPreferredSize());
		botleft.setBackground(Color.green);
 
		JPanel botright = new JPanel();
		//botright.setPreferredSize(getDPreferredSize());
		botright.setBackground(Color.BLUE);
 
		GridBagConstraints gbc = new GridBagConstraints();
		gbc.gridx = gbc.gridy = 0;
		gbc.gridheight = gbc.gridwidth = 1;
		//gbc.anchor = GridBagConstraints.BASELINE_LEADING;
		gbc.weightx=gbc.weighty=0.5;
		gbc.fill=GridBagConstraints.BOTH;
		gbc.insets = new Insets(5, 5, 5, 5);
		content.add(headline, gbc);
 
		gbc = new GridBagConstraints();
		gbc.gridx = 1;
		gbc.gridy = 0;
		gbc.gridwidth = GridBagConstraints.REMAINDER;
		//gbc.anchor = GridBagConstraints.BASELINE_LEADING;
		gbc.weightx=gbc.weighty=0.5;
		gbc.fill=GridBagConstraints.BOTH;
		gbc.insets = new Insets(5, 5, 5, 5);
		content.add(vertical, gbc);
 
		gbc = new GridBagConstraints();
		gbc.gridx = 0;
		gbc.gridy = 1;
		//gbc.anchor = GridBagConstraints.BASELINE_LEADING;
		gbc.weightx=gbc.weighty=0.5;
		gbc.fill=GridBagConstraints.BOTH;
		gbc.insets = new Insets(5, 5, 5, 5);
		content.add(botleft, gbc);
 
		gbc = new GridBagConstraints();
		gbc.gridx = 1;
		gbc.gridy = 1;
		gbc.gridwidth = gbc.gridheight = GridBagConstraints.REMAINDER;
		//gbc.anchor = GridBagConstraints.BASELINE_LEADING;
		gbc.insets = new Insets(5, 5, 5, 5);
		gbc.weightx=gbc.weighty=0.5;
		gbc.fill=GridBagConstraints.BOTH;
		content.add(botright, gbc);
 
		setLayout(new BorderLayout());
		this.add(content, BorderLayout.CENTER);
		//this.add(content);
 
	}

tu remarqueras (important) que je n'utilise pas la même instance de GridBagConstraints pour chaque composant !

Les gbc.weightx=gbc.weighty=0.5; permettent de répartir les composants dans le panel principal et le BorderLayout permet au panel de s'étendre dans l'espace total du panel TodayArticles, donné par le layout de son parent (à priori le contentpane de la fenêtre, à moins qu'il soit lui même le contentpane de la fenêtre)

**JeremGamer** · 29/09/2013, 13h27

Envoyé par joel.drigo

pourquoi ne pas utiliser un GridLayout dans ce cas ?

Aucun de mes 4 composants n'aura la même taille (pour l'instant c'est le cas mais je modifierais plus tard), j'ai donc opté pour le GridBagLayout.
Mais si il est possible d'avoir des "cases" de tailles différentes avec le GridLayout (je ne sais pas si c'est possible) c'est vrai que ce serait plus pratique!

Je vais tester avec ton code et je te dirais si ça me conviens. Merci beaucoup!

**awane** · 29/09/2013, 15h01

Bonjour,

Voici le code que ton constructeur doit contenir :

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
 
	public TodayArticles() {
 
		content.setLayout(new GridBagLayout());
 
		JPanel headline = new JPanel();	
		headline.setBackground(Color.orange);
 
		JPanel vertical = new JPanel();	
		vertical.setBackground(Color.cyan);
 
		JPanel botleft = new JPanel();
		botleft.setBackground(Color.green);
 
		JPanel botright = new JPanel();
		botright.setBackground(Color.BLUE);
 
		GridBagConstraints gbc = new GridBagConstraints();
		gbc.weightx = gbc.weighty = 1.0;
		gbc.fill = GridBagConstraints.BOTH ;
		gbc.insets = new Insets(5, 5, 5, 5);
 
		gbc.gridx = 0;
		gbc.gridy = 0;
		content.add(headline, gbc);
 
		gbc.gridx = 1;
		gbc.gridy = 0;
		content.add(vertical, gbc);
 
		gbc.gridx = 0;
		gbc.gridy = 1;
		content.add(botleft, gbc);
 
		gbc.gridx = 1;
		gbc.gridy = 1;
		content.add(botright, gbc);
 
 
		setLayout(new BorderLayout());
		add(content,BorderLayout.CENTER);
	}

**JeremGamer** · 29/09/2013, 16h36

Merci awane, ça allège beaucoup mon code mais le problème subsiste...

Il n'y a que headline et vertical qui sont visibles ( j'ai fait un copier-coller de ton code). C'est incompréhensible, il n'y a aucune taille préférée et pourtant ça ne fonctionne pas! Juste une précision, qui je pense n'a pas d'impact sur la chose: le JPanel TodayArticles est utilisé dans un CardLayout, donc il ne prend pas la taille de toute la fenêtre, une bande en haut sert d'interface de navigation.

J'ai aussi testé avec le code de joel: même résultat...

**joel.drigo** · 29/09/2013, 17h07

Ca allège peut être le code, mais à la moindre modif pour avoir un cas spécifique il faut bien faire gaffe : perso, je préfère toujours ne pas recycler mes constraints (enfin je prefèrais parce que je fais plus de swing depuis des années). Pareil pour les poids, je préfère des poids répartis sur un total = 1.0 ou à la rigueur sur 100, c'est plus facile de s'y retrouver. Pour alléger, il suffit d'avoir une méthode qui créé un constraint, en gérant les paramètres en commun et en passant en arguments les paramètres spécifiques.

Ton code d'affichage c'est quelque chose comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
public static void main(String[] args) {
 
		JFrame frame=new JFrame();
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.getContentPane().setLayout(new CardLayout());
		frame.getContentPane().add(new TodayArticles(), ""); // "" est un id de "card" unique dans le layout, mais pour l'exemple pas besoin d'avoir d'id discréminant
		frame.setVisible(true);
 
	}
}

ou autre chose ? (avec le code ci-dessus ça fonctionne)

**JeremGamer** · 29/09/2013, 17h44

Je ne comprend pas... En quoi modifier mon main pourra résoudre mon problème? Surtout que ce panel est déjà par défaut dans mon CardLayout donc il s'affiche tout de suite. En plus j'ai un autre panel qui doit être affiché avant: l'interface de navigation qui elle est visible tout le temps.

**joel.drigo** · 29/09/2013, 17h50

Envoyé par JeremGamer

Je ne comprend pas... En quoi modifier mon main pourra résoudre mon problème? Surtout que ce panel est déjà par défaut dans mon CardLayout donc il s'affiche tout de suite. En plus j'ai un autre panel qui doit être affiché avant: l'interface de navigation qui elle est visible tout le temps.

Non, mais bien sûr que ça n'a rien à voir avec le main. C'était pour savoir comment ton panel était layouté dans la fenêtre qui l'affiche : si tu ne vois que ta première ligne de composants, c'est qu'il y a un problème de layout dans ton affichage de composants. J'ai mis un code qui marchait avec les 2 propositions de sources (la mienne et celle de @awane) : si ça marche dans ce cas simple et pas le tien, c'est qu'il y a une problème quelque part dans ton code, et impossible de pouvoir t'aider si on ne l'a pas...

**JeremGamer** · 29/09/2013, 18h11

En réalité il n'y a que les propriétés de ma fenêtre et l'ajout d'un panel conteneur de tous les autres ( qui est une autre class ) dans la class où se trouve le main.
Mes panels dans ce conteneur sont layoutés avec un CardLayout qui est lui layouté dans un FlowLayout (donc par défaut) Et ça fonctionne bien pour un autre panel qui fait partie du CardLayout mais celui-ci est en FlowLayout car je n'ai que deux composants dedans pour l'instant et les deux sont positionnés comme je le souhaite.

Si vous y tenez, voici le code du conteneur général: (Je précise que le panel "ArticleList" est vide, il y a juste un fond bleu)

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
public class Mpanel extends JPanel {
 
	public JButton bouton1 = new JButton("Today's articles");
	public JButton bouton2 = new JButton("Articles List");
	public JButton bouton3 = new JButton("Software Infos");
	public JPanel inter = new JPanel();
	public InfoPanel info = new InfoPanel();
	public TodayArticles today = new TodayArticles();
	public ArticleList list = new ArticleList();
 
	public CardLayout cl = new CardLayout();
	String [] listContent = {"list", "info", "today"};
	public JPanel content = new JPanel();
 
	public Mpanel() {
 
		this.setBackground(Color.white);
 
		inter.add(Box.createRigidArea(new Dimension(-10 , 300)));
		bouton1.setPreferredSize(new Dimension(225, 35));
		bouton2.setPreferredSize(new Dimension(225, 35));
		bouton3.setPreferredSize(new Dimension(225, 35));
		inter.add(bouton1);
		inter.add(bouton2);
		inter.add(bouton3);
		inter.setBackground(new Color(0, 0, 0, 0));
 
		this.add(inter);
 
		content.setLayout(cl);	
		content.add(today, listContent[0]);
		content.add(list, listContent[1]);
                content.add(info, listContent[2]);
 
 
 
		bouton1.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent event){
				cl.show(content, listContent[0]);
			}
 
		});
 
		bouton2.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent event){
				cl.show(content, listContent[1]);
			}
 
		});
 
		bouton3.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent event){
				cl.show(content, listContent[2]);
			}
 
		});		
 
		this.add(content);
 
	}
 
		public void paintComponent (Graphics g){
 
			 try{
				 Image logo = ImageIO.read (Img.class.getResource("logo.png"));
				 g.drawImage(logo, this.getWidth() / 2 - 687 / 2, 10, 687, 121, this);
			 }catch (IOException e){
				 e.printStackTrace();
			 }
		 }
}

**joel.drigo** · 29/09/2013, 18h36

Quand j'utilise ton code, on mettant directement le MPanel dans le contentpane de ma JFrame, je vois bien ton composant et toutes ses lignes, mais simplement elles sont toutes petites, parce que le panel interne de MPanel (content) n'as pas de layout.

Je suppose que tu veux un box layout vu ta contrainte sur inter ; au début de ta construction ajoute :

Code :

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

		this.setLayout(new BoxLayout(this,BoxLayout.Y_AXIS));

et là je vois bien tout, avec TodayArticles en entier qui s'étend bien à la taille qu'on lui laisse. Bon le panel des boutons est un peu maousse (300 de haut c'est beaucoup), mais ça c'est toi qui voit.

Par ailleurs, je ne comprends pas pourquoi tu créé systématiquement un sous panel (content) à tes extensions de JPanel : pourquoi ne pas directement ajouter les composants directement dans MPanel (et TodayArticles) : bon à la limite, je me dis que c'est pour empêcher un code appelant de modifier le layout interne, mais il faut bien pense à mettre un layout à chaque niveau (si tu ne gères pas "à la main" les coordonnées et taille des composants internes)

**awane** · 30/09/2013, 10h13

La solution de joel.drigo d'utiliser un BoxLayout fonctionne très bien

Tu peux également utiliser un BorderLayout pour ton Mpanel.
Ajouter ton panel inter au North et puis ton panel content au centre comme ceci :

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
 
 
 
public class Mpanel extends JPanel {
 
 
    public static void main(String[] args) {
        final JFrame f = new JFrame("Test ");
        f.add(new Mpanel(), BorderLayout.CENTER);
 
        f.setSize(800, 600);
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
        SwingUtilities.invokeLater(new Runnable() {
            @Override
            public void run() {
                f.setVisible(true);
            }
        });
    }
 
 
    private static final long serialVersionUID = 1L;
    public JButton bouton1 = new JButton("Today's articles");
    public JButton bouton2 = new JButton("Articles List");
    public JButton bouton3 = new JButton("Software Infos");
    public JPanel inter = new JPanel();
    public InfoPanel info = new InfoPanel();
    public TodayArticles today = new TodayArticles();
    public ArticleList list = new ArticleList();
 
 
 
    public CardLayout cl = new CardLayout();
    String [] listContent = {"list", "info", "today"};
    public JPanel content = new JPanel();
 
    public Mpanel() {
        this.setLayout(new BorderLayout());
        this.setBackground(Color.white);
 
        bouton1.setPreferredSize(new Dimension(225, 35));
        bouton2.setPreferredSize(new Dimension(225, 35));
        bouton3.setPreferredSize(new Dimension(225, 35));
 
        inter.add(bouton1);
        inter.add(bouton2);
        inter.add(bouton3);
        inter.setBackground(new Color(0, 0, 0, 0));
 
 
        content.setLayout(cl);
        content.add(today, listContent[0]);
        content.add(list, listContent[1]);
        content.add(info, listContent[2]);
 
 
        bouton1.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent event){
                cl.show(content, listContent[0]);
            }
 
        });
 
        bouton2.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent event){
                cl.show(content, listContent[1]);
            }
 
        });
 
        bouton3.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent event){
                cl.show(content, listContent[2]);
            }
 
        });
 
        this.add(inter,BorderLayout.NORTH);
        this.add(content, BorderLayout.CENTER);
    }
}

**JeremGamer** · 30/09/2013, 15h43

Merci beaucoup à vous deux ça fonctionne parfaitement!

J'ai modifié la taille de mes panels pour obtenir ce que je voulais sans souci! C'est beaucoup plus simple sur ce point grâce au GridBagLayout!

Problème avec GridBagLayout

Agents de placement/Fenêtres Java

Discussions similaires

Partager

Partager