JButton setIcon Ошибка обновления

1. загрузить Icon / ImageIcon как локальную переменную один раз, нет причин повторно загружать изображение из ActionListener

2. в API есть описание, что Image#ScaledInstance довольно асинхронно

3. иначе придется звонить

.

labelicon.getImage().flush();
img_btn.setIcon(labelicon);

РЕДАКТИРОВАТЬ

@akp написал, но .. как бы вы изменили размер изображения значка .. ??

есть два или три других способа, как поместить Icon /ImageIcon и изменить размер с его родителем, JLabel может быть самым простым способом

import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.image.BufferedImage;
import java.io.IOException;
import java.util.Random;
import javax.swing.*;

public class JButtonAndIcon {

    private static JLabel label = new JLabel();
    private static Random random = new Random();
    private static ImageIcon image1; // returns null don't worry about 
    private static ImageIcon image2; // returns null don't worry about 
    private static Timer backTtimer;
    private static int HEIGHT = 300, WEIGHT = 200;

    public static void main(String[] args) throws IOException {
        label.setPreferredSize(new Dimension(HEIGHT, WEIGHT));
        final JButton button = new JButton("Push");
        button.setBorderPainted(false);
        button.setBorder(null);
        button.setFocusable(false);
        button.setMargin(new Insets(0, 0, 0, 0));
        button.setContentAreaFilled(false);
        button.setLayout(new BorderLayout());
        button.add(label);
        button.addActionListener(new ActionListener() {

            @Override
            public void actionPerformed(ActionEvent e) {
                if (button.getIcon() == image1) {
                    label.setIcon(image2);
                } else {
                    label.setIcon(image1);
                }
            }
        });
        JFrame frame = new JFrame("Test");
        frame.add(button);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.pack();
        startBackground();
        frame.setVisible(true);
    }

    private static void startBackground() {
        backTtimer = new javax.swing.Timer(750, updateBackground());
        backTtimer.start();
        backTtimer.setRepeats(true);
    }

    private static Action updateBackground() {
        return new AbstractAction("Background action") {

            private static final long serialVersionUID = 1L;

            @Override
            public void actionPerformed(ActionEvent e) {
                label.setIcon(new ImageIcon(getImage()));
            }
        };
    }

    public static BufferedImage getImage() {
        int w = label.getWidth();
        int h = label.getHeight();
        GradientPaint gp = new GradientPaint(0f, 0f, new Color(
                127 + random.nextInt(128),
                127 + random.nextInt(128),
                127 + random.nextInt(128)),
                w, w,
                new Color(random.nextInt(128), random.nextInt(128), random.nextInt(128)));
        BufferedImage bi = new BufferedImage(w, h, BufferedImage.TYPE_INT_RGB);
        Graphics2D g2d = bi.createGraphics();
        g2d.setPaint(gp);
        g2d.fillRect(0, 0, w, h);
        g2d.setColor(Color.BLACK);
        return bi;
    }
}

Проблема здесь в том, что вы обновляете внутренности файла Icon. Метод setIcon будет думать, что это тот же значок, что и у кнопки. Я бы порекомендовал вам сделать два разных объекта Icon, чтобы использовать их для обновления значка. Это решит проблемы.

Пример (с двумя разными значками):

public static void main(String[] args) throws IOException {

    final ImageIcon redIcon = createImageIcon(10, 10, Color.RED);
    final ImageIcon blueIcon = createImageIcon(10, 10, Color.BLUE);

    final JButton button = new JButton("Push", blueIcon);
    button.addActionListener(new ActionListener() {
        @Override
        public void actionPerformed(ActionEvent e) {
            if (button.getIcon() == redIcon)
                button.setIcon(blueIcon);
            else
                button.setIcon(redIcon);
        }
    });

    JFrame frame = new JFrame("Test");
    frame.add(button);
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.pack();
    frame.setVisible(true);
}

private static ImageIcon createImageIcon(int w, int h, Color color) {
    Image image = new BufferedImage(w, h, BufferedImage.TYPE_INT_ARGB);
    Graphics g = image.getGraphics(); 
    g.setColor(color); 
    g.fillRect(0, 0, w, h); 
    g.dispose();
    return new ImageIcon(image);
}

Задний план:

Глядя на источник AbstractButton.setIcon, вы можете видеть, что он не узнает об обновлении, если ссылка «не обновлена»:

    .....
    if (defaultIcon != oldValue) {
        if (defaultIcon == null || oldValue == null ||
            defaultIcon.getIconWidth() != oldValue.getIconWidth() ||
            defaultIcon.getIconHeight() != oldValue.getIconHeight()) {
            revalidate();
        } 
        repaint();
    }

Примечание для @HarryJoy: вы действительно были правы, хотя и не знали почему... :) Извините! +1 еще раз!

//Позвонить img_btn.revalidate() и img_btn.repaint()

Исправление, setIcon уже должен это делать. Лично я использую хакерский способ img_btn.setText("<HTML><BODY><IMG SRC=\"/path/to/img.jpg\"/></BODY</HTML>");.