javaアプレット

質問

javaアプレットの勉強をしているのですが、教えてください。
マウスを押すと位置を取得して、線を描くアプレットをつくってみたいと思っています。
一回目に押した位置を始点として、二回目の位置まで線を引きます。このとき、二回目にマウスを押したときにはじめて線を描画するとします。その後、三回目、四回目とマウスを押していくと、前に押した位置を始点として線を引くようなアプレットです。

そこで以下のようにコードを書いて実行してみました。（部分）
public void mousePressed(MouseEvent e)
{
n=n+1;
if(n==1){
x1=e.getX();
y1=e.getY();}
else{
x2=e.getX();
y2=e.getY();
repaint();
x1=x2;
y1=y2;
}
}
public void paint(Graphics g)
{
g.drawLine(x1,y1,x2,y2);
}
ところが、実行してみると、マウスを押す位置に点が移動するだけでした。おそらく、コード中のx1=x2;y1=y2;がrepaint();より前に実行されてしまうのではないかと思います。どう書き直せばいいのでしょうか？また、どうしてrepaint();より前に実行されてしまうのでしょうか？

回答No.1

vaguechat · ベストアンサー

もちろんrepaint()がx1=x2;y1=y2;より先に実行されている。
ただComponentクラスのrepaintメソッドのAPIドキュメントによれば、
repaint()はpaintまたはupdateメソッドを「可能なかぎりすみやかに」呼び出します、とある。
つまり、repaint()は再描画を指示するだけ、つまり描画の予約をするだけで、
実際に再描画がいつ行われるかは（できるだけ早くやろうとはするだろうが）わからない。
したがって、質問者のコードだと再描画を指示した直後にx1とy2をx2,y2と等しくしてしまっているので、
その後実際にpaintが呼ばれて再描画が行われる時点では点しか描画されない。

この場合、問題点は、マウスを押した時にその時点での再描画のために必要なx2,y2の取得と、
「次に」マウスが押された時に必要になるx1,y1の値の設定を一緒に行っていることにある。
したがって、マウスが押されるたびに、その時の線の描画に必要なx1,y1,x2,y2を設定してから再描画を指示すればいい。
x1,y1は前回押されたときのx2,y2の値であるから、2回目以降においてx2,y2を更新する前にx1,y1にこれを代入すればいい。
そして1回目はx1,y1を更新するのでなく2回目以降でx1,y1として使うためのx2,y2を更新すればいい。

public void mousePressed(MouseEvent e) {
n = n + 1;
if (n == 1) {
x2 = e.getX();
y2 = e.getY();
} else {
x1 = x2;
y1 = y2;
x2 = e.getX();
y2 = e.getY();
repaint();
}
}

もう少しまとめられると思うがこのような感じで書けると思う。

ところで、これだとマウスを押すたびに新たな座標に対する線しかメモリーに残らないので、
もし描いた線を全部保存したいなら点の座標を記録して再描画のたびに全部の線を描くか、
オフスクリーンバッファに描画しておきそれを再描画のたびにコピーするかすればいい。

お礼

ありがとうございます。実行したらできました。repaint();はいつ実行されるかわからないんですね。線を全部保存するのもやってみようと思います。