java　画像の表示

コード
import java.applet.Applet;
import java.awt.*;

public class sample extends Applet{
	
	public void paint (Graphics g){
		Image img1 = getImage(getDocumentBase(), "image1.jpg");
		Image img2 = getImage(getDocumentBase(), "image2.jpg");
		
		for(int i = 0; i < 2; i++){
			g.drawImage(img1,  0,  20,  this);
			sleep(20000);
			
			g.drawImage(img2,  0,  20,  this);
			sleep(20000);
		}
	}
	
	public void sleep(int count){
		for(int i = 0; i < count; i++){
			for(int j = 0; j < 100000; j++){
				int h = i * j;
			}
		}
	}

}

画像表示のサンプルコードで上記のようなものがあったのですが、これはimage1とimage2を時間をあけて交互に表示するというものです。

このプログラムで動くことは動くのですが、for文で繰り返しを２回にしているのに３回動いてしまいます。
その原因がわかる方がおられましたら教えていただきたいです。

またpublic void sleep(int count)メソッドで何をしているのかよくわからないのですが、これは20000×100000回もの計算をすることで時間をあけているという解釈であっていますでしょうか？

よろしくお願いします！

swordone

2015/09/01 09:39

このサンプルコードはどこにあったものですか？

satanosaka18419

2015/09/07 08:50

返事が遅くなってしまいすみません。「例題で学ぶJavaアプレット入門」という本のP79に載っていました！

行動規範の内容に同意します

回答3件

ベストアンサー

正直に言ってしまうと、あまり良い例ではないですね。

スリープはビジーウェイト（空回りさせるウェイト）でなくThread.sleepを使うほうが良いです。
それと、Javaではpaintなどの描画メソッドの中ではウェイト（スリープ）を行うべきではありません。
もし他の描画する要素を追加することになると、サンプルコードのような書き方では破綻してしまいます。

下記のように、描画メソッドの外側でバックグラウンドのスレッドを実行して、そこでウェイトと再描画を繰り返すようにしてください。

lang
1import java.applet.Applet;
2import java.awt.*;
3
4public class sample extends Applet extends Applet {
5
6    boolean usesImage2 = false;
7
8    @Override
9    public void init() {
10        super.init();
11        Thread thread = new Thread() {
12            @Override
13            public void run() {
14                repaint();
15                for (int i = 0; i < 4; i++) {
16                    usesImage2 = !usesImage2; // 反転
17                    try {
18                        sleep(1000L);
19                    } catch (InterruptedException e) {
20                        e.printStackTrace();
21                    }
22                    repaint();
23                }
24            }
25        };
26        thread.start();
27    }
28
29    public void paint(Graphics g) {
30        String imgName;
31        if (usesImage2) {
32            imgName = "image2.jpg";
33        }
34        else {
35            imgName = "image1.jpg";
36        }
37        Image img = getImage(getDocumentBase(), imgName);
38        g.drawImage(img, 0, 20, this);
39    }
40
41}

ご質問とは関係ありませんが、JavaでGUIを練習したいのでしたら、AppletよりもSwingの方をお薦めします。アプレットを使いたいのでしたら、JAppletのほうをお薦めします。

参考リンク：
無限ループとrepaintメソッド - スレッドの利用 - Javaアプレット入門

投稿2015/09/01 11:33

argius

総合スコア9388

anonymouskawa

2015/09/01 11:57

>AppletよりもSwingの方をお薦めします。アプレットを使いたいのでしたら、JAppletのほうをお薦めします。 Java8から、JavaFXが正式なGUIライブラリになったという話です。もちろん、Swingを勉強して損はないですが、FXも視野に入れた方が良いと思います。

argius

2015/09/01 12:00

JavaFXはまだサンプルが充実していないので、初心者のかたにはあまりお勧めできないと思っています。もちろん、将来的にはJavaFXに切り替えていかないといけないんですけど。

satanosaka18419

2015/09/07 08:59

お返事遅くなってすみません。 Thread.sleepでやってみたらできました。ありがとうございました。色々ご意見ありがとうございます！ Swing、JApplet、JavaFXを一通り見たいと思います。

行動規範の内容に同意します

swordoneさんが回答されているので、sleepについてでですが、
おそらく仰るとおり計算する間時間をあけるという処理でしょう。

しかし、実際はintではあまり意味がありません。

Java
1public class Test {
2
3    public static void main(String[] args) { 
4    	System.out.println("start:" + (new Date()));
5    	sleep(3000000000l);
6    	System.out.println("end:" + (new Date()));
7    } 
8    public static void sleep(long count){
9        for(long i = 0; i < count; i++){
10            for(int j = 0; j < 100000; j++){
11                long h = i * j;
12            }
13        }
14    }
15}