[Android]AlertDialog.Builder- dismiss() 사용 남발

요약

AlertDialog.Builder로 다이얼로그 생성시,

1. 버튼의 리스너 안에 다이얼로그의 dismiss()를 호출할 필요없다.

2. 캔슬 리스너 안에 다이얼로그의 dismiss()를 호출할 필요없다.

---

회사 프로젝트의 기존 코드를 보다가 의문이 생겼다. 이번에는 확실히 알고 넘어가야지 싶어서 기록한다.

 

기존 코드

Dialog = new AlertDialog.Builder(context)
	    .setTitle("타이틀")
            .setMessage("메시지")
            .setPositiveButton("확인", (dialog, which) -> doWork())
            .setNegativeButton("취소", (dialog, which) -> dialog.dismiss()) //문제의 구간1
            .setOnCancelListener((dialog) -> dialog.dismiss()); //문제의 구간2
            .create();

 

문제의 구간1 : 버튼 클릭 시 dismiss()를 구지 호출해야 하는가?

1) AlertDialog.java 

        /**
         * Set a listener to be invoked when the negative button of the dialog is pressed.
         * @param textId The resource id of the text to display in the negative button
         * @param listener The {@link DialogInterface.OnClickListener} to use.
         *
         * @return This Builder object to allow for chaining of calls to set methods
         */
        public Builder setNegativeButton(@StringRes int textId, final OnClickListener listener) {
            P.mNegativeButtonText = P.mContext.getText(textId);
            P.mNegativeButtonListener = listener;
            return this;
        }

mNagativeButtonListener를 추적한다.

 

 2) AlertController.java

 

         public void apply(AlertController dialog) {
            ...
            if (mNegativeButtonText != null || mNegativeButtonIcon != null) {
                dialog.setButton(DialogInterface.BUTTON_NEGATIVE, mNegativeButtonText,
                        mNegativeButtonListener, null, mNegativeButtonIcon);
            }
           ...
        }

 

setButton()을 추적한다.

 

mNegativeButtonText를 통해 negative button 객체를 추적한다.

 

3) AlertController.java

private void setupButtons(ViewGroup buttonPanel) {
        ...

        mButtonNegative = buttonPanel.findViewById(android.R.id.button2);
        mButtonNegative.setOnClickListener(mButtonHandler);

        if (TextUtils.isEmpty(mButtonNegativeText) && mButtonNegativeIcon == null) {
            mButtonNegative.setVisibility(View.GONE);
        } else {
            mButtonNegative.setText(mButtonNegativeText);
            if (mButtonNegativeIcon != null) {
                mButtonNegativeIcon.setBounds(0, 0, mButtonIconDimen, mButtonIconDimen);
                mButtonNegative.setCompoundDrawables(mButtonNegativeIcon, null, null, null);
            }
            mButtonNegative.setVisibility(View.VISIBLE);
            whichButtons = whichButtons | BIT_BUTTON_NEGATIVE;
        }

        ...
    }

mButtonNegative라는 객체가 있음을 확인했다.

이 버튼의 클릭 리스너 객체인 mButtonHandler를 추적한다.

 

4) AlertContoller.java

    private final View.OnClickListener mButtonHandler = new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            final Message m;
            if (v == mButtonPositive && mButtonPositiveMessage != null) {
                m = Message.obtain(mButtonPositiveMessage);
            } else if (v == mButtonNegative && mButtonNegativeMessage != null) {
                m = Message.obtain(mButtonNegativeMessage);
            } else if (v == mButtonNeutral && mButtonNeutralMessage != null) {
                m = Message.obtain(mButtonNeutralMessage);
            } else {
                m = null;
            }

            if (m != null) {
                m.sendToTarget();
            }

            // Post a message so we dismiss after the above handlers are executed
            mHandler.obtainMessage(ButtonHandler.MSG_DISMISS_DIALOG, mDialog)
                    .sendToTarget();
        }
    };

mButtonNegativeMessage가 null이 아닐 경우 m에 Message 참조가 된다.

m.sendtoTarget() 실행 후, 핸들러가 ButtonHandler.MSG_DISMISS_DIALOG를 담은 Message를 실행하도록 한다.

 

mHandler는 어떤 객체인지 추적한다.

 

5) AlertController.java

    public AlertController(Context context, AppCompatDialog di, Window window) {
        ...
        mHandler = new ButtonHandler(di);

       ...
    }

ButtonHandler는 어떤 클래스인지 추적한다.

 

6) AlertController.java

    private static final class ButtonHandler extends Handler {
        // Button clicks have Message.what as the BUTTON{1,2,3} constant
        private static final int MSG_DISMISS_DIALOG = 1;

        private WeakReference<DialogInterface> mDialog;

        public ButtonHandler(DialogInterface dialog) {
            mDialog = new WeakReference<>(dialog);
        }

        @Override
        public void handleMessage(Message msg) {
            switch (msg.what) {

                case DialogInterface.BUTTON_POSITIVE:
                case DialogInterface.BUTTON_NEGATIVE:
                case DialogInterface.BUTTON_NEUTRAL:
                    ((DialogInterface.OnClickListener) msg.obj).onClick(mDialog.get(), msg.what);
                    break;

                case MSG_DISMISS_DIALOG:
                    ((DialogInterface) msg.obj).dismiss();
            }
        }
    }

MSG_DISMISS_DIALOG 값을 받으면 다이얼로그를 dismiss()시킨다.

 

다이얼로그의 버튼 리스너에 구지 다이얼로그의 dismiss() 코드를 삽입할 필요가 없다. 클릭하면 dismiss()가 실행되도록 기본적으로 짜여져 있다.

 

 

문제의 구간2 : 캔슬 리스너에 구지 dismiss()를 호출해야 하는가

1) Dialog.java

    /**
     * Cancel the dialog.  This is essentially the same as calling {@link #dismiss()}, but it will
     * also call your {@link DialogInterface.OnCancelListener} (if registered).
     */
    @Override
    public void cancel() {
        if (!mCanceled && mCancelMessage != null) {
            mCanceled = true;
            // Obtain a new message so this dialog can be re-used
            Message.obtain(mCancelMessage).sendToTarget();
        }
        dismiss();
    }

 

cancel()을 호출하면 dismiss()를 알아서 호출해준다. 따라서 OnCancleListsener에 dialog를 dismiss()시키는 코드를 삽입할 필요가 없다.

 

 

스택오버플로우에 답변으로 나오는 코드들 중 negative 버튼의 리스너에 dialog.dismiss()가 작성된 경우들이 종종 있는 것을 확인했다. 스택오버플로우 맹신 금물.