2

А главное как(сделать это правильно)? Здесь предположение о отсутствии утечки из-за того, что контекст приложения тоже вроде как один. Контекст мне нужен в других методах Presenter-a, но студия ругается, введите сюда описание изображения что может быть утечка памяти, но(!) я то знаю что экземпляров application context, которые принимаются от класса MyApplication,- всего один, так что можно игнорировать предупреждение? или я вообще ничего не понимаю

Singletone-класс, где нужен контекст приложения

public class MyPresenter implements Contract.ContractPresenter{
    private Context mContext;

    private static MyPresenter presenterInstance;
    private Contract.ContractView mViewInstance;

    private MyPresenter(Context context){
        mContext = context;
    }

    public static synchronized void initPresenterInstance(Context context) {
        if (presenterInstance == null) {
            presenterInstance = new MyPresenter(context);
        }
    }
    public static MyPresenter getPresenterInstance() {
        return presenterInstance;
    }
}

и Application-класс

public class MyApplication extends Application {

@Override
public void onCreate() {
        super.onCreate();
        MyPresenter.initPresenterInstance(getApplicationContext());
    }

}
9
  • 2
    правильно сделать: передавать контекст в нужные функции, и не хранить его в статике
    – Jarvis_J
    30 июн 2018 в 16:02
  • @Jarvis_J , насколько я понял, статические контексты все таки могут вызывать утечки памяти, а чтоб сделать контекст доступным в другом классе, его в MyApplication нужно будет объявлять статическим 30 июн 2018 в 16:44
  • Тоесть как его вообще можно получить в не activity-классе 30 июн 2018 в 17:13
  • Блин, ну правильно жеш! в этом ответе написано, что с application context можно делать вообще что угодно, stackoverflow.com/questions/5498669/… , я правильно понимаю? 30 июн 2018 в 17:17
  • А почему Вы считаете, что приведенный выше код есет в себе Singletone паттерн? У Вас публичный конструктор. И зачем Вы сделали переменную, несущую объект presenter'а статичной? Вам бы стоило написать get метод под это поле.
    – Kostya M
    30 июн 2018 в 17:29

1 ответ 1

1

В ходе длительных рассуждения, я решил, что я знаю ответ. Есть два выхода из сложившейся ситуации.
1)Использовать Dagger 2, как в этом примере.
2) НЕ ДЕЛАТЬ ТАК, а если быть точнее, то выполнять всю работу Dagger'а ручками. А именно, если требуется работа с Context или подобными классами, создавать дополнительный класс (Не Singleton). Сам Singleton же должен держать в себе ссылку на этот класс. Вырисовывается вот такая картина:

Сам Singleton

public class Singleton {
private static final Singleton ourInstance = new Singleton();
private static SomeService ss;
public static Singleton getInstance(Context context) {
    if(ss==null){
        ss=new SomeService(context);
    }
    return ourInstance;
}

private Singleton() {
}

public SomeService getSs() {
    return ss;
}

}

Сервис, работающий с Context, ссылка на объект которого хранится в Singleton

public class SomeService {
    private int result=0;
    SomeService(Context context){
        result=doSomeBusyJob(context);
    }
    int doSomeBusyJob(Context context){
        return 0;
    }

    public int getResult() {
        return result;
    }
}

Ну и затем уже делать как-то вот так

Singleton.getInstance(applicationContext).getSs().getResult()

В таком случае, мы убиваем двух зайцев: не допускаем случайного размножения класса SomeService, впринципе, для чего и нужен Singleton паттерн, а во-вторых, не храним в глобальном окружении синглтона такие классы, как Context, надеюсь, Вам это поможет решить Вашу задачу

9
  • Вполне даже элегантно! Но вот если понадобится расположить в SomeService два разных метода doSomeBusyJob1 и 2, (оба работают с Context) , то нужно либо делать приватную переменную контекста в классе, и при инстансе обьекта класса инициализировать контекстную переменную, или делать так Singleton.getInstance(applicationContext).getSs().getResult2(applicationContext)) 1 июл 2018 в 16:07
  • В singletone SomeService-a может жеш храниться контекст, принятый из конструктора, это ж синглтон, ничего не будет(?) 1 июл 2018 в 16:48
  • Тоесть я понимаю, что моделей может быть больше, если больше всяких задач, тип doGetModel,doPostModel, к примеру, но это можно ведь и абстрагировать и в этом классе содержать нужные методы. Хотя тут на помощь нам может придти наследование и есть одна идея... 1 июл 2018 в 17:46
  • идею на потом, а я вот подумал, - методы типа getResult вызываются в тех же местах, что и инстанс синглтонов и прочьего(не всегда,нужно обдумать этот момент),так почему бы не инстанциировать их без каких либо контекстов, а вот их методы вызывать передавая в них контекст. Вот клик на кнопку "скачать" в MainActivity, - из onclicklistener-a вызываем метод презентера runDownload(this.context), (передали ему контекст) , runDownload вызывает метод Model.download(context) передав ему переданный ему (полученный им(методом runDownload)) контекст. Что скажете? no memory leaks, no архитектурных проблем? 1 июл 2018 в 20:53
  • Так делать, конечно, можо и может являться вариантом, но только в случае, когда не нужен целый огромный объект с кучей методов работающих с Context'ом, ибо это очень энергозатратно.
    – Kostya M
    2 июл 2018 в 7:05

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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