弹出菜单正在累积事件（没有正确解雇）

Question

我的应用程序中有一个奇怪的错误。我有一个包含列表项的RecyclerView，其中包含溢出菜单的图标。见下文.enter image description here

当我触摸菜单外部（取消它）时会出现问题。下次选择菜单时，它会呈现2个菜单。这种情况持续了很多次。见下文。 enter image description here

我无法弄清楚为什么会这样。这是我在ViewHolder列表项中显示菜单的代码：

final PopupMenu popupMenu = new PopupMenu(mContext, mOptionsMenu);

                mOptionsMenu.setOnClickListener(new View.OnClickListener() {
                    @Override
                    public void onClick(View v) {

                        popupMenu.inflate(R.menu.menu_edit_delete_dq_competitor);

                        Menu menu = popupMenu.getMenu();
                        MenuItem edit = menu.findItem(R.id.edit);
                        MenuItem delete = menu.findItem(R.id.delete);
                        MenuItem dq = menu.findItem(R.id.dq);
                        MenuItem undoDq = menu.findItem(R.id.undo);

                        // change options if this competitor is DQ
                        if (competitor.getStatus() == Status.DQ) {
                            edit.setVisible(false);
                            delete.setVisible(false);
                            dq.setVisible(false);
                            undoDq.setVisible(true);
                        }
                        popupMenu.show();
                    }
                });

以下是我选择菜单项时的代码：

      popupMenu.setOnMenuItemClickListener(new PopupMenu.OnMenuItemClickListener() {
                    @Override
                    public boolean onMenuItemClick(MenuItem item) {
                        FragmentManager manager = getActivity().getSupportFragmentManager();
                        switch (item.getItemId()) {
                            case R.id.edit:
                                // todo finish this
//                                Intent intent = CompetitorInMatchForm.editIntent(mContext, competitor, true);
//                                startActivity(intent);
                                break;
                            case R.id.delete:
                                CompetitorStatusInMatchDialog deleteDialog = CompetitorStatusInMatchDialog.newInstance(competitor, "delete", getAdapterPosition());
                                deleteDialog.setTargetFragment(CompsInMatchFragment.this, REQUEST_ACTION);
                                deleteDialog.show(manager, DELETE_COMP_FROM_MATCH);
                                break;
                            case R.id.dq:
                                CompetitorStatusInMatchDialog dqDialog = CompetitorStatusInMatchDialog.newInstance(competitor, "dq", getAdapterPosition());
                                dqDialog.setTargetFragment(CompsInMatchFragment.this, REQUEST_ACTION);
                                dqDialog.show(manager, DELETE_COMP_FROM_MATCH);
                                break;
                            case R.id.undo:
                                CompetitorStatusInMatchDialog undoDialog = CompetitorStatusInMatchDialog.newInstance(competitor, "undo", getAdapterPosition());
                                undoDialog.setTargetFragment(CompsInMatchFragment.this, REQUEST_ACTION);
                                undoDialog.show(manager, DELETE_COMP_FROM_MATCH);
                                break;
                            default:
                                popupMenu.dismiss();
                        }

                        return true;
                    }
                });

任何帮助确定对话框累积菜单出现的原因都会很棒。我之前使用弹出菜单没有这个问题。我希望你在菜单外面触摸它时会被解雇。

Answer 1

这是因为你在onClick监听器内膨胀菜单，因此实际上每次显示菜单时都会添加所有元素。

如果您希望dismiss将从菜单中删除项目，那就错了：它只会隐藏弹出窗口。

您可以将此行移到侦听器之外：

popupMenu.inflate(R.menu.menu_edit_delete_dq_competitor);

像这样：

final PopupMenu popupMenu = new PopupMenu(mContext, mOptionsMenu);
popupMenu.inflate(R.menu.menu_edit_delete_dq_competitor);