3个JButton无法触发drawImage绘制，红绿灯项目故障排查求助

Hey Mary, let's figure out why your buttons aren't updating the traffic light images even though you've confirmed the listeners work and drawing works without conditions. Here are the most likely culprits and fixes:

1. You're not triggering a repaint after updating the light state #

The paint/paintComponent method only runs automatically when the component first loads, resizes, or the OS tells it to refresh. When you click a button, you probably update a variable that controls which image to draw—but if you don't call repaint() on your drawing component, the UI never knows to redraw with the new state.

Fix: Add a call to repaint() right after you update your light state variable in the ActionListener. For example:

// Inside your button's ActionListener
redBtn.addActionListener(e -> {
    JOptionPane.showMessageDialog(null, "Red clicked");
    currentLight = "RED"; // Update your state variable
    yourDrawingPanel.repaint(); // Critical: Tell the component to redraw
});

2. Your state variable has scope/visibility issues #

If the variable you're updating in the ActionListener isn't the same one your paint method checks, the condition will never trigger the new image. Common mistakes here include:

• Declaring the state variable as a local variable inside a method (instead of a class-level member variable)
• Accidentally creating a new instance of your drawing component in the listener, instead of referencing the one already on your UI

Fix: Make sure your light state variable is a member variable of your custom panel/window class, and that your ActionListener modifies that exact variable. For example:

public class TrafficLightFrame extends JFrame {
    private String currentLight = "RED"; // Class-level variable, accessible to both listeners and paint
    private JPanel lightPanel;

    // Constructor and setup code...

    @Override
    public void paint(Graphics g) {
        super.paint(g);
        if ("RED".equals(currentLight)) {
            g.drawImage(redImg, 10, 10, this);
        }
        // ... other light conditions
    }
}

3. Your condition statement has a logical error #

Since removing the condition makes images display, your conditional checks are likely mismatched with how you're setting the state. Examples include:

• Case sensitivity (e.g., checking currentLight == "red" but setting it to "RED")
• Using incorrect comparison operators (e.g., = instead of == for primitives, or not using .equals() for Strings)
• Mismatched variable types (e.g., using an Integer for state but checking against String values)

Fix: Double-check that the value you set in the listener exactly matches what your condition checks. For Strings, always use .equals() instead of == to avoid reference comparison issues:

// Correct String comparison
if ("GREEN".equals(currentLight)) {
    g.drawImage(greenImg, 10, 10, this);
}

Quick Bonus Check #

Make sure you're overriding paintComponent (not just paint) for Swing components, and always call super.paintComponent(g) first to clear old graphics:

@Override
protected void paintComponent(Graphics g) {
    super.paintComponent(g); // Clears previous drawings
    // Your drawImage logic here
}

Try these steps one by one—chances are the missing repaint() call is the main issue!

内容的提问来源于stack exchange，提问作者Mary