Javaのアプレットが途中で止まる

Question

#問題
Javaのアプレットビューワーでタイマーを使った簡単な描画を行いたいのですが、途中までは動くのですが、急に止まってしまいます。自分なりに調べたところペイントメソッドが急に呼ばれなくなっているみたいです。この原因がわかる方、おられましたら回答お願い致します。
#ソース

import java.applet.*;
import java.io.*;
import javax.swing.*;
import java.awt.Graphics;
import java.awt.Dimension;
import java.awt.*;
import java.awt.event.*;


/*
<applet code="test.class" width="150" height="150">
</applet>
*/

public class test extends Applet implements ActionListener{
	int X = 75, Y = 30, W = 30, H = 50, K = 10 ;
	Timer timer, timer2;
	int x;
	Button btn = new Button("スタート");
	boolean btnClicked = false;

	public void  actionPerformed(ActionEvent e){
		if (e.getSource() == timer2&&this.btnClicked==true){
			x+= 2;
			if(x == 8){
				x = 0;
			}
		}
		if (e.getSource() == timer&&this.btnClicked==true){
			
			timer2.start();
		}
		if (e.getSource()==this.btn){
			this.btnClicked = true;
			this.timer.start();
		}
		repaint() ;
	}
	@Override
	public void init(){
		timer = new Timer(20, this); 
		timer2 = new Timer(2, this);
		add(this.btn);
		this.btn.addActionListener(this);
	}

	省略
#追記
timer2のインスタンスの生成をinitの中で行うことでアプレットは止まらなくなりました。

Answer 1

###論理が不自然な点

timerが何度も発火するようになっていますので、発火の度にtimer2のインスタンスが繰り返し(無限に)生成され、そのたびにタイマーイベントの発火契機が増えていきます。

100や200程度のTimerインスタンスやタイマーイベントが登録された状態になったからといってjavax.swing.Timerのイベント通知機構が破綻するようなやわな作りにはなっていないと思いますが、それでも不必要にTimerインスタンスを生成してしまっているのは問題だと思います。

###アニメーションが止まった原因
それについては自分の環境(Windows10, 主記憶8GB, JDK 1.8.0_152)では再現しませんでした。
なお、IDE(IntelliJ IDEA)からのデバッグ実行によりAppletViewerの元で実行しています。

推測ですが、...

(1)timer2にTimer@1のインスタンスが設定されていてイベント発火を待機しているとする
(2)timerが発火してtimer2に新たなTimerインスタンスTimer@2が設定されてしまう
(3)古いインスタンスTimer@1のイベントが発火するがif (e.getSource() == timer2 ...)の判定で、timer2のインスタンスが変わってしまっているためにこの条件文が成立しない。ゆえにアニメーションの更新処理がされずにrepaintのみが実行される。

タイミングによって上記のような現象が起こっているのではないでしょうか？

###対処

(A) timerを何度も発火させる必要はない。＝＞ワンショットタイマーで充分。
この対策が本質的な対策だと思います。

しかし論理をわかりやすくするという意味で気になる点があるのでそちらも変更をお勧めします。

(B) 同一のイベントハンドラーで複数のイベントをまとめてハンドリングしようとすること

Event#getSource()によりイベント発火元に応じた処理を切り分けるという手法は「同じような処理を複数のコンポーネントに対して同一のハンドラーでまとめて処理する」というような場合なら便利です。例えばオセロのようなもので、駒をそれぞれ独立したコンポーネントにしておき、それら全てのクリックイベントを同一のハンドラーで処理するような場合です。イベントソースからどの駒であるかはわかるので、その違いを除き共通的なハンドリングを行いたくなることでしょう。

しかしながら本件ではボタンのクリック、timerの発火、timer2の発火のそれぞれはまったく違う処理ですので無理に一つのハンドラーで処理するのはかえって分かりにくくなると思います。たまたまだとは思いますがgetSource()を使ったために作りこまなくてよいバグを作ってしまったとも言えますし。

具体的にはアプレットのクラスからimplements ActionListenerを取り去り、ActionListenerのインスタンスをthisではなくて、それぞれのイベント設定ごとの個別のリスナー（ハンドラー）を無名クラスやラムダ式やメソッド参照を用いて設定することをお勧めします。

以下、ラムダ式の例を挙げてみます。 (ちなみにbtnClickedというフラグも必要ないように思えますので以下の例では省略してます。)

// Javaではクラス名は必ず大文字で始めましょう。
// 小文字で始めても動きはしますが、習慣に反しているため多くのプログラマーの目には
// 大変奇妙に映ります。
public class Test extends Applet {
  // 定数ならstatic finalの方が良い
  // (プログラムの今後の改造によっては定数ではなくなるのかも知れませんが)
  static final int X = 75, Y = 30, W = 30, H = 50, K = 10 ;
  int x;

  @Override
  public void init() {
    Button btn = new Button("スタート");
    btn.addActionListener(ev -> onStartButtonClicked());
    add(btn);
  }

  // 何がおきたかはっきりわかるようなメソッド名にする
  private void onStartButtonClicked() {
    // timerはこのメソッド以外からさわる必要はないのでフィールドにしなくてよい
    Timer timer = new Timer(20, event -> onAnimationStart());
    timer.setRepeats(false); // ワンショットタイマーにする
    timer.start();
  }

  // 現在のコードではtimer2をフィールドにする必要はないが
  // 後でアニメーションを止めるような処理を追加することを想定すると
  // 元のコードにあるとおり、フィールドにしておいた方がよいでしょう
  Timer timer2;

  private void onAnimationStart() {
    timer2 = new Timer(2, event -> onUpdateAnimationFrame());
    timer2.start();
  }

  private void onUpdateAnimationFrame() {
    ...アニメーション用の状態変更処理...
    repaint();
  }

  ...
}

Comments

[KSwordOfHaste]

余談ですが、ご質問にあるコードはインデントに空白文字とタブ文字をまぜて使っているためご自分のエディター上ではそろっているように見えてもマークダウンによりタブ幅が変わるとインデントが乱れて見えてしまいます。こうした場で質問する際のコードは「閲覧者に正確にインデントを示し、コードを正確に把握しやすくする」ことに注意を払ってください。不正確なインデントは閲覧者へ無用な混乱を与えます。
ご質問にあるコードはそれほどの乱れはないですが、ひどい人になると「インデントがない方がまし」なようなハチャメチャなコードを貼り付けておられたりしますのでこうした点にも注意を払うとよいと思います。

[aiueoaiueoaiue]

ありがとうございます。timer2のインスタンスをif文の中で生成していたのは最終的にタイマーのスピードを徐々に遅くしようと考えていたからです。そのためにはtimer2 = new Timer(2, this);の2秒遅延の部分を変数にして、いちいちインスタンスを生成していくしかないかなぁと考えていました。この方法が使えないとなると、どのようにすればタイマーのスピードを徐々に遅くするという事が出来るでしょうか？
ちなみにtimer2のインスタンスの生成をinitの中で行うことでアプレットは止まらなくなり、思い通りの結果になりました。

[KSwordOfHaste]

なるほど。そういう意図でしたか。

ところで、回答する際には質問文やその中のコードから読み取れる範囲でコメントするしかありません。おっしゃるとおりの意図があるなら質問文に明記して「なぜこういうコードを書きたかったのか」を示していただけると最初からそれを想定したコメントができます。

> どのようにすればタイマーのスピードを徐々に遅くするという事が出来るでしょうか？

これは最初の質問にかかれていませんね？質問の内容を途中であれこれいじるのはよくありません。なぜかというと、質問内容が大きく変化すると編集後の質問内容と回答の内容が不一致になり、一般の閲覧者が「この質問に対してなんでこんな回答してるの？」と混乱するからです。元の質問に充分質問意図が含まれていなかった場合は質問意図を捕捉する意味で編集するのはよいことなのですが、内容が大きく変わってしまう場合は新たに質問を挙げて「知りたいことをより端的に尋ねる」方がよい場合もあります。このあたりの匙加減が難しいかもしれませんね。本件の場合、個人的には新たに質問を起こした方がよいと思います。タイトルを「Appletでアニメーションの時間間隔を徐々に変更したい」というズバリの内容にするといいのではないでしょうか？

[aiueoaiueoaiue]

わかりました。ありがとうございました。