ファイル保存完了タイミングを検知したい

Question

###前提・実現したいこと
JAVAでAndroidスマホ用アプリを作っています。
カメラ機能を使用したいため、
Android Studioからimportしたsampleパッケージ「Camera2Basic」をベースに必要な機能を追加実装しています。
「撮影」ボタンの押下をトリガーとして、写真を撮影し、保存した上で、写真をLayoutに設定されたImageViewに表示したいです。

※初心者なので、基本的なところを理解できていないと思います。この質問では、上記の問題を解決するにあたり、論理的な意味で、どのあたりをチェック・修正すればよいか、などを教えていただきたいです。

※一旦、ポイントと思われる箇所のソースのみを抜粋して添付し、必要に応じて、追加したいと思います。

###発生している問題・エラーメッセージ
「撮影」ボタンをトリガーとして、写真撮影し、保存する事、まではできましたが、保存処理の完了を知ることができないため、ファイルが保存されたタイミングでファイルをImageViewに保存する事ができません。

###該当のソースコード

<fragment_camera2_basic.xml>
…
            <ImageView
                android:id="@+id/foto"
                android:layout_width="match_parent"
                android:layout_height="wrap_content"
                android:layout_marginTop="10dp"
                android:layout_marginLeft="10dp"
                android:layout_marginEnd="10dp"
                android:layout_marginRight="10dp"
                android:layout_toEndOf="@+id/rl1"
                android:adjustViewBounds="true"
                android:src="@drawable/foto"
                tools:layout_alignParentTop="true" />
…

<fragment_camera2_basic.java>

public class Camera2BasicFragment extends Fragment
        implements View.OnClickListener, FragmentCompat.OnRequestPermissionsResultCallback {

…
    private ImageView mImageView;
…
    private final ImageReader.OnImageAvailableListener mOnImageAvailableListener
            = new ImageReader.OnImageAvailableListener() {

        @Override
        public void onImageAvailable(ImageReader reader) {
//            mBackgroundHandler.post(new ImageSaver(reader.acquireNextImage(), mFile));
            mBackgroundHandler.post(new ImageSaver(reader.acquireNextImage(), mFile, mImageView));
        }

    };
…
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
                             Bundle savedInstanceState) {
        View v = inflater.inflate(R.layout.fragment_camera2_basic, container, false);
        return v;
…
    public void onViewCreated(final View view, Bundle savedInstanceState) {
        //freddie
        view.findViewById(R.id.picture).setOnClickListener(this);
        view.findViewById(R.id.sb).setOnClickListener(this);
        mTextureView = (AutoFitTextureView) view.findViewById(R.id.texture);

        mImageView = (ImageView) view.findViewById(R.id.foto);
    }
…
    private void takePicture() {
        lockFocus();
        Bitmap src = BitmapFactory.decodeFile(mFile.toString());
        mImageView.setImageBitmap(src);★
    }
…
＜追加＞
        public void run() {
            ByteBuffer buffer = mImage.getPlanes()[0].getBuffer();
            byte[] bytes = new byte[buffer.remaining()];
            buffer.get(bytes);
            FileOutputStream output = null;
            try {
                output = new FileOutputStream(mFile);
                output.write(bytes);
            } catch (IOException e) {
                e.printStackTrace();
            } finally {
                mImage.close();
                if (null != output) {
                    try {
                        output.close();
                    } catch (IOException e) {
                        e.printStackTrace();
                    }
                }

//            Bitmap src = BitmapFactory.decodeFile(mFile.toString());
//            mImageView.setImageBitmap(src);
            }
…

★この処理はエラーにならないものの、takePitureで撮影された写真が保存される前のファイルが表示されてしまう。

###試したこと
保存される写真ファイル名を固定にした上で、写真撮影処理の直後に、ImageViewへの写真表示処理を追加したところ、写真表示を行うことはできました。
※ただし、そのタイミングでは、まだ保存が完了していないため、前回、撮影されたファイルが表示されてしまいます。

###補足情報(言語/FW/ツール等のバージョンなど)
Android Studioバージョン：2.3.1
Androidデバイス 6.0.1

Comments

[KSwordOfHaste]

読み手にコードを見やすく提示することも大切なので、ぜひ質問を編集しコードを```で囲み字下げしてみてください。

[foomo]

コメントありがとうございます。便利な機能があるんですね。対応しました。

[KSwordOfHaste]

面倒かもですが半角スペースなどを行の先頭へ挿入してきちんと字下げしましょう。字下げはコードの論理構造を一目でわかるようにするために(読み手のレベルにかかわらず)重要です。せっかくタグで囲んでも字下げがないと苦労して編集した甲斐がないですよ～

Answer 1

https://github.com/googlesamples/android-Camera2Basic/blob/master/Application/src/main/java/com/example/android/camera2basic/Camera2BasicFragment.java#L916

ソースコードを見る限り、ImageSaver クラスは保存完了しても何も通知しないようです。

そのため、public void run() へ保存完了のコールバックを用意してやる必要があります。
そしてそのコールバック関数の中で、★ をしてやれば画像保存後ですので期待している動きになります。

Comments

[foomo]

コメントありがとうございました。
追加したrun()の末尾に、表示処理を追加して実行する、までは試していたのですが、うまく動いてくれませんでした。
いただいたコメントをとっかかりに進めてみようと思いますが、あと少し教えていただいてもよいでしょうか。
添付のrun()で言うと、どこにコールバックを仕掛けるのが良いのでしょうか。

[KSwordOfHaste]

多分、TakeoAsaiさんは暗黙的にUIスレッド用のHandlerへpostすることを意図していると思います。直接メソッドを呼び出すと「バックグラウンドスレッドからUIコンポーネントを触ってはいけない」という制約に抵触すると思います。
---
自分はmBackgroundHandler.post(...)がバックグラウンド(UIスレッドと違うスレッド)でImageServerを動かしていると解釈したので上のように思いました。勘違いだったらスミマセン

[foomo]

コメントありがとうございます。
スキル的に、いただいたコメントを元に手が動くレベルではないのですが、
先に進める糸口はつかめました。

[KSwordOfHaste]

自分は例えば、こんな記事をみてAndroidのスレッドの考え方を理解しようとしました。
http://qiita.com/yamacraft/items/a10b5d0135d72258ac03
キーワードとしては「Android UIスレッド Handler」あたりでしょうか・・・

[foomo]

参考情報ありがとうございます。
小さなプログラムを作りながら、
動作を把握してみたいと思います。