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

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);
        }
}

Zuishin

2021/09/04 01:56

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

2r4h8k3m

2021/09/04 02:20 編集

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

m.ts10806

2021/09/04 02:02

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

2r4h8k3m

2021/09/04 02:20 編集

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

Zuishin

2021/09/04 02:21

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

2r4h8k3m

2021/09/04 02:28 編集

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

Zuishin

2021/09/04 02:50

> デバッグ自体が何度かやってみたのですが結局何をしているのかわからないまま終わってしまっていたのでもう一度やってみます。わからないなら何度やっても無駄でしょう。わかるようになってからやるという選択肢があります。まずドキュメントを読んでください。そんなに難しいことは書いていないはずです。

Zuishin

2021/09/04 02:51

https://developer.android.com/studio/debug?hl=ja ここです。図入りで簡単でしょう？

m.ts10806

2021/09/04 03:02

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

jimbe

2021/09/04 03:21

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

dodox86

2021/09/04 03:44

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

dodox86

2021/09/04 03:47

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

hoshi-takanori

2021/09/04 04:53

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

2r4h8k3m

2021/09/04 06:52 編集

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

行動規範の内容に同意します

回答1件

ベストアンサー

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

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

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

MainActivity.java

java
1package com.teratail.q357676;
2
3import androidx.appcompat.app.AppCompatActivity;
4
5import android.graphics.*;
6import android.graphics.drawable.*;
7import android.os.Bundle;
8import android.view.View;
9import android.widget.ImageView;
10
11public class MainActivity extends AppCompatActivity {
12  private ImageView src_img1;
13  private ImageView src_img2;
14  private ImageView result_img;
15
16  @Override
17  protected void onCreate(Bundle savedInstanceState) {
18    super.onCreate(savedInstanceState);
19    setContentView(R.layout.activity_main);
20
21    src_img1 = (ImageView)findViewById(R.id.src_img1);
22    src_img1.setImageResource(R.drawable.k_202105_0); //テスト画像1
23
24    src_img2 = (ImageView)findViewById(R.id.src_img2);
25    src_img2.setImageResource(R.drawable.k_202105_1); //テスト画像2 ※src_img1と同じ大きさであること
26
27    result_img = (ImageView)findViewById(R.id.result_img);
28  }
29
30  public void onCameraDevitionClick(View view) {
31    Bitmap src_bitmap1 = getBitmap(src_img1);
32    Bitmap src_bitmap2 = getBitmap(src_img2);
33
34    Bitmap result_bitmap = createDiffBitmap(src_bitmap1, src_bitmap2);
35    result_img.setImageBitmap(result_bitmap);
36  }
37
38  private Bitmap getBitmap(ImageView imageView) {
39    if(imageView.getDrawable() instanceof BitmapDrawable) {
40      return ((BitmapDrawable)imageView.getDrawable()).getBitmap();
41    }
42    Drawable d = imageView.getDrawable();
43    Bitmap bitmap = Bitmap.createBitmap(d.getIntrinsicWidth(), d.getIntrinsicHeight(), Bitmap.Config.ARGB_8888);
44    d.draw(new Canvas(bitmap));
45    return bitmap;
46  }
47
48  private Bitmap createDiffBitmap(Bitmap src_bitmap1, Bitmap src_bitmap2) {
49    final int width = src_bitmap1.getWidth();
50    final int height = src_bitmap1.getHeight();
51    Bitmap dist_bitmap = Bitmap.createBitmap(width, height, Bitmap.Config.ARGB_8888);
52
53    for (int x = 0; x < width; x++) {
54      for (int y = 0; y < height; y++) {
55        int src_color1 = src_bitmap1.getPixel(x, y);
56        int src_color2 = src_bitmap2.getPixel(x, y);
57        int dist_color = isDifference(src_color1, src_color2) ? -11111111 : src_color1;
58        dist_bitmap.setPixel(x, y, dist_color);
59      }
60    }
61
62    return dist_bitmap;
63  }
64
65  private boolean isDifference(int color1, int color2) {
66    return Math.abs(color1 - color2) > 500000;
67  }
68}

レイアウト: activity_main.xml

xml
1<?xml version="1.0" encoding="utf-8"?>
2<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
3    xmlns:app="http://schemas.android.com/apk/res-auto"
4    xmlns:tools="http://schemas.android.com/tools"
5    android:layout_width="match_parent"
6    android:layout_height="match_parent"
7    tools:context=".MainActivity">
8
9  <ImageView
10      android:id="@+id/src_img1"
11      android:layout_width="0dp"
12      android:layout_height="0dp"
13      app:layout_constraintBottom_toTopOf="@id/src_img2"
14      app:layout_constraintLeft_toLeftOf="parent"
15      app:layout_constraintRight_toRightOf="parent"
16      app:layout_constraintTop_toTopOf="parent" />
17  <ImageView
18      android:id="@+id/src_img2"
19      android:layout_width="0dp"
20      android:layout_height="0dp"
21      app:layout_constraintBottom_toTopOf="@id/compare_button"
22      app:layout_constraintLeft_toLeftOf="parent"
23      app:layout_constraintRight_toRightOf="parent"
24      app:layout_constraintTop_toBottomOf="@id/src_img1" />
25  <Button
26      android:id="@+id/compare_button"
27      android:layout_width="wrap_content"
28      android:layout_height="wrap_content"
29      android:text="比較"
30      android:onClick="onCameraDevitionClick"
31      app:layout_constraintBottom_toTopOf="@id/result_img"
32      app:layout_constraintLeft_toLeftOf="parent"
33      app:layout_constraintRight_toRightOf="parent"
34      app:layout_constraintTop_toBottomOf="@id/src_img2" />
35  <ImageView
36      android:id="@+id/result_img"
37      android:layout_width="0dp"
38      android:layout_height="0dp"
39      app:layout_constraintBottom_toBottomOf="parent"
40      app:layout_constraintLeft_toLeftOf="parent"
41      app:layout_constraintRight_toRightOf="parent"
42      app:layout_constraintTop_toBottomOf="@id/compare_button" />
43
44</androidx.constraintlayout.widget.ConstraintLayout>

投稿2021/09/04 07:34

編集2021/09/04 11:59

jimbe

総合スコア13209

2r4h8k3m

2021/09/04 11:25

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

行動規範の内容に同意します