android アニメーション

Question

コード
public class MainActivity extends Activity implements Runnable{

	private Handler mHandler = new Handler();				
	CanvasView cv = new CanvasView(this);

	@Override
	protected void onCreate(Bundle savedInstanceState){
		super.onCreate(savedInstanceState);
		setContentView(R.layout.activity_main);
		setContentView(cv);				
		mHandler.postDelayed(this, 1000);
	}

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

コード
public class CanvasView extends View{

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

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

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

}

円を動かすプログラムを作ろうと思い、上記のようなプログラムを作りました。
エラーはないのですが、エミュレーターで実行すると「問題が発生したため、○○を終了します」とでて終わってしまいます。
プログラムがおかしいからなのだとは思うのですが、
何が原因なのでしょうか？

LogCatの内容
error opening trace file: No such file or directory (2)
setTargetHeapUtilization:0.25
setTargetHeapIdealFree:8388608
setTargetHeapConcurrentStart:2097152
Shutting down VM
threadid=1: thread exiting with uncaught exception (group=0x40b4a450)
FATAL EXCEPTION: main
java.lang.RuntimeException: Unable to instantiate activity ComponentInfo{com.example.fireworks/com.example.fireworks.MainActivity}: java.lang.NullPointerException
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:1994)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2095)
at android.app.ActivityThread.access$600(ActivityThread.java:137)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1206)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:213)
at android.app.ActivityThread.main(ActivityThread.java:4786)
at java.lang.reflect.Method.invokeNative(Native Method)
at java.lang.reflect.Method.invoke(Method.java:511)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:789)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:556)
at dalvik.system.NativeStart.main(Native Method)
Caused by: java.lang.NullPointerException
at android.content.ContextWrapper.getResources(ContextWrapper.java:81)
at android.view.View.<init>(View.java:3284)
at com.example.fireworks.CanvasView.<init>(CanvasView.java:17)
at com.example.fireworks.MainActivity.<init>(MainActivity.java:12)
at java.lang.Class.newInstanceImpl(Native Method)
at java.lang.Class.newInstance(Class.java:1319)
at android.app.Instrumentation.newActivity(Instrumentation.java:1053)
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:1985)

Answer 1

cvにインスタンスが入っていないため、ヌルポが発生しているのではないでしょうか。

また、問題が発生したではわからないので、質問にはログを記載すべきです。

Comments

[kisaragizinzin7]

回答ありがとうございます！
CanvasView cv = new CanvasView(this);
すみませんが、このようにしても同じように終了してしまうので他に原因があると思われます。

ログとはなんでしょうか？
「問題が発生したため、○○を終了します」としか表示されないのですが...

[yona]

ログキャットのログです。問題の内容が記載されています。

[kisaragizinzin7]

LogCatには
error opening trace file : No such file or directory (2)
と書いてあります。

[yona]

ログの一部分だけを出されても困ります。
問題が発生したタイミングからのログを全て質問に追記してください。

[kisaragizinzin7]

全て書いておきました！

[yona]

CanvasView.javaの17行目でヌルポインターが出ていますよ。
質問のコードのどこの部分でしょう。

[kisaragizinzin7]

super(context);です。
これがだめってことですか？

[yona]

contextがnullみたいですね。
下記の2点を確認お願い
・MainActivity.java:12はどの行か。
・CanvasView.javaの17行目にブレークポイントを配置しデバッグ実行し、contextがnullになっているかを確認

[kisaragizinzin7]

MainActivity.javaの12行目はCanvasView cv;です。
nullになっていました！

[yona]

質問のコードは古いんじゃないですか？
フィールド変数のCanvasViewの箇所でCanvasView cv = new CanvasView(this);とかしてませんか？

[kisaragizinzin7]

public class MainActivity extends Activity implements Runnable{

private Handler mHandler = new Handler();
CanvasView cv = new CanvasView(this);

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

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

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

[kisaragizinzin7]

すみません。
変えていました。
CanvasView cv = new CanvasView(this);
これが12行目です

[yona]

Contextは一定のライフサイクルを経ないと作成されません。フィールド変数の初期化時は存在しません。
onCreateの中でCanvasViewをインスタンス化してください。

また、コードを修正した場合はコメントではなく、質問自体を編集してください。

[kisaragizinzin7]

すみません。
質問変えておきました。

onCreateメソッドでCanvasViewをインスタンス化しないといけないとすると、runメソッドでも変数cvを使っているのでrunの中でもインスタンス化しないといけないということになりますよね？

[yona]

なりません。なぜそう思うのかが気になりますが。
同じインスタンスを扱うためにフィールド変数にしているんじゃないんですか？

[kisaragizinzin7]

onCreateメソッドの中ででインスタンス化したものを別メソッドのrunメソッドでも使えるのですか？
フィールド変数CanvasView cv;としたらrunメソッドでインスタンス化しないと初期化されていないことになるんじゃないでしたっけ？

[yona]

今回のコードではonCreateの中でrunの実行命令(mHandler.postDelayed(this, 1000))を書いてあります。そのため必ずonCreate→run→run...と言う実行順序になります。
必ずonCreateが先に呼ばれるのでonCreateの中で初期化すれば、runは初期化済みのインスタンスを使用することが出来ます。

[kisaragizinzin7]

そうなんですね！
ありがとうございました！