swingのGUIプログラムについて

Question

前提・実現したいこと

swingのGUIプログラミングについて．
現在swingを用いてライフゲームを作成しているのですが，
盤面をクリックやドラッグでの生死の変更，クリックによる次世代への遷移，以前の画面への遷移を実装しました．今，0.5秒ごとに自動的に次世代へ更新していくAutoボタン機能を付随しようと考えているのですがうまくいきません．

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

Autoを押すと初めのうちだけ正常に動くがすぐに盤面の動き方がおかしくなりウインドウが固まってしまう

該当のソースコード

package lifegame;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.Timer;

public class AutoListener implements ActionListener{
    private BoardModel model; //Boardmodel=盤面のクラス
    private Timer timer;
    public AutoListener(BoardModel model){
        this.model = model;
    }
    public void actionPerformed(ActionEvent e){
        timer = new Timer(500, this);
        timer.start();
        model.next();  //nextメソッドは盤面を次世代へ進めるもの
        return;
}
}

試したこと

ネットでいろいろ調べてみたがマルチスレッドというものが必要らしいというのはわかっているが何をすれば良いかいまいちわからないです．よろしくお願いします．

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

ここにより詳細な情報を記載してください。

Comments

[jimbe]

> 盤面の動き方がおかしくなり
どうなるはずがどうなるのでしょう.

[yuto1123581321]

ご回答ありがとうございます
期待する動きは要するにnextボタンを同じタイミングで連打した時の動きなのですが，(要するに滑らかに次世代へと盤面表示が切り替わる),
Timerを使っただけだと途中かくついて急にありえない遷移をしたり重たくなってウインドウのボタンが反応しなくなったりしてしまいます

Answer 1

まず, 書かれたコードがどう動いているか想像してみてください.
おそらく AutoListener は AUTOボタンのリスナとして登録していると思いますので, AUTOボタンを押したとしましょう.

AutoListener.actionPerformed が呼ばれ, 0.5 秒おきの Timer(1つ目) が生成(new)・開始(start) されて, model.next が実行されます.

0.5 秒後, timer(1つ目) によって AutoListener.actionPerformed が呼ばれ, 0.5 秒おきの Timer(2つ目) が生成(new)・開始(start) されて, model.next が実行されます.

0.5 秒後, timer(1つ目) によって AutoListener.actionPerformed が呼ばれ, 0.5 秒おきの Timer(3つ目) が生成(new)・開始(start) されて, model.next が実行されると同じ頃に timer(2つ目) によって AutoListener.actionPerformed が呼ばれ, 0.5 秒おきの Timer(4つ目) が生成(new)・開始(start) されて, model.next が実行されます.

0.5 秒後, timer(1つ目) によって AutoListener.actionPerformed が呼ばれ, 0.5 秒おきの Timer(5つ目) が生成(new)・開始(start) されて, model.next が実行されると, timer(2つ目) によって AutoListener.actionPerformed が呼ばれ, 0.5 秒おきの Timer(6つ目) が生成(new)・開始(start) されて, model.next が実行され, さらに timer(3つ目) によって AutoListener.actionPerformed が呼ばれ, 0.5 秒おきの Timer(7つ目) が生成(new)・開始(start) されて, model.next が実行され, もう一つ timer(4つ目) によって AutoListener.actionPerformed が呼ばれ, 0.5 秒おきの Timer(8つ目) が生成(new)・開始(start) されて, model.next が実行されます.
:
:
お分かりでしょうか.
Timer は止めるまで AutoListener.actionPerformed を呼び続け, AutoListener.actionPerformed が呼ばれる度に新たな Timer が生成・開始されてゆきます.
何時かは資源が尽き, 異常終了します.

Timer から呼び出す actionPerformed はボタンとは別にしたほうが良いかと思います.

Comments

[yuto1123581321]

GUIを初めて使うのでよくわからいので伺ってもよろしいでしょうか．
Timerから呼び出すactionPerformedをボタンと別にするとはどういうことか教えていただけるとありがたいです．
Mainクラスのrun内のAndoボタンの処理は次のように書いてます
JButton autobutton = new JButton("Auto");
buttonPanel.add(autobutton); //(buttonPanel = JFrameのパネル)
autobutton.addActionListener(new AutoListener(model));

[yuto1123581321]

理想の挙動はAutoボタンを押すと世代を更新し続けるが他の機能(例えばクリックでセルを動いている途中に変更などの操作が可能である)の邪魔をしないようにしたいです

[jimbe]

> timer = new Timer(500, this);
を
timer = new Timer(500, new ActionListener(){
public void actionPerformed(ActionEvent e){
model.next();
}
});
としてみては如何でしょうか.

[jimbe]

0.5 秒おきに更新されているライフゲームのセルを, 動かしたままクリックで更新するのは現実的では無いように思えます.
AUTO中は更新できないことにして, STOPボタンを新設しSTOPしたら更新可能とし, 更新が終わったらまた AUTOボタンで開始するようなほうが, 任意の形のセルを設定したりがやり易いのではないでしょうか.

[yuto1123581321]

上記のコードに変更したら動かすことができました．
ありがとうございます．
おっしゃる通りそちらの機能の方が理にかなっているので実装させていただきます．ありがとうございました
最後に1つだけ伺いたいのですがなぜ上記のように書き換えると動くかもう少しわかりやすく教えていただけると嬉しいです!!(理解力がなくてすみません)

[jimbe]

Timer の仕様は, 一定時間毎に第二引数に指定した ActionListener の actionPerformed を呼び出すことです.
件の場合, 一定時間毎にしたいのは( Timer の生成では無く) model.next だけです.
ですので, AUTOボタンのリスナである AutoListener とは別に, model.next だけを行う actionPerformed を持つ ActionListener を作って Timer に渡しています.

[yuto1123581321]

理解できました！ありがとうございました!