<问题笔记> Android click与longclick冲突

2018-03-19

这几天赶工一个志愿者的项目，出现了这么一个问题：

E/WindowManager: android.view.WindowLeaked: Activity com.zeros.zeros.volunteer.HavePutActivity has leaked window com.android.internal.policy.PhoneWindow$DecorView{46e4818 V.E...... R....... 0,0-729,272} that was originally added here
                     at android.view.ViewRootImpl.<init>(ViewRootImpl.java:368)
                     at android.view.WindowManagerGlobal.addView(WindowManagerGlobal.java:299)
                     at android.view.WindowManagerImpl.addView(WindowManagerImpl.java:85)
                     at android.app.Dialog.show(Dialog.java:319)
                     at android.app.AlertDialog$Builder.show(AlertDialog.java:1112)
                     at com.zeros.zeros.volunteer.HavePutActivity$4.onItemLongClick(HavePutActivity.java:133)
                     at android.widget.AbsListView.performLongPress(AbsListView.java:3127)
                     at android.widget.AbsListView$CheckForLongPress.run(AbsListView.java:3059)
                     at android.os.Handler.handleCallback(Handler.java:739)
                     at android.os.Handler.dispatchMessage(Handler.java:95)
                     at android.os.Looper.loop(Looper.java:148)
                     at android.app.ActivityThread.main(ActivityThread.java:5417)
                     at java.lang.reflect.Method.invoke(Native Method)
                     at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
                     at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)
E/Surface: getSlotFromBufferLocked: unknown buffer: 0xae471700

问题是窗口泄露，当我们依托于一个activity来新建一个dialog的时候，如果在dialog还未dismiss之前，就将activity finish()掉，
就会产生这样的问题，于是，我开始分析自己的代码，难道我在dismiss之前关掉了这个activity？
然后发现，既要满足触发条件，有会产生触发结果的位置一定是这个listview，
因为我既要finish()，又要产生这样的错误，一定在点击事件里，

listview.setOnItemClickListener(new AdapterView.OnItemClickListener() {
            @Override
            public void onItemClick(AdapterView<?> parent, View view, int position, long id) {
                String activity_id=haveput_id.get(position);
                Bundle bundle=new Bundle();
                bundle.putString("activity_id",activity_id);
                Intent intent=new Intent(HavePutActivity.this,PersonsActivity.class);
                intent.putExtra("activity_id",bundle);
                startActivity(intent);
                finish();
            }
        });
        listview.setOnItemLongClickListener(new AdapterView.OnItemLongClickListener() {
            @Override
            public boolean onItemLongClick(AdapterView<?> parent, View view, int position, long id) {
                final String activity_id=haveput_id.get(position);
                new AlertDialog.Builder(HavePutActivity.this)
                        .setMessage("确定要删除这个活动吗？")
                        .setPositiveButton("确定", new DialogInterface.OnClickListener() {
                            @Override
                            public void onClick(DialogInterface dialog, int which) {
                                delete_request(activity_id);
                            }
                        })
                        .setNegativeButton("取消", new DialogInterface.OnClickListener() {
                            @Override
                            public void onClick(DialogInterface dialog, int which) {
                                dialog.dismiss();
                            }
                        }).show();
                return false;
            }
        });

难道这两个会冲突？答案是，设置出现纰漏确实会冲突。
让我们看看源码：

/**
     * Interface definition for a callback to be invoked when an item in this
     * AdapterView has been clicked.
     */
    public interface OnItemClickListener {
        /**
         * Callback method to be invoked when an item in this AdapterView has
         * been clicked.
         * <p>
         * Implementers can call getItemAtPosition(position) if they need
         * to access the data associated with the selected item.
         *
         * @param parent The AdapterView where the click happened.
         * @param view The view within the AdapterView that was clicked (this
         *            will be a view provided by the adapter)
         * @param position The position of the view in the adapter.
         * @param id The row id of the item that was clicked.
         */
        void onItemClick(AdapterView<?> parent, View view, int position, long id);
    }
    /**
     * Interface definition for a callback to be invoked when an item in this
     * view has been clicked and held.
     */
    public interface OnItemLongClickListener {
        /**
         * Callback method to be invoked when an item in this view has been
         * clicked and held.
         *
         * Implementers can call getItemAtPosition(position) if they need to access
         * the data associated with the selected item.
         *
         * @param parent The AbsListView where the click happened
         * @param view The view within the AbsListView that was clicked
         * @param position The position of the view in the list
         * @param id The row id of the item that was clicked
         * @return true if the callback consumed the long click, false otherwise
         */
        boolean onItemLongClick(AdapterView<?> parent, View view, int position, long id);
    }

注意下满参数注释：当我们将长事件callback返回值设置为true，系统才会消耗掉长事件！