Layout non spécifié lors d'une insertion de composant

Version imprimable

19/07/2014, 15h11
stoner

Layout non spécifié lors d'une insertion de composant

Bonjour tout le monde!

Je viens de rencontrer un phénomène surprenant.
Je possède un JPanel avec un BorderLayout nommé container. Je souhaite ajouter 3 autres JPanel dans container. Je spécifie pour seulement 2 d'entre eux la position du layout. Soit, SOUTH pour le premier et CENTER pour le deuxième.

Visuellement ça fonctionne bien. Mon troisième élément va automatiquement se placer en haut de container. Cependant, la gestion des événements lié à ce composant ne fonctionne pas. En effet, si je ne lui affecte pas une position dans son layout, la gestion des événements ne fonctionne pas.

Quelqu'un comprend pourquoi ça réagit de cette manière ?

Merci de votre aide!:)
19/07/2014, 15h48
tchize_

Quand tu ne précise pas de contrainte, la contrainte appliquée est Center. Tu ne peux avoir qu'un seul composant en Center.
19/07/2014, 15h56
joel.drigo

Salut,

Tu peux nous mettre le code exact ? Parce que si on ajoute un composant sans contrainte à un container agencé par un BorderLayout, la contrainte Center lui est affectée par défaut : on ne devrait donc voir que 2 composants dans ton container, et donc forcément le troisième, invisible, ne peut recevoir d'événements. D'un autre côté, les contraintes doivent être précisées : normalement, tout est fait pour que ça fonctionne dans le cas contraire, ou qu'une exception soit soulevée, éventuellement. Mais il ne serait pas surprenant qu'un comportement non prévu soit observé lorsqu'on ne suit pas les règles d'usage d'un composant.
19/07/2014, 20h21
tchize_

@Joel: pas vraiment: le border layout ejecte bien l'ancien center au niveau de SES calculs, mais le conteneur a toujours les trois composant. Du coup, t'as deux composants qui se superposent, l'ancien qui a gardé ses anciennes métriques, et le nouveau, qui viens se remettre dedans. Bref, c'est le bordel :)
19/07/2014, 21h37
joel.drigo

Citation:

Envoyé par tchize_

Bref, c'est le bordel :)

C'est bien ce que je voulais dire quand je parlais de comportement imprévu. En tout cas, quand je teste, en Java7, je ne vois que 2 composants (un centré, le dernier que j'ajoute au container, et un au "sud") : j'imagine qu'avec un code différent du mien, on peut en effet avoir un comportement différent, et peut être voir 3 composants, comme l'indique @stoner, ou 2 qui se superposent, cas le plus probable en effet (auquel cas, ça expliquerait pourquoi l'un des deux ne reçoit pas d'évenements...)

Merci pour vos réponses!

Voici le code:

TestGUI.java

Code:

1
2
3
4
5
6
public class TestGUI {
 
    public static void main(String[] args) {   
        Fenetre test = new Fenetre();
    }  
}

Fenetre.java

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
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.ItemEvent;
import java.awt.event.ItemListener;
 
import javax.swing.JButton;
import javax.swing.JComboBox;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
 
public class Fenetre extends JFrame{
  private Panneau pan = new Panneau();
  private JButton bouton = new JButton("Go");
  private JButton bouton2 = new JButton("Stop");
  private JPanel container = new JPanel();
  private JLabel label = new JLabel("Choix de la forme");
  private int compteur = 0;
  private boolean animated = true;
  private boolean backX, backY;
  private int x, y;
  private Thread t;  
  private JComboBox combo = new JComboBox();
 
  public Fenetre(){
    this.setTitle("Animation");
    this.setSize(300, 300);
    this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    this.setLocationRelativeTo(null);
 
    container.setBackground(Color.white);
    container.setLayout(new BorderLayout());
    container.add(pan, BorderLayout.CENTER);
 
    bouton.addActionListener(new BoutonListener());     
    bouton2.addActionListener(new Bouton2Listener());
    bouton2.setEnabled(false);    
    JPanel south = new JPanel();
    south.add(bouton);
    south.add(bouton2);
    container.add(south, BorderLayout.SOUTH);
 
    combo.addItem("ROND");
    combo.addItem("CARRE");
    combo.addItem("TRIANGLE");
    combo.addItem("ETOILE");    
    combo.addActionListener(new FormeListener());
 
    JPanel top = new JPanel();
    top.add(label);
    top.add(combo);    
//___________________________________ICI________________________________________
    container.add(top, BorderLayout.NORTH);   //à tester à la place => container.add(top); 
//_____________________________________________________________________________
    this.setContentPane(container);
    this.setVisible(true);          
  }
 
  private void go(){
    x = pan.getPosX();
    y = pan.getPosY();
    while(this.animated){
      if(x < 1) backX = false;
      if(x > pan.getWidth() - 50) backX = true;    
      if(y < 1) backY = false;
      if(y > pan.getHeight() - 50) backY = true;      
      if(!backX) pan.setPosX(++x);
      else pan.setPosX(--x);
      if(!backY) pan.setPosY(++y);
      else pan.setPosY(--y);
      pan.repaint();
      try {
        Thread.sleep(3);
      } catch (InterruptedException e) {
        e.printStackTrace();
      }
    }    
  }
 
  public class BoutonListener implements ActionListener{
     public void actionPerformed(ActionEvent arg0) {
      animated = true;
      t = new Thread(new PlayAnimation());
      t.start();
      bouton.setEnabled(false);
      bouton2.setEnabled(true); 
    }
  }
 
  class Bouton2Listener implements ActionListener{
    public void actionPerformed(ActionEvent e) {
      animated = false;  
      bouton.setEnabled(true);
      bouton2.setEnabled(false);
    }  
  }  
 
  class PlayAnimation implements Runnable{
    public void run() {
      go();    
    }  
  }
 
  class FormeListener implements ActionListener{
    public void actionPerformed(ActionEvent e) {
      pan.setForme(combo.getSelectedItem().toString());
    }  
  }    
}

Panneau.java

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
import java.awt.Color;
import java.awt.Font;
import java.awt.GradientPaint;
import java.awt.Graphics;
import java.awt.Graphics2D;
 
import javax.swing.JPanel;
 
public class Panneau extends JPanel {
  private int posX = -50;
  private int posY = -50;
  private String forme = "ROND";
 
  public void paintComponent(Graphics g){
    g.setColor(Color.white);
    g.fillRect(0, 0, this.getWidth(), this.getHeight());
    g.setColor(Color.red);
    draw(g);      
  }
 
  private void draw(Graphics g){      
    if(this.forme.equals("ROND")){
      g.fillOval(posX, posY, 50, 50); 
    }
    if(this.forme.equals("CARRE")){
      g.fillRect(posX, posY, 50, 50);
    }
    if(this.forme.equals("TRIANGLE")){
      int s1X = posX + 25;
      int s1Y = posY;
 
      int s2X = posX + 50;
      int s2Y = posY + 50;
 
      int s3X = posX;
      int s3Y = posY + 50;      
 
      int[] ptsX = {s1X, s2X, s3X};
      int[] ptsY = {s1Y, s2Y, s3Y};      
 
      g.fillPolygon(ptsX, ptsY, 3);
    }
    if(this.forme.equals("ETOILE")){      
 
      int s1X = posX + 25;
      int s1Y = posY;
      int s2X = posX + 50;
      int s2Y = posY + 50;        
      g.drawLine(s1X, s1Y, s2X, s2Y);      
      int s3X = posX;
      int s3Y = posY + 17;
      g.drawLine(s2X, s2Y, s3X, s3Y);      
      int s4X = posX + 50;
      int s4Y = posY + 17;
      g.drawLine(s3X, s3Y, s4X, s4Y);            
      int s5X = posX;
      int s5Y = posY + 50;
      g.drawLine(s4X, s4Y, s5X, s5Y);       
      g.drawLine(s5X, s5Y, s1X, s1Y);  
    }
  }
 
  public void setForme(String form){
    this.forme = form;
  }
 
  public int getPosX() {
    return posX;
  }
 
  public void setPosX(int posX) {
    this.posX = posX;
  }
 
  public int getPosY() {
    return posY;
  }
 
  public void setPosY(int posY) {
    this.posY = posY;
  }
}

L'instruction en question se situe à la ligne 55 dans le fichier fenetre.java. :)

21/07/2014, 07h32
joel.drigo

Et quel est le problème avec ce code ? On voit trois composants, au nord, centre et sud, comme voulu, et le combo au nord capte les évenements, avec la contrainte NORTH, et, sans contrainte explicite, 2 composants, top, au centre, et south, au sud, avec toujours captation des évenements (l'instance de Panneau est de taille (0,0,0,0)).