GridBagLayoutは、Jlabelのカスタムサブクラスを使用するときにラベルをスタックします

Question

私はSwingでGUIを書いています。私はGridBagLayoutを使って複数のJLabelsをグリッドに表示しています（基本的にはチェスボードのようです）。 JLabelの代わりにJLabelの代わりにJLabelから得られた自己ラベルクラスを使用すると、GridBagLayoutはすべてのラベルをJPanelの左上隅に積み重ねます。

私のサブクラスTileLabelが間違っているか、レイアウトと制約を正しく使用していません。私はそのような最小のサブクラスで何が問題になるかを見ることができないので、最後のものと思います。

これは（Lはラベルを表します）、それはJLabelを使用してどのように見えるかです：

(MenuBar) 
L L L L L L L L L 
L L L L L L L L L 
L L L L L L L L L 

これは、（Sが積み重ねられたすべてのラベルを表している）、それはTileLabelを使用してどのように見えるかです：

(MenuBar) 
S 

これは、 JLabelの簡単なサブクラス：

import javax.swing.JLabel; 

public class TileLabel extends JLabel { 
    private static final long serialVersionUID = 6718776819945522562L; 
    private int x; 
    private int y; 

    public TileLabel(int x, int y) { 
     super(); 
     this.x = x; 
     this.y = y; 
    } 

    public int getX() { 
     return x; 
    } 

    public int getY() { 
     return y; 
    } 
} 

これはGUIクラスですs。レイアウト問題につながるカスタムラベルを使用した3行をマークしました。

import java.awt.Color; 
import java.awt.GridBagConstraints; 
import java.awt.GridBagLayout; 

import javax.swing.JFrame; 
import javax.swing.JPanel; 

public class MainGUI extends JPanel { 
    private static final long serialVersionUID = -8750891542665009043L; 
    private JFrame frame; 
    private MainMenuBar menuBar; 
    private TileLabel[][] labelGrid; // <-- LINE 1 
    private GridBagConstraints constraints; 
    private int gridWidth; 
    private int gridHeight; 

    // Basic constructor. 
    public MainGUI(int frameWidth, int frameHeight) { 
     super(new GridBagLayout()); 
     constraints = new GridBagConstraints(); 
     buildFrame(frameWidth, frameHeight); 
     buildLabelGrid(frameWidth, frameHeight); 
    } 

    // Builds the frame. 
    private void buildFrame(int frameWidth, int frameHeight) { 
     menuBar = new MainMenuBar(); 
     frame = new JFrame("Carcasonne"); 
     frame.getContentPane().add(this); 
     frame.setJMenuBar(menuBar); 
     frame.setResizable(false); 
     frame.setVisible(true); 
     frame.setSize(frameWidth, frameHeight); 
     frame.setLocationRelativeTo(null); 
     frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
     setBackground(new Color(165, 200, 245)); 
    } 

    // Creates the grid of labels. 
    private void buildLabelGrid(int frameWidth, int frameHeight) { 
     gridWidth = frameWidth/100; 
     gridHeight = frameHeight/100; 
     labelGrid = new TileLabel[gridWidth][gridHeight]; // <-- LINE 2 
     for (int x = 0; x < gridWidth; x++) { 
      for (int y = 0; y < gridHeight; y++) { 
       labelGrid[x][y] = new TileLabel(x, y); // <-- LINE 3 
       constraints.gridx = x; 
       constraints.gridy = y; 
       add(labelGrid[x][y], constraints); // add label with constraints 
      } 
     } 
    } 

    // sets the icon of a specific label 
    public void paint(Tile tile, int x, int y) { 
     if (x >= 0 && x < gridWidth && y >= 0 && y < gridHeight) { 
      labelGrid[x][y].setIcon(tile.getImage()); 
     } else { 
      throw new IllegalArgumentException("Invalid label grid position (" + x + ", " + y + ")"); 
     } 
    } 

    // Just to test this GUI: 
    public static void main(String[] args) { 
     MainGUI gui = new MainGUI(1280, 768); 
     Tile tile = TileFactory.createTile(TileType.Road); 
     for (int x = 0; x < 12; x++) { 
      for (int y = 0; y < 7; y++) { 
       gui.paint(tile, x, x); 
      } 
     } 
    } 
} 

問題はどこですか？

Answer 1

誤ってJComponent#getX() and JComponent#getY()をオーバーライドしています。このメソッドから返される値は、ではなく、で、レイアウトが内部で設定できる値と一致します（setBounds程度の呼び出しを介して）。これはレイアウトを混乱させる。

_{（確かに、私は本当にこれが理由ですが、それは可能性があり、そしてそれは一般的に問題であるかどうかチェックをしませんでした！）}

Answer 2

あなたに修正するためのかなりの数のものがありますカスタムラベルで

1. あなたのメソッド定義：

   public class TileLabel extends JLabel { 
   
       // @Override !!!! 
       public int getX() { 
        return x; 
       } 
   
       // @Override !!!! 
       public int getY() { 
        return y; 
       } 
   } 
   

   コードが、あなたの問題は3つの事に由来します

   JComponentのgetX()とgetY()をオーバーライドしています。これは座標を返すことに責任があります。これはレイアウトを完全に駄目にする。

   paintメソッドに注意してください。同じ名前のメソッドがスーパークラスに存在しますが、引数が異なるためこのケースで保存されます。

2. ループでタイプミスがあります。gui.paint(tile, x, x)はgui.paint(tile, x, y)である必要があります。

3. メソッドを呼び出す順序が間違っています。最初に、フレームを作成して表示した後、ラベルが付いたパネルを追加して内容を変更し、ラベル内のテキストを変更します。これを逆にする必要があります。

私の提言：

• は、あなたのpaint方法は、あなたのTileLabelクラスのメンバにすることを確認します。もっと理にかなっている。
• ラベルがわからない限り、ラベルの作成中にアイコンを設定します。できない場合は、スペース要件を再計算する必要があります。
• レイアウトを画面のサイズや解像度に応じて決して使用しないでください。それは脆弱なGUIを作る（コメントに書かれているように）。正しいサイズを計算するには、フレームにpack()を使用します。