0

Как исправить утечку по отчету из leakcanary? Утечка на 1.5 мегабайта. Фото:

введите сюда описание изображения введите сюда описание изображения

mainView - ссылка на интерфейс активити ClickListener в адаптере:

private class OnItemClickListener implements View.OnClickListener {
        private String songId;

        private OnItemClickListener(String songId) {
            this.songId = songId;
        }

        @Override
        public void onClick(View view) {
            mainView.showSelectedSong(songId, listPhotoLoaded);
        }
    }

ClickListener одной из View:

private class OnItemToneClickListener implements View.OnClickListener {
        private int clickPosition;

        private OnItemToneClickListener(int clickPosition) {
            this.clickPosition = clickPosition;
        }

        @Override
        public void onClick(View view) {
            if (clickPosition != currentCheckedRadioButton) {
                radioButtonsTones.get(currentCheckedRadioButton).setChecked(false);
                radioButtonsTones.get(clickPosition).setChecked(true);
                currentCheckedRadioButton = clickPosition;
            }
        }
    }

Слушатель по нажатию избранного:

@OnClick(R.id.imageButtonFavorite)
    void favoriteClick() {
        currentSongPresenter.setFavoriteState();
    }

Ещё:

@Override
    public void showFavoriteDialog(boolean addFavorite) {
        final Dialog dialog = new Dialog(mainActivity);
        if (addFavorite) {
            dialog.setContentView(R.layout.dialog_favorite_add);
        } else {
            dialog.setContentView(R.layout.dialog_favorite_delete);
        }

        dialog.findViewById(R.id.parentLayout).setOnClickListener(v -> dialog.dismiss());
        dialog.show();
        final Handler handler = new Handler();
        handler.postDelayed(dialog::dismiss, 3000);
        setFavoriteImageState(addFavorite);
    }
4
  • 1
    Вложите фото в вопрос, а не по ссылке 27 апр 2017 в 6:14
  • @VladimirParfenov Очень длинное фото, по этому сделал по ссылке
    – iamtihonov
    27 апр 2017 в 6:38
  • Нужно разбивать на 2 фото, сжимать для того, чтобы они не весили по нескольку мегабайт и прикладывать. Надо уважать правила оформления. 27 апр 2017 в 6:46
  • Так не понятно. Нужно анализировать дамп памяти и смотреть код полностью. Попробуйте в студии сделать все действия приводящие к утечке, потом вызвать GC, а потом сразу дамп возьмите и посмотрите то там в дампе 27 апр 2017 в 8:22

2 ответа 2

1

Всё просто: нижняя строка говорит что в памяти виснет CurrentSongFragment,
строка выше указывает где создаётся удерживающая его ссылка.
Беглый взгляд по стеку и вот что мы видим.
В классе CurrentBaseFragment Вы вешаете обработчик на кнопку в тулбаре, в коде которого ссылаетесь на инстанс фрагмента (скорее всего неявно - обращение к полям и методам). Но тулбар живёт дольше фрагмента и удерживает его в памяти ссылкой в обработчике кнопки.

1

Судя по цепочке, ссылка sDefaultWindowManager содержит прямые и/или косвенные ссылки на Context. Утечки памяти возникают из-за того, что хранятся ссылки (в данном случае косвенные) на Context в статическом поле.

Организуйте код так, чтобы sDefaultWindowManager не был static, и утечка должна исчезнуть.

3
  • Ссылки sDefaultWindowManager у меня нет
    – iamtihonov
    27 апр 2017 в 7:55
  • А поворот экрана поддерживаете? Можете прислать код, где вешаете клик лисенер в CurrentSongFragment? 27 апр 2017 в 7:59
  • Поворот не поддерживаем. Добавил обработки нажатия.
    – iamtihonov
    27 апр 2017 в 8:14

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.