Попередження: Не розміщуйте контекстні класи Android у статичних полях; це витік пам'яті (а також порушує Миттєвий запуск)

Android Studio:

Не розміщуйте контекстні класи Android у статичних полях; це витік пам'яті (і також порушує миттєвий запуск)

Отже, 2 питання:

# 1 Як викликати a startServiceіз статичного методу без статичної змінної для контексту?
# 2 Як відправити localBroadcast із статичного методу (того самого)?

Приклади:

public static void log(int iLogLevel, String sRequest, String sData) {
    if(iLogLevel > 0) {

        Intent intent = new Intent(mContext, LogService.class);
        intent.putExtra("UPDATE_MAIN_ACTIVITY_VIEW", "UPDATE_MAIN_ACTIVITY_VIEW");
        mContext.startService(intent);
    }
}

або

        Intent intent = new Intent(MAIN_ACTIVITY_RECEIVER_INTENT);
        intent.putExtra(MAIN_ACTIVITY_REQUEST_FOR_UPDATE, sRequest));
        intent.putExtra(MAIN_ACTIVITY_DATA_FOR_VIEW, sData);
        intent.putExtra(MAIN_ACTIVITY_LOG_LEVEL, iLogLevel);
        LocalBroadcastManager.getInstance(mContext).sendBroadcast(intent);

Яким буде правильний спосіб зробити це без використання mContext?

ПРИМІТКА: Я думаю, моє головне питання може полягати в тому, як передати контекст класу, з якого живе метод виклику.

Просто передайте його як параметр своєму методу. Немає сенсу створювати статичний екземпляр Contextвиключно з метою запуску Intent.

Ось як повинен виглядати ваш метод:

public static void log(int iLogLevel, String sRequest, String sData, Context ctx) {
    if(iLogLevel > 0) {

        Intent intent = new Intent(ctx, LogService.class);
        intent1.putExtra("UPDATE_MAIN_ACTIVITY_VIEW", "UPDATE_MAIN_ACTIVITY_VIEW");
        ctx.startService(intent);
    }
}

Оновлення з коментарів до питання: Каскадуйте контекст від ініціюючої дії (через параметри конструктора або параметри методу) аж до того моменту, коли вам це потрібно.

Просто переконайтеся, що ви передаєте context.getApplicationContext () або викликаєте getApplicationContext () для будь-якого контексту, який передається за допомогою методів / конструктора в ваш синглтон, якщо ви вирішите зберегти його в будь-якому полі-члені.

Приклад ідіоти (навіть якщо хтось передасть якусь діяльність, вона захопить контекст програми та використає його для створення екземпляра сингтона):

public static synchronized RestClient getInstance(Context context) {
    if (mInstance == null) {
        mInstance = new RestClient(context.getApplicationContext());
    }
    return mInstance;
}

getApplicationContext () згідно з документами: "Повернути контекст єдиного, глобального об'єкта Application поточного процесу."

Це означає, що контекст, повернутий за допомогою "getApplicationContext ()", проживе весь процес, і, отже, неважливо, зберігаєте ви де-небудь статичну посилання на нього, оскільки він завжди буде там під час роботи вашого додатка (і переживе будь-які об'єкти / одиночні, створені ним).

Порівняйте це з контекстом усередині переглядів / заходів, що містять великі обсяги даних, якщо ви випустите контекст, який містить діяльність, система не зможе звільнити той ресурс, який, очевидно, є поганим.

Посилання на діяльність за її контекстом має проходити той самий життєвий цикл, що і сама діяльність, інакше воно буде утримувати контекст у заручниках, що спричиняє витік пам'яті (що є причиною попередження про ворсинки).

РЕДАГУВАТИ: Для хлопця, який розбиває приклад із наведених вище документів, у коді є навіть розділ коментарів про те, про що я щойно писав:

    // getApplicationContext() is key, it keeps you from leaking the
    // Activity or BroadcastReceiver if someone passes one in.

Це лише попередження. Не хвилюйся. Якщо ви хочете використовувати контекст програми, ви можете зберегти його в класі "singleton", який використовується для збереження всього класу singleton у вашому проекті.

У вашому випадку немає сенсу мати його як статичне поле, але я не думаю, що це погано у всіх випадках. Якщо ви зараз робите, що ви робите, ви можете мати статичне поле, яке має контекст, і обнулити його пізніше. Я створюю статичний екземпляр для мого основного класу моделі, який має контекст всередині, його контекст програми не контекст діяльності, а також у мене є поле статичного екземпляра класу, що містить Activity, яке я нулюю при знищенні. Я не бачу, що у мене витік пам'яті. Тож якщо якийсь розумний хлопець вважає, що я помиляюся, сміливо коментуйте ...

Також миттєвий запуск працює тут чудово ...

Як правило, уникайте, щоб поля контексту були визначені як статичні. Саме попередження пояснює, чому: це витік пам'яті. Порушення миттєвого пробігу, можливо, не є найбільшою проблемою на планеті.

Зараз є два сценарії, коли ви отримаєте це попередження. Для прикладу (найбільш очевидного):

public static Context ctx;

А ще є трохи хитріший, де контекст загортається в клас:

public class Example{
    public Context ctx;
    //Constructor omitted for brievety 
}

І цей клас десь визначається як статичний:

public static Example example;

І ви отримаєте попередження.

Саме рішення досить просте: не розміщуйте поля контексту у статичних екземплярах , будь то у класі обгортання або безпосередньо оголошуючи його статичним.

І рішення попередження просте: не розміщуйте поле статично. У вашому випадку передайте контекст як екземпляр методу. Для класів, де здійснюється кілька викликів контексту, використовуйте конструктор, щоб передати контекст (або Activity з цього приводу) класу.

_{Зверніть увагу, що це попередження, а не помилка. Якщо вам з якоїсь причини потрібен статичний контекст, ви можете це зробити. Хоча ви створюєте витік пам'яті, коли ви це робите.}

Якщо ви переконаєтесь, що це Контекст програми. Це має значення. Додайте це

@SuppressLint("StaticFieldLeak")

Використовуйте WeakReferenceдля зберігання контексту в класах Singleton, і попередження зникне

private WeakReference<Context> context;

//Private contructor
private WidgetManager(Context context) {
    this.context = new WeakReference<>(context);
}

//Singleton
public static WidgetManager getInstance(Context context) {
    if (null == widgetManager) {
        widgetManager = new WidgetManager(context);
    }
    return widgetManager;
}

Тепер ви можете отримати доступ до Context like

  if (context.get() instanceof MainActivity) {
            ((MainActivity) context.get()).startActivityForResult(pickIntent, CODE_REQUEST_PICK_APPWIDGET);
        }