चेतावनी: स्थिर क्षेत्रों में Android संदर्भ कक्षाएं न लगाएं; यह एक मेमोरी लीक है (और इंस्टेंट रन भी तोड़ता है)

Question 1

Android स्टूडियो:

एंड्रॉइड संदर्भ कक्षाओं को स्थिर क्षेत्रों में न रखें; यह एक मेमोरी लीक है (और इंस्टेंट रन भी तोड़ता है)

तो 2 प्रश्न:

# 1 आप startServiceसंदर्भ के लिए एक स्थिर चर के बिना एक स्थिर विधि से कैसे कॉल करते हैं ?
# 2 आप एक स्थैतिक विधि (उसी) से एक लोकलब्रॉडकास्ट कैसे भेजते हैं?

उदाहरण:

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?

नोट: मुझे लगता है कि मेरा मुख्य प्रश्न उस वर्ग के संदर्भ में कैसे हो सकता है जिसमें से कॉलिंग विधि रहती है।

Question 2

बस इसे अपनी विधि के पैरामीटर के रूप में पास करें। एक 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);
    }
}

प्रश्न पर टिप्पणियों से अपडेट करें: आरंभिक गतिविधि (निर्माण मापदंडों या विधि मापदंडों के माध्यम से) के संदर्भ में उस बिंदु तक पहुंचें, जहां तक इसकी आवश्यकता है।

Question 3

यदि आप किसी भी सदस्य क्षेत्र में इसे संग्रहीत करने का निर्णय लेते हैं, तो बस यह सुनिश्चित करें कि आप संदर्भ को भूल जाते हैं। आप किसी भी संदर्भ में संदर्भ /getApplicationContext () या कॉल getApplicationContext () को पास करें।

इडियट प्रूफ उदाहरण (भले ही कोई व्यक्ति किसी गतिविधि में पास हो जाए, यह ऐप के संदर्भ को ले लेगा और सिंगलटन को इंस्टेंट करने के लिए उपयोग करेगा):

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

डॉक्स के अनुसार getApplicationContext (): "वर्तमान प्रक्रिया के एकल, वैश्विक अनुप्रयोग ऑब्जेक्ट का संदर्भ लौटाएं।"

इसका अर्थ है कि "getApplicationContext ()" द्वारा लौटाया गया संदर्भ पूरी प्रक्रिया से गुजरेगा और इस प्रकार इससे कोई फर्क नहीं पड़ता कि आप कहीं भी इसका स्थैतिक संदर्भ संग्रहीत करते हैं क्योंकि यह आपके ऐप के रनटाइम के दौरान हमेशा रहेगा (और किसी भी ऑब्जेक्ट को आउटिवेट करता है। / सिंगलटैलन इसके द्वारा त्वरित)।

तुलना करें कि बड़ी मात्रा में डेटा रखने वाले विचारों / गतिविधियों के संदर्भ में, यदि आप किसी गतिविधि द्वारा आयोजित संदर्भ को लीक करते हैं, तो सिस्टम उस संसाधन को मुक्त नहीं कर पाएगा, जो स्पष्ट रूप से अच्छा नहीं है।

एक गतिविधि के संदर्भ में उसके संदर्भ में ही गतिविधि के रूप में एक ही जीवन चक्र रहना चाहिए, अन्यथा यह एक स्मृति रिसाव के कारण संदर्भ बंधक को धारण करेगा (जो लिंट चेतावनी के पीछे का कारण है)।

संपादित करें: ऊपर दिए गए डॉक्स से उदाहरण को कोसने वाले व्यक्ति के लिए, कोड में एक टिप्पणी अनुभाग भी है जिसके बारे में मैंने अभी लिखा है:

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

Question 4

यह सिर्फ एक चेतावनी है। चिंता मत करो। यदि आप एक एप्लिकेशन संदर्भ का उपयोग करना चाहते हैं, तो आप इसे "सिंगलटन" श्रेणी में सहेज सकते हैं, जिसका उपयोग आपकी परियोजना में सभी सिंगलटन वर्ग को बचाने के लिए किया जाता है।

Question 5

आपके मामले में इसे स्थिर क्षेत्र के रूप में रखने का कोई मतलब नहीं है, लेकिन मुझे नहीं लगता कि यह सभी मामलों में बुरा है। यदि आप अब क्या कर रहे हैं, तो आपके पास स्थैतिक क्षेत्र हो सकता है जो संदर्भ और बाद में इसे शून्य कर सकता है। मैं अपने मुख्य मॉडल वर्ग के लिए स्थिर उदाहरण बना रहा हूं, जिसका संदर्भ अंदर है, इसका अनुप्रयोग संदर्भ गतिविधि संदर्भ नहीं है और साथ ही मेरे पास गतिविधि वाले वर्ग का स्थिर उदाहरण क्षेत्र है जिसे मैं नष्ट कर देता हूं। मुझे नहीं लगता कि मेरे पास मेमोरी लीक है। तो अगर कुछ चालाक आदमी मुझे लगता है कि मैं गलत टिप्पणी करने के लिए स्वतंत्र महसूस कर रहा हूँ ...

साथ ही इंस्टेंट रन यहां ठीक काम ...

Question 6

आमतौर पर, संदर्भ क्षेत्रों को स्थिर के रूप में परिभाषित करने से बचें। चेतावनी ही बताती है कि क्यों: यह एक स्मृति रिसाव है। हालांकि तत्काल रन को तोड़ना ग्रह पर सबसे बड़ी समस्या नहीं हो सकती है ।

अब, दो परिदृश्य हैं जहाँ आपको यह चेतावनी मिलेगी। एक उदाहरण के लिए (सबसे स्पष्ट एक):

public static Context ctx;

और फिर थोड़ा और मुश्किल है, जहां संदर्भ एक वर्ग में लिपटे हुए है:

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

और उस वर्ग को कहीं स्थिर के रूप में परिभाषित किया गया है:

public static Example example;

और आपको चेतावनी मिल जाएगी।

इसका समाधान स्वयं काफी सरल है: संदर्भ क्षेत्रों को स्थिर उदाहरणों में न रखें , चाहे वह रैपिंग क्लास हो या सीधे इसे स्थिर घोषित करना।

और चेतावनी का समाधान सरल है: क्षेत्र को सांख्यिकीय रूप से न रखें। आपके मामले में, विधि को उदाहरण के रूप में पास करें। उन कक्षाओं के लिए जहां एकाधिक संदर्भ कॉल किए जाते हैं, कक्षा में संदर्भ (या उस मामले के लिए एक गतिविधि) को पास करने के लिए एक कंस्ट्रक्टर का उपयोग करें।

_{ध्यान दें कि यह एक चेतावनी है, त्रुटि नहीं। क्या आपको किसी भी कारण से स्थैतिक संदर्भ की आवश्यकता है, आप इसे कर सकते हैं। यद्यपि आप ऐसा करने पर स्मृति रिसाव बनाते हैं।}

Question 7

यदि आप सुनिश्चित करते हैं कि यह एक अनुप्रयोग संदर्भ है। यह सबसे महत्वपूर्ण बात है। इसे जोड़ो

@SuppressLint("StaticFieldLeak")

Question 8

`WeakReference`सिंगलटन क्लासेस में कॉन्सेप्ट को स्टोर करने के लिए उपयोग करें और चेतावनी समाप्त हो जाएगी

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 को एक्सेस कर सकते हैं

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

चेतावनी: स्थिर क्षेत्रों में Android संदर्भ कक्षाएं न लगाएं; यह एक मेमोरी लीक है (और इंस्टेंट रन भी तोड़ता है)

WeakReferenceसिंगलटन क्लासेस में कॉन्सेप्ट को स्टोर करने के लिए उपयोग करें और चेतावनी समाप्त हो जाएगी

`WeakReference`सिंगलटन क्लासेस में कॉन्सेप्ट को स्टोर करने के लिए उपयोग करें और चेतावनी समाप्त हो जाएगी