Java ボールが動くアニメーション

Question

ボールがウインドウ内を跳ねるプログラムを作っているのですが、ある所でフリーズしてしまいました。

class BallBound implements ActionListener {
BallBound() {
//ウインドウ作成・アクションリスナ登録・メニューバー作成＆取り付けなど
}

//アニメーション
public void byoga() {
boolean xvect = true, yvect = true;
loop = true;//グローバル変数

while (loop) {
repaint();
try { Thread.sleep(10); }
catch (InterruptedException e1) { }

if (x <= 0 || x+vx >= frame.getWidth()) {
xvect = !xvect;
}//はみ出たら逆ベクトルへ
if (y <= 0 || y+vy >= frame.getHeight()) {
yvect = !yvect;
}

if (xvect) x += xch;
else x -= xch;
if (yvect) y += ych;
else y -= ych;

}
}

public void actionPerformed(ActionEvent e) {
if(e.getSource() == start) {
if(loop == false) {
byoga();
}
}
if(e.getSource() == stop) {
loop = false;
}
}

protected void paintComponent(Graphics g) {
g.fillOval(x, y, vx, vy);
}
}

という風にactionPerformed()から、JMenuItem startが押された場合にbyoga()を呼び出しているのですが、startのメニューを押した（選んだ）瞬間にフリーズしてしまってタスクマネージャから強制終了せざるを得ない状態になってしまいます。
しかしコンストラクタの末尾からbyoga()を呼び出してみたりmain()から呼び出したりした場合はフリーズせずに起動できます。
これは一体何故なんでしょうか・・・？
どなたか教えてほしいです・・・

_ranco_ · Accepted Answer

投稿されたプログラムでは、一つのメインスレッド上でビジネスロジックとGUI操作の両方をやろうとしているため、byoga()メソッドがリターンするまでrepaint()は実行されません。

そこで、ビジネスロジックを別スレッドにするとともに、本当はGUI操作をSwingUtilities.invokeLater()を経由する等によりGUIのスレッドに礼儀正しく登録委託する必要があります（下のサンプルプログラムは超単純で問題もないのでそれを省略）。ただし、そういう面倒なことをしたのは初期の話で、最近では、単純なアニメのようなものはjavax.swing.Timer, より複雑な(I/Oがからむような)ビジネスロジックはjavax.swing.SwingWorkerを使って実装するのが定石です。
（参考URLはhttp://を省略。）
下のプログラムは、古典的に、明示的に別スレッドを作る解法です：
----------------------------------------------------
/* save and compile as BallBoundX.java */
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;

public class BallBoundX extends JPanel {
int x, y, vx, vy, xch, ych;
boolean loop;
Thread t;

public BallBoundX(){
x = y= 0;
vx = vy = 20;
xch = ych = 5;
setBackground(Color.white);
setPreferredSize(new Dimension(500, 500));
}

public void start(){
if (t == null){
t = new Thread(new Runnable(){
public void run(){
byoga();
}
});
}
t.start();
}

public void stop(){
loop = false;
}

public void byoga() {
boolean xvect = true, yvect = true;
loop = true; //グローバル変数

while (loop) {
if (x < 0 || x + vx > getWidth()) {
xvect = !xvect;
} //はみ出たら逆ベクトルへ
if (y < 0 || y + vy > getHeight()) {
yvect = !yvect;
}

if (xvect){
x += xch;
}
else{
x -= xch;
}
if (yvect){
y += ych;
}
else{
y -= ych;
}

repaint();
try {
Thread.sleep(10);
}
catch (InterruptedException e1) {
e1.printStackTrace();
}
}
t = null;
}

protected void paintComponent(Graphics g) {
super.paintComponent(g);

g.setColor(Color.green);
g.fillOval(x, y, vx, vy);
}

public static void main(String[] args){
JFrame.setDefaultLookAndFeelDecorated(true);
JFrame frame = new JFrame();
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
Container con = frame.getContentPane();

JMenuBar mb = new JMenuBar();
frame.setJMenuBar(mb);
JMenu menu = new JMenu("実行制御");
final JMenuItem start = new JMenuItem("start");
menu.add(start);
final JMenuItem stop = new JMenuItem("stop");
menu.add(stop);
mb.add(menu);

final BallBoundX bbx = new BallBoundX();
con.add(bbx, BorderLayout.CENTER);
frame.setBounds(100, 100, 500, 500);
frame.setVisible(true);

ActionListener ner = new ActionListener(){
public void actionPerformed(ActionEvent e){
if(e.getSource() == start) {
bbx.start();
}
else if(e.getSource() == stop) {
bbx.stop();
}
}
};
start.addActionListener(ner);
stop.addActionListener(ner);
}
}
-------------------------------------------------

_ranco_ · Answer

すいません。急いで書いたプログラムのデバッグです：
(1)t.start();はif節の中に入れないといけません。
(2)xvect,yvectはクラスグローバルとし、コンストラクタの中でtrueに初期化すべきです（そうしないと、stop -> 再startのとき、方向が前と違ってしまうことがある）。

syana1159 · Answer

while(loop){
}
の部分で無限ループになっているからですよ。

Java ボールが動くアニメーション

投稿されたプログラムでは、一つのメインスレッド上でビジネスロジックとGUI操作の両方をやろうとしているため、byoga()メソッドがリターンするまでrepaint()は実行されません。

すいません。

while(loop){

似たような質問が見つかりました

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング