ギャラリーから画像を選択し、リサイズして表示したい

Question

実現したいこと

ボタンクリックで本体に保存されている画像一覧を表示し、画像をリサイズして表示したいと考えています。
ライブラリにSimpleCropViewを使用して実現しようと考えています。

問題点

以下のようなサンプルアプリを作成しましたが、一度画像を読み込んだ状態で、再度「読み込み」ボタンを押すと、アプリが落ちてしまいます。

ログにはTransactionTooLargeExceptionと出ますが、画像が大きすぎる等の理由なら、一度目は落ちない理由もわからなく、行き詰まっています

実装方法に問題がありましたら、ご指摘して頂きたいです
どうかよろしくお願いします

↓「読み込み」ボタンでギャラリーから画像を読み込み

↓画像読み込み後に「読み込み」ボタンを押すとアプリが落ちる

エラーログ

Process: jp.co.sample.android.bungu.simplecropviewsample, PID: 12381
java.lang.RuntimeException: android.os.TransactionTooLargeException: data parcel size 3688300 bytes
at android.app.ActivityThread$StopInfo.run(ActivityThread.java:3877)
at android.os.Handler.handleCallback(Handler.java:751)
at android.os.Handler.dispatchMessage(Handler.java:95)
at android.os.Looper.loop(Looper.java:241)
at android.app.ActivityThread.main(ActivityThread.java:6274)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:886)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:776)
Caused by: android.os.TransactionTooLargeException: data parcel size 3688300 bytes
at android.os.BinderProxy.transactNative(Native Method)
at android.os.BinderProxy.transact(Binder.java:615)
at android.app.ActivityManagerProxy.activityStopped(ActivityManagerNative.java:3665)
at android.app.ActivityThread$StopInfo.run(ActivityThread.java:3869)
at android.os.Handler.handleCallback(Handler.java:751) 
at android.os.Handler.dispatchMessage(Handler.java:95) 
at android.os.Looper.loop(Looper.java:241) 
at android.app.ActivityThread.main(ActivityThread.java:6274)

コード

activity_main.xml

<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:orientation="vertical">

    <com.isseiaoki.simplecropview.CropImageView
        android:id="@+id/cropImageView"
        xmlns:custom="http://schemas.android.com/apk/res-auto"
        android:layout_width="match_parent"
        android:layout_height="250dp"
        android:padding="16dp"
        custom:cropMode="circle"
        />

    <Button
        android:id="@+id/readingButton"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:layout_margin="8dp"
        android:text="読み込み" />

    <Button
        android:id="@+id/cropbutton"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:layout_gravity="center"
        android:layout_margin="8dp"
        android:text="リサイズ" />

    <ImageView
        android:id="@+id/croppedImageView"
        android:layout_margin="16dp"
        android:layout_gravity="center"
        android:layout_width="100dp"
        android:layout_height="100dp"
        />

</LinearLayout>

MainActivity.java

public class MainActivity extends AppCompatActivity {

    private static final int RESULT_PICK_IMAGEFILE = 1001;
    private CropImageView mCropImageView;
    private ImageView mCroppedImageView;

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

        mCropImageView = (CropImageView)findViewById(R.id.cropImageView);
        mCroppedImageView = (ImageView)findViewById(R.id.croppedImageView);

        // 読み込みボタンがクリックされた時
        Button readingButton = (Button)findViewById(R.id.readingButton);
        readingButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {

                Intent intent = new Intent(Intent.ACTION_OPEN_DOCUMENT);
                intent.addCategory(Intent.CATEGORY_OPENABLE);
                intent.setType("image/*");
                startActivityForResult(intent, RESULT_PICK_IMAGEFILE);
            }
        });

        // 切り取りボタンがクリックされた時
        Button cropButton = (Button)findViewById(R.id.cropbutton);
        cropButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                // フレームに合わせてトリミング
                mCroppedImageView.setImageBitmap(mCropImageView.getCroppedBitmap());
            }
        });
    }


    @Override
    public void onActivityResult(int requestCode, int resultCode, Intent resultData) {

        if (requestCode == RESULT_PICK_IMAGEFILE && resultCode == Activity.RESULT_OK) {

            Uri uri = null;
            if (resultData != null) {
                uri = resultData.getData();
                Log.i("", "Uri: " + uri.toString());

                try {
                    Bitmap bmp = getBitmapFromUri(uri);

                    // トリミングしたい画像をセット
                    mCropImageView.setImageBitmap(bmp);
                } catch (IOException e) {
                    e.printStackTrace();
                }
            }
        }
    }

    private Bitmap getBitmapFromUri(Uri uri) throws IOException {
        ParcelFileDescriptor parcelFileDescriptor =
                getContentResolver().openFileDescriptor(uri, "r");
        FileDescriptor fileDescriptor = parcelFileDescriptor.getFileDescriptor();
        Bitmap image = BitmapFactory.decodeFileDescriptor(fileDescriptor);
        parcelFileDescriptor.close();
        return image;
    }
}

Answer 1

初めまして。おそらく端末がAndroid7.0以降であるという前提で進めます。もし違った場合は他の原因が考えられます。
こちらの公式ドキュメントを見ると、以下のように書いてあります。

多くのプラットフォーム API は、Binder トランザクションで送信される大きなペイロードをチェックし、暗黙的にログ記録したり、削除したりするのではなく TransactionTooLargeExceptions を RuntimeExceptions として再度スローするようになりました。一般的な例としては、Activity.onSaveInstanceState() で大量のデータを格納することです。これにより、アプリが Android 7.0 をターゲットにしている場合は、ActivityThread.StopInfo で RuntimeException がスローされます。

つまり、質問者様の場合、Imageをsetした状態でもう一度読み込みボタンを押してActivityが遷移するときにActivity#onSaveInstanceState()が呼ばれるのですが、setした画像の容量が大きすぎると上記の理由からクラッシュしていると考えられます。

パッと思いつく方法ですと、以下の3種類でしょうか。

• 画像を圧縮して別ファイルを作り、その画像を表示する。
• Glideなどのライブラリを使って画像を表示する（内部でメモリ管理などをしてくれるようです）、
• onPause()で、以下のように実装する。

@Override
  protected void onPause() {
      mCropImageView.setImageBitmap(null); // 画像が空欄になります
      super.onPause();
  }

---

追記
ライブラリのissueを見ると、ライブラリのバージョンが1.1.5以降ではそのバグを消したという旨が書いてあるのですが、ライブラリのバージョンは最新版でしょうか？？

Comments

[itouuuuuuuuu]

丁寧なご回答ありがとうございます

以下の操作を試してみましたが、以前同様のエラーが出てしまいます
・picassoを用いて画像を読み込み

```java
@Override
public void onActivityResult(int requestCode, int resultCode, Intent resultData) {

if (requestCode == RESULT_PICK_IMAGEFILE && resultCode == Activity.RESULT_OK) {

Uri uri = null;
if (resultData != null) {
uri = resultData.getData();
Log.i("", "Uri: " + uri.toString());

Picasso.with(this)
.load(uri)
.fit()
.centerInside()
.into(mCropImageView);
}
}
}
```

・`onPause()`の修正

`onPause()`で` mCropImageView.setImageBitmap(null);`を実装した際、`mCropImageView`の`mBitmap`無いはnullになっておらず、初期化出来ていないようです
（初回にbitmapをsetする前は、`mBitmap`はnullになっている）

なにか気づく点がございましたら、アドバイスいただけたらと思います

[hiramekun]

なるほど、ライブラリのレポジトリのissueにて、最新版ではこのバグが修正されていると書いてあったので、その旨を回答に追記しました。ご確認ください。

[itouuuuuuuuu]

ライブラリのバージョンが少し古かったようです
最新版にしたら問題は発生しませんでした
確認不足で申し訳ありません！

その他、onPauseでの開放方法など大変勉強になりました
ありがとうございます！