10

Что использую?

Пишу Android приложение с использованием таких библиотек:

  • Мoxy (Для реализации архитектуры MVP)
  • RxJava (Для удобной работы с данными)
  • Socket.io (Конкретно моя реализация RxSocket под RxJava)
  • Ну и многие другие (ButterKnife и для тестов)

Что делаю?

У меня есть UI: Fragments и Activities и для каждого реализован Presenter

View не показываю, ибо не поместится на фотографии

Зайдем в Presenter (AuthActivity), эта активность запускается в первую очередь, значит и её Presenter тоже.

Здесь у нас вот такой код:

@InjectViewState
public class AuthPresenter extends MvpPresenter<AuthView> 
{
    public RxSocket socket;

    private AuthView view;

    public AuthPresenter() 
    {
        // Синглтон, код синглтона будет ниже
        socket = SingletonSocket.getInstance(); 

        view = getViewState();
    }

    public void authServer(){
        socket.connect()
                .subscribe(() -> {
                    listenForErrors();
                    view.showAuthFragment();
                }, throwable -> {
                    view.showErrorFragment(null);
                    throwable.printStackTrace();

                });
    }

    private void listenForErrors(){
        socket.on(Socket.EVENT_ERROR)
                .observeOn(Schedulers.io())
                .subscribeOn(AndroidSchedulers.mainThread())
                .subscribe(objects -> {
                    view.showError("Ошибка подключения");
                    view.showErrorFragment(null);
                });
    }

    @Override
    public void onDestroy() {
        super.onDestroy();        
        socket.off(Socket.EVENT_ERROR);          
    }
}

RxSocket socket идет инициализация через синглтон:

public final class SingletonSocket 
{

    private static RxSocket socket;

    private SingletonSocket(){}

    public static RxSocket getInstance(){
        if(socket == null){
            IO.Options options = new IO.Options();
            options.transports = new String[]{"websocket"};
            options.reconnection = false;
            options.forceNew = false;
            socket = new RxSocket(Constants.URL, options);
        }
        return socket;
    }
}

Он нужен для одноразового коннекта.

Далее этот синглтон через конструктор Presenter каждого Fragment, чтобы потом из фрагментов получать нужные данные с сервера.

В чём проблема? №1

Я заметил, что когда я перехожу на вторую активность (при этом уничтожаю старую), и далее нажимаю физическую кнопку назад, то у меня все закрывается (Что правильно), но при повторном открытии из недавних приложений открывается первая активность. И при этом фрагмент уже не показывается (видимо синглтон выжил) Правильно ли это?

Это еще не все, с каждой сменой фрагмента память поднимается (методом replace)

FragmentTransaction transaction = getActivity().getFragmentManager().beginTransaction();
        RegisterFirstFragment fragment = new RegisterFirstFragment(); //Здесь класс фрагмента, в который я хочу перейти
        transaction.setCustomAnimations(R.animator.fragment_slide_left_anim, R.animator.fragment_slide_right_anim);
        transaction.replace(R.id.fragment_container, fragment);
        transaction.remove(this); //Здесь я пытался уничтожить фрагмент
        transaction.commitAllowingStateLoss();

17мб -> 18мб -> 19мб -> 20мб и GG не собирает старые не нужные фрагменты.

В чём проблема? №2

Также у меня в каждом фрагменте есть SnackBar, который вызывается в случае неверных данный введенных пользователем и с каждым его вызовом у меня память увеличивается и не очищается. Не понимаю как в таком случае освобождать память?

Так в чем же дело?

Вопрос такой - является ли это повышение памяти нормальным, если нет, как в моем случае это можно исправить?


Общие вопросы

Статических переменных, которые хранят контекст у меня нет ни в одном классе. Может быть проблема в синглтоне? Как после нажатия кнопки назад на последней Activity уничтожить и синглтон?

Заранее спасибо, за все предложенные решения и рекомендации(Например, если я что-то делаю не так, или нужен, также например, другой паттерн и т.д.).

9
  • 1
    Попробуйте github.com/square/leakcanary
    – post_zeew
    13 дек 2017 в 20:14
  • @post_zeew Она у меня подключена, но ведь библиотека не поможет исправить утечку
    – Ker_ Jen
    14 дек 2017 в 7:13
  • Она поможет ее найти.
    – post_zeew
    14 дек 2017 в 10:38
  • То есть если она не говорит, значит утечки нет?
    – Ker_ Jen
    14 дек 2017 в 10:42
  • Про LeakCanary правильно подмечено, и если вы ее подключили как надо, то скорее всего утечек нет. 14 дек 2017 в 13:01

1 ответ 1

1

Думаю надо отписывать от подписки в onDestroy

subscription = socket.connect()
                .subscribe(...)

....

@Override
public void onDestroy() {
    super.onDestroy();        
    socket.off(Socket.EVENT_ERROR);

    if (subscription != null) {
      subscription.unsubscribe();
    }      
}

Ваш ответ

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

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