アニメーション

Question

コード
public class MainActivity extends Activity implements Runnable{

	private Handler mHandler = new Handler();				
	CanvasView cv;
	Canvas c = new Canvas();

	@Override
	protected void onCreate(Bundle savedInstanceState){
		super.onCreate(savedInstanceState);
		setContentView(R.layout.activity_main);

		cv = new CanvasView(this);
		setContentView(cv);									
		mHandler.postDelayed(this, 1000);
	}

	public void run(){
		cv.x += 5;
		cv.y += 5;
		cv.draw(c);								
		mHandler.postDelayed(this, 1000);
	}
}

コード
public class CanvasView extends View{

	Paint paint = new Paint();
	int x = 100, y = 100;
	int r = 50;

	public CanvasView(Context con) {
		super(con);
	}

	public void draw(Canvas canvas){
		paint.setColor(Color.BLUE);				
        paint.setStyle(Paint.Style.FILL);		
		paint.setAntiAlias(true);				
        canvas.drawCircle(x, y, r, paint);
	}

}

円が１秒ごとにx軸に５、y軸に５だけ動くようなプログラムを作ったのですが、実行しても最初のx=100,y=100の点から動きません。

１秒後にrun()実行
↓
x,yを5だけ増やす
↓
draw()実行
↓
１秒後にrun()実行

としているつもりなのですがどこがだめなのでしょうか？
よろしくお願いします。

Answer 1

修正点が2つあります。
1つ目は、既に回答されていますが、cv.draw(c)をcv.invalidate()に変更すること。
2つ目は、CanvasViewクラスのdrawメソッドで行なっている処理を、
ViewクラスのonDrawメソッドをオーバーライドしてその中に記述することです。

cv.invalidate()の処理の中で、オーバーライドしたonDrawメソッドが呼び出されます。

Comments

[kisaragizinzin7]

回答ありがとうございます！
１つ目の修正だけでちゃんと動作できたのですが
drawメソッドではなくonDrawメソッドで書いたほうがいいんですかね？

[abs123]

公式のドキュメントによると、
onDrawメソッドをオーバーライドして追加の描画処理を行うか、
drawメソッドをオーバーライドするときは、
スーパークラス(View)のdrawメソッドを呼ぶこと。
みたいなことが書いてあるような気がします。

実際にViewクラスのソースコードを見てみると、
onDrawメソッドは、用意されているだけで、処理は何も記述されていませんが、
drawメソッドでは、なにやらごちゃごちゃ処理が記述されていました。

基本的には、onDrawメソッドをオーバーライドして使用する方が安全だと思います。

Answer 2

cv.draw(c)をcv.invalidate()に変更してみてください。

Comments

[kisaragizinzin7]

回答ありがとうございます！
変更したところちゃんと動きました！
ありがとうございました！
invalidate()は再描画するものみたいですが、
cv.invalidate();このように書くとdrawメソッドを呼び出すのはdrawメソッドに引数Canvasが使われているからでしょうか？

[yona]

プログラマはonDrawを直接呼ぶことはできません。このメソッドはシステムの描画処理から呼ばれます。
プログラマができることはシステムに再描画を依頼することだけです。この依頼するためのメソッドがinvalidateです。

今回はViewがインスタンス化された時に作成されたCanvasが使用されます。

もう一点付け加えるとonDrawメソッドのアクセス修飾子がprotectedからpublicに変更されています。もともとアクセスできないメソッドをあなたがアクセスできるようにしてしまったため、今回のような問題が発生しました。上記の理由からこのようなオーバーライドはしてはいけません。

[kisaragizinzin7]

invalidateはそういう役割だったんですね！

ということは、drawメソッドのところはアクセス修飾子がprotectedのonDrawメソッドに書き換えた方がいいのでしょうか？

[yona]

そうした方がいいですね。
また、オーバーライドしているので@overrideのアノテーションをつけた方がいいでしょう。