PopupMenu выбрасывает WindowLeaked при повороте устройства

#java #android #popupmenu #recyclerview

Реализую overflow-кнопку для элемента списка, кнопка обрабатывается в холдере адаптера:

class CustomAdapter extends RecyclerView.Adapter  {

....

 public static class ItemHolder extends RecyclerView.ViewHolder implements View.OnClickListener{

        ImageButton mMenuItem;
        CardView mCard;

        public ItemHolder(View v) {
            super(v);

            mCard = (CardView) v.findViewById(R.id.card);
            mMenuItem = (ImageButton) v.findViewById(R.id.menu_button);

            mMenuItem.setOnClickListener(this);
            mCard.setOnClickListener(this);
        }

        @Override
        public void onClick(View v) {

            int position = getAdapterPosition();
            if (position != RecyclerView.NO_POSITION) {
                switch (v.getId()) {
                    case R.id.menu_button:
                        PopupMenu popupMenu = new PopupMenu(mMenuItem.getContext(),
mMenuItem);
                        popupMenu.inflate(R.menu.popup_menu);
                        popupMenu.show();
                        break;
                    case R.id.card:
                        //
                        break;
            }
       }
  }
 }


Если меню раскрыто, то при повороте устройства получаю исключение:


  E WindowManager: android.view.WindowLeaked: Activity com.example.android.test.MyActivity


Понятно, что менюшка теряет прошлую активити при пересоздании и, в общем то, это
не приводит к падению приложения и вообще никак не проявляется, если не смотреть logcat,
но хотелось бы решить проблему.
Собственно решение, как таковое, известно - закрывать PopupMenu в методах закрытия
активити, например onPause() - popupmenu.dissmis(). Проблема в том, что это меню создается
в адаптере и ему о жизненном цикле активити ничего не известно. Можно, конечно, уведомить
адаптер, что вызван метод onPause() и закрыть эту менюшку, но получается все довольно
коряво - нужно переносить инициализацию меню в сам адаптер и тп. нежелательные решения.

Посмотрел большое количество примеров реализации, в том числе эталонный iosched,
везде эта проблема просто игнорируется, даже гуглом.

Вопрос такой:  как решить и стоит ли вообще обращать внимание на эту проблему или
какие то альтернативные способы реализации меню в айтеме для RecyclerView.
    

Ответы

Ответ 1

Поскольку вложенный в адаптер класс ItemHolder статический, то для решения проблемы
сделал следующее:


Объявил PopupMenu статическим полем класса - теперь все экземпляры меню будут "связаны",
как будто это один экземпляр:

public static class ItemHolder extends RecyclerView.ViewHolder implements View.OnClickListener{

 ImageButton mMenuItem;
 CardView mCard;
 public static PopupMenu popupMenu;
 ...

  @Override
  public void onClick(View v) {
  ...
    if (position != RecyclerView.NO_POSITION) {
        switch (v.getId()) {
            case R.id.menu_button:
                popupMenu = new PopupMenu(mMenuItem.getContext(), mMenuItem);
                ...
    }
 }
}

В onPause() активити, в которую выводится список, обращаюсь напрямую к этому статическому
экземпляру через класс адаптера с предложением закрыться:

@Override
protected void onPause() {
  super.onPause();

  PopupMenu popupMenu = CustomAdapter.ItemHolder.popupMenu;
  if (popupMenu != null) popupMenu.dismiss();
  popupMenu = null; // защита от утечек памяти
}



При желании можно написать геттер, чтобы отдавал PopupMenu. В данном случае я не
вижу в этом особой надобности.

PS: По прежнему хотел бы услышать более оптимальное решение.


Ответ 2

Для решения проблемы нам надо:


В нужный момент (при "смерти" активити) закрыть PopupMenu.
А для этого надо получить ссылку на объект PopupMenu.
А оно определено в RecyclerView.ViewHolder.
Значит нам надо получить ссылку на RecyclerView.ViewHolder
И в нём-то и будет искомое PopupMenu.


Добавим поле в холдер для хранения меню, а также геттер:

public static class ItemHolder extends RecyclerView.ViewHolder implements View.OnClickListener
{

    ImageButton mMenuItem;
    CardView mCard;
    //сюда будем ссылку на меню сохранять
    PopupMenu popup;
    //так будем его из холдера получать
    public PopupMenu getPopup()
    {
        return popup;
    }

    public ItemHolder(View v) {
        super(v);

        mCard = (CardView) v.findViewById(R.id.card);
        mMenuItem = (ImageButton) v.findViewById(R.id.menu_button);

        //устанавливаем тэг одному из элементов разметки
        //чтобы потом вызовом View.getTag() получить ссылку на экземпляр холдера
        mMenuItem.setTag(this);

        mMenuItem.setOnClickListener(this);
        mCard.setOnClickListener(this);
    }

    @Override
    public void onClick(View v) {

        int position = getAdapterPosition();
        if (position != RecyclerView.NO_POSITION) {
            switch (v.getId()) {
                case R.id.menu_button:
                    //инициализируем поле класса
                    popup = new PopupMenu(mMenuItem.getContext(), mMenuItem);

                    popupMenu.inflate(R.menu.popup_menu);
                    popupMenu.show();
                    break;
                case R.id.card:
                    //
                    break;
        }
    }
}


Все инструменты готовы, теперь можно в методах жизненного цикла Activity или Fragment,
используя лишь ссылку на RecyclerView, находить PopupMenu в элементах списка и вызывать
над ними PopupMenu.dismiss(), что устранит вываливание в логи сообщения об ошибке:

    //можно оптимизировать, но для простоты пробежимся по всем элементам
    for (int i = 0; i < recyclerView.getLayoutManager().getChildCount(); i++)
    {
        View recyclerViewElement = recyclerView.getLayoutManager().findViewByPosition(i);
        if (recyclerViewElement == null)
        {
            continue;
        }
        //находим в элементе View, хранящую ссылку на холдер
        View viewThatHoldsReferenceToHolder = recyclerViewElement.findViewById(R.id.menu_button);
        if (viewThatHoldsReferenceToHolder == null)
        {
            continue;
        }
        //получаем ссылку на холдер
        ItemHolder holder = ((ItemHolder) viewThatHoldsReferenceToHolder .getTag());
        if (holder != null)
        {
            //Вот оно - наше меню!
            PopupMenu popup = holder.getPopup();
            if (popup != null)
            {
                //Закрываем его и задачу на правку бага
                popup.dismiss();
                Log.d(LOG, "Красота, а не ругань системы в логах! =)");
            }
        }
    }


В случае же вызывания PopupMenu из адаптера всё гораздо проще - получаем ссылку на
меню из адаптера и... Всё, вопрос решён. Буду считать сей случай доводом в пользу писания
всей логики обработки элемента списка в адаптере, а не в классе холдера.