java - 打印出自定义按钮并使 MouseListener 工作时出现问题

Question

我想制作一个自定义按钮，它由一个大方 block 和一个小方 block 组成，分别具有这些颜色:#2980b9 #3498db。小方 block 会在大方 block 的里面，当光标放在上面或点击它时，它会增加它的大小，同时颜色会变得更清晰 (#4AA3DF).

问题是只打印出小一点的，而且还打印不好；因为它出现在窗口的左上角。此外，根本没有使用 MouseListener 函数。

这是 Button 类:

public class Button extends JComponent implements MouseListener {
private static final long serialVersionUID = 1L;

JFrame frame = new JFrame();

public Button(JFrame frame) {
    enableInputMethods(true);
    addMouseListener(this);
    this.frame = frame;
}

// Mouse activity       //DELETED
MouseEvent mouseEvent;  //DELETED

// Window's width and height.
int width = (int) frame.getWidth();
int height = (int) frame.getHeight();

// Squares's sizes.
int bigSquareXSize = 200;
int bigSquareYSize = 200;
int smallSquareXSize = 180;
int smallSquareYSize = 180;

// smallSquare color.
volatile String color = "#3498db";

//I think that I should do something with the update method,
//but i'm not sure about what (sorry, I know this is stupid).
public void update() {

}

@Override
public void paintComponent(Graphics g) {
    Graphics2D g2 = (Graphics2D) g;

    // Squares's X and Y positions.
    int bigSquareXPosition = width / 2 - bigSquareXSize / 2;
    int bigSquareYPosition = height / 2 - bigSquareYSize / 2;
    int smallSquareXPosition = width / 2 - smallSquareXSize / 2;
    int smallSquareYPosition = height / 2 - smallSquareYSize / 2;

    g.setColor(Color.decode("#2980b9"));
    g2.setColor(Color.decode(color));
    g.fillRect(bigSquareXPosition, bigSquareYPosition, bigSquareXSize, bigSquareYSize);
    g2.fillRect(smallSquareXPosition, smallSquareYPosition, smallSquareXSize, smallSquareYSize);

}

// Returns a true value if the cursor is placed over the smallSquare.
public boolean insideArea(MouseEvent e) {
    boolean value = false;
    int smallSquareXPosition = width / 2 - smallSquareXSize / 2;
    int smallSquareYPosition = height / 2 - smallSquareYSize / 2;
    if (e.getX() > smallSquareXPosition && e.getX() < smallSquareXPosition + smallSquareXSize) {
        if (e.getY() > smallSquareYPosition && e.getY() < smallSquareYPosition + smallSquareYSize) {
            value = true;
        }
    }
    return value;
}

volatile boolean clicked = false;

@Override
public void mouseClicked(MouseEvent e) {
    if (insideArea(e)) {
        clicked = !clicked;
        if (clicked) {
            color = "#4AA3DF";
            smallSquareXSize = 190;
            smallSquareYSize = 190;
        }
    } else {
        color = "#3498db";
        smallSquareXSize = 180;
        smallSquareYSize = 180;
    }
    this.repaint();
}

@Override
public void mouseEntered(MouseEvent e) {
    if (!clicked) {
        if (insideArea(e)) {
            color = "#4AA3DF";
            smallSquareXSize = 190;
            smallSquareYSize = 190;
        }
    }
    this.repaint();
}

@Override
public void mouseExited(MouseEvent e) {
    if (!clicked) {
        if (insideArea(e)) {
            color = "#3498db";
            smallSquareXSize = 180;
            smallSquareYSize = 180;
        }
    }
    this.repaint();
}

@Override
public void mousePressed(MouseEvent e) {
    // TODO Auto-generated method stub
}

@Override
public void mouseReleased(MouseEvent e) {
    // TODO Auto-generated method stub
}

这是 StartingPoint 类:

public class StartingPoint implements Runnable {

Thread thread = new Thread(this);
static JFrame frame = new JFrame("BUTTON!");
static Button button = new Button(frame);

public static void main(String[] args) {

    //Frame creation
    JFrame frame = new JFrame("BUTTON!");
    frame.setSize(600, 400);
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setVisible(true);

    JPanel panel = new JPanel();
    panel.setLayout(new FlowLayout());
    panel.add(button);
    frame.add(panel);

}

@Override
public void run() {
    while (true) {
        button.update();
        try {
            Thread.sleep(17);
        } catch (InterruptedException e) {
            e.printStackTrace();
        }
    }
}
}

控制台是这样说的:

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at Button.insideArea(Button.java:62)
at Button.mouseEntered(Button.java:92)

据我了解，问题与frame有关，但我不知道如何解决。我查找了 java.lang.NullPointerException 异常，但我的理解是当您不调用 setSize() 方法或类似方法时会发生这种情况，但是我做到了。

问题是什么，解决问题的最佳方法是什么？

已编辑:我发现导致控制台说出它所说的是 mouseEvent。现在每次我想引用游标时，我都会在方法定义的括号之间使用 e。控制台没有错误；但是，内容以相同的方式打印，并且 MouseListener 仍然无法正常工作。

编辑:我不再有异常，因为有 MouseEvent mouseEvent，但其他问题仍然存在。

Answer 1

有几个问题。

这些行应该在 paint 方法中:

int smallSquareXPosition = width / 2 - smallSquareXSize / 2;
int smallSquareYPosition = height / 2 - smallSquareYSize / 2;

就目前而言，这些值在对象初始化时分配一次。但是您的绘制方法期望它们是动态的，因为您的事件监听器更改了 smallSquareXSize 和 smallSquareXSize。

因为这些变量只能从 paint 方法访问，所以将所有 sch 变量声明移到 paint 方法中:

public void paintComponent(Graphics g) {
    int bigSquareXPosition = width / 2 - bigSquareXSize / 2;
    int bigSquareYPosition = height / 2 - bigSquareYSize / 2;
    int smallSquareXPosition = width / 2 - smallSquareXSize / 2;
    int smallSquareYPosition = height / 2 - smallSquareYSize / 2;

    Graphics2D g2 = (Graphics2D) g;
    g.setColor(Color.decode("#2980b9"));
    g2.setColor(Color.decode(color));
    g.fillRect(bigSquareXPosition, bigSquareYPosition, bigSquareXSize, bigSquareYSize);
    g2.fillRect(smallSquareXPosition, smallSquareYPosition, smallSquareXSize, smallSquareYSize);    
}

这也遵循了一个良好的通用编码实践，即声明变量，使其具有尽可能小的范围。在这种情况下，您会将范围从所有方法缩小到仅使用它们的方法。尽管它不适用于这种情况，但如果可能，您应该进一步将范围缩小到仅一个代码块(例如在循环或 if block 等内)。

---

另一个问题更微妙。它是并发性之一，即在多线程环境中运行。发送鼠标事件的线程是与主线程不同的线程。此外，每个鼠标事件可能是一个不同的线程(我不确定)。并发线程的含义之一是在一个线程中对实例变量所做的更改可能不会被其他线程“看到”。具体来说，这个字段:

boolean clicked = false;

还有这一行:

clicked = !clicked;

这个“问题”是由于 java 内存模型造成的，它简单地说就是每个线程都可以缓存字段值的副本，这意味着如果一个线程为其分配一个新值，另一个线程可能会也可能不会看到这个改变。

但不要 panic - 有一个简单的解决方法:

volatile boolean clicked = false;

volatile 关键字告诉 java 从不缓存值 - 始终检查创建对象的线程管理的内存中的值。

我不确定您使用的框架是否可能对各种类型的鼠标事件使用不同的线程，但如果是这样，您需要 volatile。否则，如果不需要的话，让它 volatile 也没有坏处。