drawTextでRGB値を表示するとアプリケーションが強制終了する

Question

カメラで撮影中の画面をタッチして、その箇所のRGBの値を表示するandroidアプリケーションを開発しているのですが、数回タッチするとアプリケーションが終了してしまいます。RGB値は下のように配列で取得しています。
data = new double[3];
data = mrgb.get(pointX, pointY);
配列として確保したメモリがメモリリークを起こしている原因ではないかと考えているのですが、Javaはガベージコレクションが自動でメモリを解放すると調べてわかりました。今回のケースではガベージコレクションが働いていないのでしょうか？明示的にメモリの開放を実装すべきか、または別の表示方法、取得方法を実装すべきか具体的な解決案を教えていただきたいです。

ログを掲載します

01-23 13:18:03.450: I/OpenCVManager/Helper(28337): -----------------------------------------------------------------
01-23 13:18:04.380: E/libEGL(28337): validate_display:255 error 3008 (EGL_BAD_DISPLAY)
01-23 13:18:08.590: E/AndroidRuntime(28337): FATAL EXCEPTION: main
01-23 13:18:08.590: E/AndroidRuntime(28337): Process: com.example.colorre, PID: 28337
01-23 13:18:08.590: E/AndroidRuntime(28337): java.lang.NullPointerException: Attempt to read from null array
01-23 13:18:08.590: E/AndroidRuntime(28337): at com.example.colorre.MainActivity$OverlayView.onDraw(MainActivity.java:303)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.View.draw(View.java:15038)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.View.draw(View.java:14946)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.ViewGroup.drawChild(ViewGroup.java:3304)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.ViewGroup.dispatchDraw(ViewGroup.java:3096)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.View.draw(View.java:14941)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.ViewGroup.drawChild(ViewGroup.java:3304)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.ViewGroup.dispatchDraw(ViewGroup.java:3096)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.View.draw(View.java:14941)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.ViewGroup.drawChild(ViewGroup.java:3304)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.ViewGroup.dispatchDraw(ViewGroup.java:3096)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.View.draw(View.java:15041)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.widget.FrameLayout.draw(FrameLayout.java:594)
01-23 13:18:08.590: E/AndroidRuntime(28337): at com.android.internal.policy.impl.PhoneWindow$DecorView.draw(PhoneWindow.java:2611)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.ViewRootImpl.drawSoftware(ViewRootImpl.java:2714)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.ViewRootImpl.draw(ViewRootImpl.java:2601)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.ViewRootImpl.performDraw(ViewRootImpl.java:2399)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:2023)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:1087)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:6040)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.Choreographer$CallbackRecord.run(Choreographer.java:793)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.Choreographer.doCallbacks(Choreographer.java:606)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.Choreographer.doFrame(Choreographer.java:575)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:779)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.os.Handler.handleCallback(Handler.java:739)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.os.Handler.dispatchMessage(Handler.java:95)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.os.Looper.loop(Looper.java:135)
01-23 13:18:08.590: E/AndroidRuntime(28337): at android.app.ActivityThread.main(ActivityThread.java:5538)
01-23 13:18:08.590: E/AndroidRuntime(28337): at java.lang.reflect.Method.invoke(Native Method)
01-23 13:18:08.590: E/AndroidRuntime(28337): at java.lang.reflect.Method.invoke(Method.java:372)
01-23 13:18:08.590: E/AndroidRuntime(28337): at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:960)
01-23 13:18:08.590: E/AndroidRuntime(28337): at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:755)
01-23 13:18:08.800: I/Process(28337): Sending signal. PID: 28337 SIG: 9

Comments

[swordone]

強制終了時のログを掲載してください。

Answer 1

どうやらこのMat#getは引数で画像の範囲外の座標を指定すると、例外ではなく、nullを返すようです。

そのためpointX,pointYを事前にチェックする処理を追加することで解決に繋がると思います。

Comments

[mad]

範囲外でnullが返ってくるいうことは、アクティビティ作成の段階で躓いている可能性がありそうですね。座標のチェックも視野に入れながら改良してみます。

Answer 2

AndroidRuntime(28337): java.lang.NullPointerException: Attempt to read from null array

強制終了の原因はこれです。
nullの配列から読み込もうとしたと言っていますね。

AndroidRuntime(28337):     at com.example.colorre.MainActivity$OverlayView.onDraw(MainActivity.java:303)

MainActivityの303行目で問題が発生しています。

Comments

[mad]

うまくデータを取得できていないということでしょうか、１回目のタッチで終了するときもあれば、４,５,回目で終了するときもあり、同様のエラーが表示されます。データが取得できる時とできない時の違いがわからないと解決は難しそうです。

[mad]

３０３行目はcanvas.drawText("R:"+data[0]+"G:"+data[1]+"B:"+data[2], 135, 575, p);でRGB値を表示させています。
ちなみにフォントの設定は下のとおりです。
p.setAntiAlias(true);
p.setColor(Color.BLACK);
p.setStyle(Paint.Style.FILL);
p.setTextAlign(Paint.Align.CENTER);
p.setTextSize(40);
p.setStrokeWidth(2);
p.setTypeface(Typeface.SANS_SERIF);

[swordone]

解決するだけなら、303行目の前でdataのnullチェックを行えば済みます。
「なぜnullが返ってくるか」はdataを設定しているmrgb.getがどういうメソッドかで分かります。
mrgbの型は何でしょうか？

[mad]

mrgbはprivate Mat mrgb;で定義しています。
mrgb.getは画像処理ライブラリのopencvの関数を使用しています。

[swordone]

であるならば、そのメソッドのドキュメントがあれば、その説明にnullを返すパターンが書いてないでしょうか？

[mad]

getの宣言を辿ってみました。
public double[] get(int row, int col) {
return nGet(nativeObj, row, col);
}
さらにnativeObjに関して辿ると
public final long nativeObj;

public Mat(long addr)
{
if (addr == 0)
throw new java.lang.UnsupportedOperationException("Native object address is NULL");
nativeObj = addr;
}
このような箇所を見つけたのですが、これがnullを返すパターンなのでしょうか。だとするとここを修正する必要がありますか？

[swordone]

そこはいじらないほうがいいです。
やはりnullチェックで例外回避するくらいでしょうか…

[mad]

下のようにnullチェックをおこなったところ、強制終了を避けることが出来ました。ただ、データが取得できない時があるので、その対策を別に考える必要がありそうです。
if(data!=null){
canvas.drawText("R:"+data[0]+"G:"+data[1]+"B:"+data[2], 135, 575, p);
}