自作でjavaAppletでゲームを作ろうとしています。

質問

自作でjavaAppletでゲームを作ろうとしています。
その前段階としてapplet上に円を表示してその位置を(nowx,nowy)としてマウスでクリックもしくはドラッグした先（aimx,aimy）に向かい少しずつ一定速で向かうプログラムを作っているのですが、一度クリックして(nowx,nowy)が（aimx,aimy）に着く前に再び（aimx,aimy）以外の点をクリックして行き先を変えると円の動くスピードが早くなってしまいます。後ドラッグの方はマウスのドラッグしてる位置に円がきてしまっています。
javaをまだちゃんと理解しきってい焼き付け刃の自分ではイロイロ考えてもよくわからず…
助言いただければお願いいたします。

以下ソースコードになります
ー－ーーーーーーーーーーーーーーーーーーーーーーーーーーーー
import java.applet.*;
import java.awt.*;
import java.awt.event.*;

public class kadai extends Applet{
Color c1=Color.blue;
double aimx=0.0,aimy=0.0,vx,vy;//aimは行き先、ｖｘ、ｖｙはaimとnowを繋いだ線１を進むだけに進むためのｘ、ｙの距離
double nowx=0.0,nowy=0.0;//今いる円の位置
boolean go=true;
public void init(){
addMouseListener(new MouseAdapter() {
public void mousePressed(MouseEvent evt){
if(evt.getX()!=nowx||evt.getY()!=nowy){
stop();c1=Color.red;moveTo(evt.getX(),evt.getY());
aimx=evt.getX();aimy=evt.getY();
start();
}}});
addMouseMotionListener(new MouseMotionAdapter() {
public void mouseDragged(MouseEvent evt) {
if(evt.getX()!=nowx||evt.getY()!=nowy){
stop();c1=Color.black;moveTo(evt.getX(),evt.getY());
aimx=evt.getX();aimy=evt.getY();
start();
}}});
}
public void start(){
go=true;
new Thread(new Runnable(){
public void run(){
while(go){
try{Thread.sleep(10); }catch(Exception ex){ }
moveTime();
}}}).start();
}
public void stop(){ go=false;vx=vy=0.0;repaint(); }
public void stop1(){ go=false;vx=vy=0.0;nowx=aimx;nowy=aimy;repaint(); }
public void paint(Graphics g){
g.setColor(c1);
g.fillOval((int)nowx-20,(int)nowy-20,40,40);
}
public void moveTo(double x,double y){
double c;
vx=x-nowx;vy=y-nowy;
c=Math.sqrt(vx*vx+vy*vy);
vx=vx/c;vy=vy/c;
nowx+=vx;nowy+=vy;repaint();
}
public void moveTime(){
if(aimx-1<=nowx&&nowx<=aimx+1&&aimy-1<=nowy&&nowy<=aimy+1)stop1();
nowx+=vx;nowy+=vy;
repaint();
}}

回答No.1

PecoPlus · ベストアンサー

こんにちは。

　スレッドが二重に起動されていますよ。

while (go) {
　　try {
　　　　Thread.sleep(10);
　　} catch (Exception ex) {
　　}
　　moveTime();
}

　スレッドは、go が true の限り回り続けますが、気をつけなければいけないことは、while ステートメントのところで評価されると言うことです。
　while ステートメントで評価する時間はほんの一瞬であり、このスレッドが過ごす時間の99.99%は、Thread.sleep(10);で、何もしていない状態です。

　マウスイベントでは、stop()メソッドで、go に false を設定していますが、この時もまず間違いなく、スレッドは Thread.sleep(10); で待機状態でしょう。
　すぐさま、マウスイベントでは、start() メソッド内で、新たなスレッドを起動し、go を true に戻してしまっています。

　これで、最初のスレッドは、終了する時機を逸してしまい二つのスレッドが同時に走るという事態になってしまいます。
　そして、二つのスレッドが先を争うように円を動かすものだから、速度が倍になってしまいます。

　マウスドラッグイベントはさらにひどい状態です。
　ものすごい量のスレッドが多重起動されています。
　円が瞬間移動しているように見えるのは、超速で移動しているだけです。

　スレッドが多重起動しないように工夫してみましょう。

お礼

そんなとこに原因があったとは気づきませんでした。
言われてみればその通りですね。
なんとか多重起動しないようにできました。
ちょっと時間なかったんであせっていたのですばやい回答本当にありがとうございました。