编辑: 好的,我找到了原因。
我的getX和getY覆盖了Component。所以我的输出就像1像素一样...应该早点想到它。
感谢那些试图帮助我的人!
原始问题:
我有这个继承自JPanel的Board类,它应该显示由黑色或白色标签的正方形组成的板(严重)。我正在使用GridLayout但是当我启动应用程序时(在将我的JPanel放入一些虚拟JFrame之后),JLabel似乎在左上角堆叠在一起,这不是我想要的。然而,鼠标监听器似乎行为正常,因为我得到了良好的坐标和标签在黑白之间的颜色切换。
以下是代码:
public class Board extends JPanel {
private int dimx,dimy;
private Square[][] squares;
public Board(int x, int y){
super();
this.setLayout(new GridLayout(x,y));
dimx = x;
dimy = y;
squares = new Square[dimx][dimy];
for(int i=0; i<dimx; i++){
for(int j=0; j<dimy; j++){
Square sq = new Square(i,j);
squares[i][j] = sq;
this.add(sq);
sq.addMouseListener(new SquareListener(i,j,this));
}
}
}
public Square[][] getSquares() {
return squares;
}
}
public class Square extends JLabel {
private boolean black;
private int x,y;
private char c;
public Square(int x, int y){
super();
setBackground(Color.WHITE);
Border blackline = BorderFactory.createLineBorder(Color.black);
setBorder(blackline);
setOpaque(true);
setHorizontalAlignment(JLabel.CENTER);
this.x = x;
this.y =y;
c = ' ';
}
public void toggle(){
black = !black;
}
public boolean isBlack(){
return black;
}
public int getX(){
return x;
}
public int getY(){
return y;
}
public char getC() {
return c;
}
public void setC(char c) {
this.c = c;
}
protected void paintComponent(Graphics g){
super.paintComponent(g);
if(isBlack()){
setBackground(Color.BLACK);
}
else{
setBackground(Color.WHITE);
}
}
}
public class SquareListener implements MouseListener{
private int x,y;
private Board board;
public SquareListener(int x, int y, Board b){
this.x = x;
this.y = y;
this.board = b;
}
@Override
public void mouseClicked(MouseEvent arg0) {
board.getSquares()[x][y].toggle();
board.repaint();
System.out.println("Clicked on "+x+","+y);
}
答案 0 :(得分:3)
您不应更改paintComponent()
方法中的背景颜色。你甚至不应该有paintComponent()
方法。而你也不应该打电话给repaint()
。您的toggle()
方法应该是改变背景的方法:
public void toggle(){
black = !black;
setBackGround(black ? Color.BLACK : Color.WHITE);
}
最后,getX()
和getY()
方法会覆盖JComponent中的方法。为这些方法选择其他名称。
答案 1 :(得分:0)
好的,我找到了原因。
我的getX和getY覆盖了Component。所以我的输出就像是1像素而不是1平方明智......应该早点想到它。
感谢那些试图帮助我的人!