画像が切り替わらないことについて

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
javaでおみくじを作ってます

guiで画像表示もしました

発生している問題・エラーメッセージ

エラーメッセージ
```なし

### 該当のソースコード

```ここに言語名を入力
java
```import javax.swing.*;
import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;
import java.awt.Color;
import java.awt.GridLayout;
import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;

public class omikuzi extends JFrame{

  public static void main(String[] args){
  
    omikuzi frame = new omikuzi();
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setSize(600,600);
    frame.setTitle("おみくじ");
    frame.setVisible(true);
  }
ImageIcon image = new ImageIcon("zinnzya.jfif");
  omikuzi(){
    JLabel lbl = new JLabel(image);
   JLabel label = new JLabel("抽選ボタンを押してください。");
    label.setPreferredSize(new Dimension(300,300));

    label.setVisible(true);
    label.setBounds(100, 210, 280, 30);
    label.setBackground(Color.red);
    label.setOpaque(true);
    JPanel p = new JPanel();
    p.setLayout(null);
    lbl.setIcon(image);
    setLayout(new GridLayout(2, 1,5,5));
    add(lbl); 


    JButton button1 = new JButton("抽選");
    int r = new java.util.Random().nextInt(6);
    button1.setBounds(100, 0, 80, 30);
    button1.addActionListener( new ActionListener() {
     public void actionPerformed (ActionEvent e){
           label.setText("抽選中・・・");
if (r == 0) {image = new ImageIcon("zinnzyan.jfif");//これが写らない
           lbl.setIcon(image);
try{Thread.sleep(1000);}catch(InterruptedException a){}
 image = new ImageIcon("kyou.jfif");//これは写る
           lbl.setIcon(image); 
           label.setText("今日の運勢は凶です。");}

           if (r == 1) { image = new ImageIcon("zinnzyan.jfif");//これが写らない
           lbl.setIcon(image);
try{Thread.sleep(1000);}catch(InterruptedException a){}
image = new ImageIcon("syoukiti.jfif");//これは写る
           lbl.setIcon(image); 
           label.setText("今日の運勢は小吉です。");}

           if (r == 2) {image = new ImageIcon("zinnzyan.jfif");//これが写らない
           lbl.setIcon(image);
try{Thread.sleep(1000);}catch(InterruptedException a){}
 image = new ImageIcon("suekiti.jfif");//これは写る
           lbl.setIcon(image); 
           label.setText("今日の運勢は末吉です。");}

           if (r == 3) { image = new ImageIcon("zinnzyan.jfif");//これが写らない
           lbl.setIcon(image);
try{Thread.sleep(1000);}catch(InterruptedException a){}
image = new ImageIcon("kiti.jfif");//これは写る
           lbl.setIcon(image); 
           label.setText("今日の運勢は吉です。");}

           if (r == 4) {image = new ImageIcon("zinnzyan.jfif");//これが写らない
           lbl.setIcon(image);
try{Thread.sleep(1000);}catch(InterruptedException a){}
 image = new ImageIcon("daikiti.jfif");//これは写る
           lbl.setIcon(image); 
           label.setText("今日の運勢は大吉です。");}

           if (r == 5) { image = new ImageIcon("zinnzyan.jfif");//これが写らない
           lbl.setIcon(image);
try{Thread.sleep(1000);}catch(InterruptedException a){}
image = new ImageIcon("tyuukiti.jfif");//これは写る
           lbl.setIcon(image); 
           label.setText("今日の運勢は中吉です。");}
     }
});

    p.add(button1);
    p.add(label);
    
    getContentPane().add(p, BorderLayout.CENTER);
  }
}







### 試したこと

画像がいかんのかなと思って変えてみたが変わらなかった

### 補足情報（FW/ツールのバージョンなど）

JDK java

Comments

[jimbe]

コードはマークダウンの[コードを入力]を用いてご提示ください。
https://teratail.com/help#about-markdown

Answer 1

Swing は、setIcon をすれば即表示されるものではありません。ボタンを押して実行される actionPerformed メソッドの場合は、そのメソッドが終わってから setIcon の効果が現れます。
複数回 setIcon をしていても、表示されるのは最後に実行したものだけです。いくら sleep しても actionPerformed を実行している限り意味はありません。

ついでに、actionPerformed 等画面を構成するものから呼ばれるメソッド内では、一秒でも止まってはいけません。その間は Swing が反応しなくなってしまうためです。

上記のような現象は、"EDT(イベント・ディスパッチ・スレッド)"や"シングルスレッド"といったワードで検索されると情報があると思いますので、お調べになっては如何でしょうか。

Comments

[soragonn_]

どちらもありがとうございます。
どっちもベストアンサーです！

Answer 2

GUI物ではThread.sleepはタブーです。
一切操作ができなくなりますし、質問のように思った動作になりません。

基本的にはタイマーを使うことになります。
Timer (Java Platform SE 8 )

import java.awt.Color;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.net.MalformedURLException;
import java.net.URL;
import java.util.Random;
import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.SwingUtilities;
import javax.swing.Timer;


public class Omikuzi extends JFrame {
    public static void main(String[] args) throws MalformedURLException {
        Omikuzi frame = new Omikuzi();
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setSize(600, 600);
        frame.setLocationRelativeTo(null);
        frame.setTitle("おみくじ");
        frame.setVisible(true);
    }

    private final ImageIcon jinjaIcon = new ImageIcon(new URL("https://teratail-v2.storage.googleapis.com/uploads/avatars/u13/132786/KnkDDC5A_thumbnail.jpg"));
    private final ImageIcon daikitiIcon = new ImageIcon(new URL("https://teratail-v2.storage.googleapis.com/uploads/avatars/u19/195094/hlb8TKl6_thumbnail.jpg"));
    private final JLabel messageLabel = new JLabel("抽選ボタンを押してください。");
    private final JLabel imageLabel = new JLabel(jinjaIcon);

    private final Random random = new Random();
    private final Timer timer = new Timer(1000, this::onTimer);

    private String kekkaText;
    private ImageIcon kekkaIcon;

    Omikuzi() throws MalformedURLException {
        messageLabel.setBounds(100, 210, 280, 30);
        messageLabel.setBackground(Color.RED);
        messageLabel.setOpaque(true);

        JButton button1 = new JButton("抽選");
        button1.setBounds(100, 0, 80, 30);

//        button1.addActionListener(new ActionListener() {
//            @Override public void actionPerformed(ActionEvent e) {
//                onClick(e);
//            }
//        });
        // ↑が↓こうなって
//        button1.addActionListener(e -> onClick(e));
        // さらに↓こうなった（３つとも意味は同じ）
        button1.addActionListener(this::onClick);

        JPanel panel = new JPanel(null);
        panel.add(button1);
        panel.add(messageLabel);

        setLayout(new GridLayout(2, 1, 5, 5));
        add(imageLabel);
        add(panel);
    }

    private void onTimer(ActionEvent e) {
        timer.stop();
        imageLabel.setIcon(kekkaIcon);
        messageLabel.setText(kekkaText);
    }

    private void onClick(ActionEvent e) {
        messageLabel.setText("抽選中・・・");
        imageLabel.setIcon(jinjaIcon);

        switch (random.nextInt(6)) {
            case 0:
            case 1:
            case 2:
                kekkaIcon = daikitiIcon;
                kekkaText = "今日の運勢は大吉です。";
                timer.start();
                break;
            case 3:
            case 4:
            case 5:
                // どうしても Thread.sleep したいならこんなん
                new Thread(() -> {
                    try { Thread.sleep(1000); } catch (Exception ignored) { }
                    SwingUtilities.invokeLater(() -> {
                        imageLabel.setIcon(daikitiIcon);
                        messageLabel.setText("今日の運勢は凶です。");
                    });
                }).start();
                break;
        }
    }
}