画像の切り替え

Question

コード
public class formal extends Applet implements Runnable{

	Thread th;
	int n = 0;
	Image img1;
	Image img2;
	Image img3;
	Image img4;
	
	
	
	public void start(){
		th = new Thread(this);
		th.start();
	}
	
	
	
	public void run(){
		
		for(int i = 0; i < 4; i++){
			
			repaint();
			
			try{
				Thread.sleep(2000); //2秒あける
			}
			catch(InterruptedException e){
			}
		}
		
	}
	
	
	
	public void paint(Graphics g){
		img1 = getImage(getDocumentBase(), "image1.jpg");
		img2 = getImage(getDocumentBase(), "image2.jpg");
		img3 = getImage(getDocumentBase(), "image3.jpg");
		img4 = getImage(getDocumentBase(), "image4.jpg");
		
		if(n == 0){
			n = 1;
			g.drawImage(img1, 100, 100, this);
		}
		else if(n == 1){
			n = 2;
			g.drawImage(img2, 100, 100, this);
		}
		else if(n == 2){
			n = 3;
			g.drawImage(img3, 100, 100, this);
		}
		else if(n == 3){
			g.drawImage(img4, 100, 100, this);
		}
	}
	
}

画像１から画像４を２秒間隔で切り替えていくようなプログラムを作ったのですが、上のプログラムでは画像４がずっと出ている状態になってしまいます。

nを変えることで画像も変わるようにプログラムしているつもりなのですが、どこが間違っているのでしょうか？

Answer 1

こんにちは。
今のコードですと
・paintが呼ばれる度に対象画像を指すnの値が変化する(3を上限として)
・runによって、2秒おきに再描画を要求する(repaint)
と読み取れます。

そして、以下が期待する動作になるのではないかと思います
paint→2秒待機→paint→2秒待機→…
なのですが、必ずしも期待通りにはならない事が考えられます。

以前の回答と重複になって申し訳ないですが、paintは**「描画が必要になったタイミングで都度呼ばれる」**性格の物になると思います。

Appletを表示しているブラウザ(など)が「隠れていた状態から前面に表示された」や「リサイズされた」などによってもpaintは実行される事が考えられます。

なので、runの中で記述しているrepaintとrepaintの間(sleep中)で1回以上のpaintが発生しうる事を想定した記述が必要になります。

目的の通りに2秒おきに画像を変更させたいとすれば
このコードの場合、nの値の変更はpaintの中ではなくrunの中で行う事で実現できるのではと思います。

Comments

[satanosaka18419]

いつも回答ありがとうございます＾＾
nの値の変更をrunメソッドでするとうまく動作できました！

paintは「描画が必要になったタイミングで都度呼ばれる」というのをちゃんとわかっていないとダメですね！
次からはそこに気を付けたいと思います。
ありがとうございました！

Answer 2

nの値が3である時nの値を0に戻す必要はないのですか？

Comments

[satanosaka18419]

回答ありがとうございます＾＾
指摘していただいたところを直すと画像が表示されたのですが、
実行するたびに画像２から切り替えが始まったり、画像３から始まったりします。
毎回画像１から始まって画像４で終わるようにするにはどうしたらよいのでしょうか？