Problème avec GridBagLayout

Version imprimable

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:

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!

29/09/2013, 13h06
joel.drigo

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) !
29/09/2013, 13h24
JeremGamer

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

pourquoi ne pas utiliser un GridLayout dans ce cas ?

Sinon modifie ton constructeur comme suit :

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

29/09/2013, 14h27
JeremGamer

Citation:

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

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

Bonjour,

Voici le code que ton constructeur doit contenir :

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
 
	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);
	}

29/09/2013, 17h36
JeremGamer

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

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:

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)

29/09/2013, 18h44
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.
29/09/2013, 18h50
joel.drigo

Citation:

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

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:

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

29/09/2013, 19h36
joel.drigo

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:

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)

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

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:

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);
    }
}

30/09/2013, 16h43
JeremGamer

Merci beaucoup à vous deux ça fonctionne parfaitement! :ccool:

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