mokkouyouの開発日記

ダイアログ表示中に画面をローテートしてみてください。

バージョンによってはエラーが発生すると思います。

java.lang.IllegalArgumentException: View not attached to window manager

Dialog#dismiss()

でエラーになります。

Dialog自体は、エンクロージングActivityにアタッチされていますが、

このActivityは画面切替時に破棄（だか再利用だか）されてしまいます。

この際ダイアログもまとめて処理してくれればいいのですが、

残念ながら対応してくれません。

じゃぁ、ちゃんとActivityのonPauseあたりでハンドリングしてやりましょう。

きっとキャンセル呼んであげればいいんだ・・・って事でこんな感じにしてみます。

    private DemoAsync mAsync;
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);
        mAsync = new DemoAsync();
        mAsync.execute();
    }
    
    @Override
    protected void onPause() {
        if(mAsync != null && !mAsync.isCancelled()) {
            mAsync.cancel(true);
        }
        super.onPause();
    }

はい。残念ながら駄目だったりします。

じゃぁどうするの？ということでひとまずの解はこんな感じですかね。

※実際に、不要な処理になっている箇所もあります。

※ただ、このサンプルだと、onCreateでダイアログ表示しているので、切替後も表示されているように見えますけど、

これは、再度表示されているだけです。

public class AsyncDemo extends Activity {
    private DemoAsync mAsync;
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);
        mAsync = new DemoAsync();
        mAsync.execute();
    }
    
    @Override
    protected void onPause() {
        if(mDialog != null && mDialog.isShowing()) {
            mDialog.dismiss();
            mDialog = null;
        }
        if(mAsync != null && !mAsync.isCancelled()) {
            mAsync.cancel(true);
        }
        super.onPause();
    }

    private ProgressDialog mDialog;
    class DemoAsync extends AsyncTask<Void, Void, Void> {
        
        public DemoAsync() {
            mDialog = new ProgressDialog(AsyncDemo.this);
        }
        @Override
        protected Void doInBackground(Void... params) {
            try {
                TimeUnit.SECONDS.sleep(5l);
            } catch (InterruptedException e) {
            }
            return null;
        }

        @Override
        protected void onCancelled() {
            Log.d("", "onCancel");
            if(mDialog != null && mDialog.isShowing()) {
                mDialog.dismiss();
                mDialog = null;
            }
            super.onCancelled();
        }

        @Override
        protected void onPostExecute(Void result) {
            Log.d("", "onPostExecute");
            super.onPostExecute(result);
            if(mDialog != null && mDialog.isShowing()) {
                mDialog.dismiss();
                mDialog = null;
            }
        }

        @Override
        protected void onPreExecute() {
            super.onPreExecute();
            mDialog.setIndeterminate(true);
            mDialog.setMessage("Dialog");
            mDialog.show();
        }
    }
}

とりあえず対策してみましたが、そもそもローテート禁止にする

などという対応やら色々あると思いますので、

識者の方々などからの突っ込みをお待ちしています。