Question

我有一个扩展Cell的课程JComponent。目标是显示一个单元格网格，每个单元格都能够处理自己的点击事件等。它基本上是一个平面按钮。

当我向JPanel添加几个单元格时，只显示其中一个单元格。如果，使用相同的代码，我用按钮替换我的单元格，一切都按预期工作。

我错过了什么？

主要方法

public static void main(String[] args){
    JFrame f = new JFrame();
    f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    f.setSize(new Dimension(300,300));
    JPanel jp = new JPanel();
    jp.setLayout(new GridLayout(1, 3));
    if(true){//Use buttons instead of cells
        jp.add(new JButton("Button 1"));
        jp.add(new JButton("Button 2"));
        jp.add(new JButton("Button 3"));
    }
    else{ //Use cells instead of buttons
        Cell a = new Cell(10,0,0);
        Cell b = new Cell(10,0,1);
        Cell c = new Cell(10,0,2);
        jp.add(a,0);
        jp.add(b,1);
        jp.add(c,2);
    }

    f.add(jp);
    f.setVisible(true);
    }

细胞分类

public class Cell extends JComponent{
    private static int numCells=0;
    private Dimension size;
    private int dt;
    private int dl;
    private Color color;
    public Cell(int size, int dt, int dl){
        numCells++;
        Random rand = new Random();
        this.size = new Dimension(size,size);
        this.dt = dt;
        this.dl = dl;
        this.color = new Color(//Random color, but only in one :r, g, or b
            (numCells%3==0)?rand.nextInt(255):0,
            (numCells%3==1)?rand.nextInt(255):0,
            (numCells%3==2)?rand.nextInt(255):0
    );
        this.setPreferredSize(this.size);
        this.setMaximumSize(this.size);
        this.setMinimumSize(this.size);
        this.setBackground(color);
        this.setVisible(true);
        this.setOpaque(true);
    }
    public void amClicked(){
        JOptionPane.showMessageDialog(this.getParent(), 
                this.toString());
    }

    public String toString(){
        return ""+dt+","+dl;
    }
    public void paintComponent(Graphics g){
                Graphics ng = g.create();
    try{
        super.paintComponent(ng);
        ng.setColor(color); 
        System.out.println(String.format("%d,%d,%d,%d(%d,%d,%d)",
                this.getX(), this.getY(), this.getWidth(), this.getHeight(),
                this.color.getRed(),this.color.getGreen(),this.color.getBlue()));
        ng.fillRect(this.getX(), this.getY(), this.getWidth(), this.getHeight());
    }
    finally{
        ng.dispose();
    }
    }


}

Answer 1

此代码中的一个明显错误是您通过setColor（）将更改应用于Graphics对象，但您不会将它们回滚。

Javadocs：

中明确说明了这一点

如果在子类中覆盖它，则不应对传入的Graphics进行永久性更改。

一般解决方案是通过Graphics.create()从您获取的参数中生成一个新的Graphics对象，将代码包装在try-finally块中，并通过{{1}处理新的Graphics对象在finally子句中。

Answer 2

您正在添加3个组件，但只有一个组件被涂成黑色。在单元格中添加红线边框以查看：

  public Cell(int size, int dt, int dl) {
     numCells++;
     //.... code deleted

     // !!this.color = new Color(Color.BLACK); // *** won't compile!
     color = Color.black;

     //.... code deleted

     this.setOpaque(true);
     setBorder(BorderFactory.createLineBorder(Color.red, 2));  // **** add this
  }

编辑：这条线看起来很粗略：

     g.fillRect(this.getX(), this.getY(), this.getWidth(), this.getHeight());

为什么你在这里使用getX和getY？这些方法返回相对于容器而不是单元格的位置信息，但是你使用它来绘制相对于单元格而不是容器的位置，因此黑色矩形将从可见单元格中拉出，这可能不是不是你想要的。也许你需要使用0代替：

     g.fillRect(0, 0, this.getWidth(), this.getHeight());

JPanel将正确显示按钮，但不会正确显示自定义JComponent

2 个答案: