Javaソースで作ってしまったバグが分からない

Question

AndroidStudioにて2つの画像の一致部分を結果として示すメソッドを作り、そのアプリ一旦稼働していたのですが、
自分でどこか触ってしまったのか比較を実行するとAndroidの方で実行は出来るのですが、この比較ボタンを押した際に繰り返し停止エラーが出てしまいます。ちなみに比較ボタンを押した際にonCameraDevitionClickが実行されます。原因が何かお分かりでしたらお願い致します

public void onCameraDevitionClick(View view) {
   ImageView src_img1 = (ImageView) findViewById(R.id.src_img1);
   ImageView src_img2 = (ImageView) findViewById(R.id.src_img2);
   ImageView result_img = (ImageView) findViewById(R.id.result_img);

       Bitmap bitmap1,bitmap2;
      Bitmap bitmap3;
      if (src_img1.getDrawable() instanceof BitmapDrawable) {
       bitmap1 = ((BitmapDrawable) src_img1.getDrawable()).getBitmap();
      } else {
         Drawable d = src_img1.getDrawable();
          bitmap1 = Bitmap.createBitmap(d.getIntrinsicWidth(), d.getIntrinsicHeight(), Bitmap.Config.ARGB_8888);
         Canvas canvas = new Canvas(bitmap1);
         d.draw(canvas);
      }

      if (src_img2.getDrawable() instanceof BitmapDrawable) {
           bitmap2 = ((BitmapDrawable) src_img2.getDrawable()).getBitmap();
      } else {
         Drawable d = src_img2.getDrawable();
          bitmap2 = Bitmap.createBitmap(d.getIntrinsicWidth(), d.getIntrinsicHeight(), Bitmap.Config.ARGB_8888);
         Canvas canvas = new Canvas(bitmap2);
         d.draw(canvas);
      }

      if (result_img.getDrawable() instanceof BitmapDrawable) {
          bitmap3= ((BitmapDrawable) result_img.getDrawable()).getBitmap();
      } else {
         Drawable d = result_img.getDrawable();
          bitmap3 = Bitmap.createBitmap(d.getIntrinsicWidth(), d.getIntrinsicHeight(), Bitmap.Config.ARGB_8888);
         Canvas canvas = new Canvas(bitmap3);
         d.draw(canvas);
      }


      if (isImageSame) {
          final   int wbmp1=bitmap1.getWidth();
           final  int hbmp1=bitmap1.getHeight();
            for (int i = 0; i < wbmp1; i++) {
                for (int j = 0; j < hbmp1; j++) {
                   final  int pbmp1=bitmap1.getPixel(i, j);
                   final int pbmp2=bitmap2.getPixel(i, j);
                    if ((pbmp1-pbmp2<=500000)&&(pbmp1-pbmp2>=-500000)) {
                  //   Do whatever you want Correct Image.. // both image Are Same
                  bitmap3.setPixel(i, j, pbmp1);
               }     else {
                        // both image Are diffrent
                        //  bitmap3.setPixel(i,j,-16777216);
                        bitmap3.setPixel(i,j,-11111111);
                    }
                }
            }
            result_img.setImageBitmap(bitmap3);
        }
}

Comments

[Zuishin]

まずエラーメッセージを読んでください。

[2r4h8k3m]

アプリの実行はできるのですが画像1と画像2を選択し、こちらのボタンを押した時にonCameraDevitionClickクラスが発動するのですがそちらでAndroid端末で繰り返しエラーが出てしまいます。何かお分かりでしたらお願いいたします。

[m.ts10806]

コードしか提示されないので質問者さんが”何を”「エラー」と言っているのか分からないのです。
https://teratail.com/help/question-tips#questionTips3-4

[2r4h8k3m]

Androidでの実行なので開発環境で特にエラー文はなく実行したAndroid端末でその比較ボタンでエラーが出るということです。バグのことについてお分かりでしたらよろしくお願いいたします。

[Zuishin]

Android Studio であれば実機もしくは仮想マシンでデバッグできるはずです。
まずはどこでどのような問題が起きているかを調べるために、このメソッドの最初の行にブレークポイントを張ってステップ実行してください。

[2r4h8k3m]

こちら最終的にカメラ連携なども組み込んだアプリになるので実機の方が良いのではとアドバイスをいただきました。デバッグ自体が何度かやってみたのですが結局何をしているのかわからないまま終わってしまっていたのでもう一度やってみます。ありがとうございます。

[Zuishin]

> デバッグ自体が何度かやってみたのですが結局何をしているのかわからないまま終わってしまっていたのでもう一度やってみます。

わからないなら何度やっても無駄でしょう。
わかるようになってからやるという選択肢があります。
まずドキュメントを読んでください。
そんなに難しいことは書いていないはずです。

[Zuishin]

https://developer.android.com/studio/debug?hl=ja

ここです。
図入りで簡単でしょう？

[m.ts10806]

質問者さんが”何を”「エラー」と言っているのか分からないのです。
https://teratail.com/help/question-tips#questionTips3-4
「エラーが出ます」→「どんな？」
画面キャプチャでも提示されたほうが良いですね。
口頭説明には限界があります。特に文章だけのやり取り、赤の他人ですから。

[jimbe]

Zuishin さん御提示のリンク先にある
「システムログを表示する」
https://developer.android.com/studio/debug?hl=ja#systemLogView
が出来れば、落ちたときの例外が見えて取っ掛かりになると思います。

[dodox86]

> デバッグ自体が何度かやってみたのですが結局何をしているのかわからないまま終わってしまっていた

この状況ではこの先、別の問題が起きたらまた聞くはめになるでしょう。それではデバッグ、作業依頼になってしまいます。主体的に自分でデバッグしましょう。自分でどこを触ったか分からない、というのもちょっとあんまりです。意外と実は触っていなくて、やっとバグ（問題）が発現するようになった、ということもあります。

[dodox86]

コードを触ったり修正するなら、gitやsubversionなどのvcsの利用は必須だと思います。（質問とは直接関係ないですけど）

[hoshi-takanori]

Android のデバッグに関しては、デバッガを使うのも良いですが、個人的にはまず Logcat のエラーメッセージ (スタックトレース) を読める (または質問文に貼れるようになる) ことが重要だと思います。詳しくは jimbe さんのリンクか、こちらをどうぞ。
https://developer.android.com/studio/debug/am-logcat?hl=ja

ソースに関しては、bitmap1 と 2 を ImageView から取ってくるのはいいとして、bitmap3 は出力用なので自分で一から作るべきでは…。

[2r4h8k3m]

皆さんありがとうございます、こちらのサイト見ながら以前https://techacademy.jp/magazine/2441#sec1何度かやってみたのですが、ブレイクポイントステップ実行と具体的にセットで助言頂きまして、なんとなくイメージがわかりました。デバッガー画面は左下に出てきて、logcatはこちらのURLには出ていないけどそれだとViewのツールウィンドウから出すのかなhttps://aws-mobile-development.hatenablog.com/entry/2016/11/06/065030最近はソース更新する時はコピーして少しずつ進めていたのですがサボってしまい、gitツール出来そうな時に設定しようかと思います。

Answer 1

デバッガ・Logcat・VCS等環境は今後の為にも是非設定して頂いて、画像差分のコードの問題に付きましては hoshi-takanori さんの bitmap3 に関するご指摘が当たりのように思います。

カメラが必要なアプリであるということで実機でテストされているようですが、 onCameraDevitionClick 自体はカメラは関係無いようですので、このメソッドだけ抜いてエミュレータで動かすことも出来るでしょう。
(下のコードは抜き出した上に弄っちゃってますが…。)

3つの ImageView がどのような状態で onCameraDevitionClick が呼ばれることになるのかはハッキリしませんが、リソースから src_img1/2 に設定し、result_img は空の状態としました。

MainActivity.java

package com.teratail.q357676;

import androidx.appcompat.app.AppCompatActivity;

import android.graphics.*;
import android.graphics.drawable.*;
import android.os.Bundle;
import android.view.View;
import android.widget.ImageView;

public class MainActivity extends AppCompatActivity {
  private ImageView src_img1;
  private ImageView src_img2;
  private ImageView result_img;

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

    src_img1 = (ImageView)findViewById(R.id.src_img1);
    src_img1.setImageResource(R.drawable.k_202105_0); //テスト画像1

    src_img2 = (ImageView)findViewById(R.id.src_img2);
    src_img2.setImageResource(R.drawable.k_202105_1); //テスト画像2 ※src_img1と同じ大きさであること

    result_img = (ImageView)findViewById(R.id.result_img);
  }

  public void onCameraDevitionClick(View view) {
    Bitmap src_bitmap1 = getBitmap(src_img1);
    Bitmap src_bitmap2 = getBitmap(src_img2);

    Bitmap result_bitmap = createDiffBitmap(src_bitmap1, src_bitmap2);
    result_img.setImageBitmap(result_bitmap);
  }

  private Bitmap getBitmap(ImageView imageView) {
    if(imageView.getDrawable() instanceof BitmapDrawable) {
      return ((BitmapDrawable)imageView.getDrawable()).getBitmap();
    }
    Drawable d = imageView.getDrawable();
    Bitmap bitmap = Bitmap.createBitmap(d.getIntrinsicWidth(), d.getIntrinsicHeight(), Bitmap.Config.ARGB_8888);
    d.draw(new Canvas(bitmap));
    return bitmap;
  }

  private Bitmap createDiffBitmap(Bitmap src_bitmap1, Bitmap src_bitmap2) {
    final int width = src_bitmap1.getWidth();
    final int height = src_bitmap1.getHeight();
    Bitmap dist_bitmap = Bitmap.createBitmap(width, height, Bitmap.Config.ARGB_8888);

    for (int x = 0; x < width; x++) {
      for (int y = 0; y < height; y++) {
        int src_color1 = src_bitmap1.getPixel(x, y);
        int src_color2 = src_bitmap2.getPixel(x, y);
        int dist_color = isDifference(src_color1, src_color2) ? -11111111 : src_color1;
        dist_bitmap.setPixel(x, y, dist_color);
      }
    }

    return dist_bitmap;
  }

  private boolean isDifference(int color1, int color2) {
    return Math.abs(color1 - color2) > 500000;
  }
}

レイアウト: activity_main.xml

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".MainActivity">

  <ImageView
      android:id="@+id/src_img1"
      android:layout_width="0dp"
      android:layout_height="0dp"
      app:layout_constraintBottom_toTopOf="@id/src_img2"
      app:layout_constraintLeft_toLeftOf="parent"
      app:layout_constraintRight_toRightOf="parent"
      app:layout_constraintTop_toTopOf="parent" />
  <ImageView
      android:id="@+id/src_img2"
      android:layout_width="0dp"
      android:layout_height="0dp"
      app:layout_constraintBottom_toTopOf="@id/compare_button"
      app:layout_constraintLeft_toLeftOf="parent"
      app:layout_constraintRight_toRightOf="parent"
      app:layout_constraintTop_toBottomOf="@id/src_img1" />
  <Button
      android:id="@+id/compare_button"
      android:layout_width="wrap_content"
      android:layout_height="wrap_content"
      android:text="比較"
      android:onClick="onCameraDevitionClick"
      app:layout_constraintBottom_toTopOf="@id/result_img"
      app:layout_constraintLeft_toLeftOf="parent"
      app:layout_constraintRight_toRightOf="parent"
      app:layout_constraintTop_toBottomOf="@id/src_img2" />
  <ImageView
      android:id="@+id/result_img"
      android:layout_width="0dp"
      android:layout_height="0dp"
      app:layout_constraintBottom_toBottomOf="parent"
      app:layout_constraintLeft_toLeftOf="parent"
      app:layout_constraintRight_toRightOf="parent"
      app:layout_constraintTop_toBottomOf="@id/compare_button" />

</androidx.constraintlayout.widget.ConstraintLayout>

Comments

[2r4h8k3m]

ソース完全に書いていただきすみません...アプリは起動するのに思うように動かない場合の対処を沢山教えて頂いたので、今こちらのソースを全部見るのでなくそちら試してみて答え合わせ的に参考にさせて頂けたらなと思います。比較だけの部品を動かしたい場合はdrawableを使ってアプリ内に用意して動かせばよいのかな、フォルダ内の画像をImageViewとして設定する命令も知らなかったので参考になりました。ありがとうございました。