คำเตือน: อย่าวางคลาสบริบทของ 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 () บนบริบทใด ๆ ที่ส่งผ่าน method / constructor ไปยัง singleton ของคุณหากคุณตัดสินใจที่จะจัดเก็บไว้ในฟิลด์สมาชิกใด ๆ

ตัวอย่างการพิสูจน์คนงี่เง่า (แม้ว่าใครบางคนจะผ่านไปในกิจกรรมก็จะคว้าบริบทของแอปและใช้สิ่งนั้นเพื่อสร้างอินสแตนซ์ซิงเกิลตัน)

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

getApplicationContext () ตามเอกสาร: "ส่งคืนบริบทของอ็อบเจ็กต์แอ็พพลิเคชันส่วนกลางเดียวของกระบวนการปัจจุบัน"

หมายความว่าบริบทที่ส่งคืนโดย "getApplicationContext ()" จะใช้งานได้ตลอดทั้งกระบวนการดังนั้นจึงไม่สำคัญว่าคุณจะจัดเก็บการอ้างอิงแบบคงที่ไว้ที่ใดก็ตามเนื่องจากจะอยู่ที่นั่นเสมอในระหว่างรันไทม์ของแอปของคุณ (และอยู่ได้นานกว่าวัตถุใด ๆ / singletons สร้างอินสแตนซ์โดยมัน)

เปรียบเทียบกับบริบทภายในของมุมมอง / กิจกรรมที่มีข้อมูลจำนวนมากหากคุณรั่วไหลบริบทที่จัดขึ้นโดยกิจกรรมระบบจะไม่สามารถปลดปล่อยทรัพยากรนั้นซึ่งเห็นได้ชัดว่าไม่ดี

การอ้างอิงถึงกิจกรรมตามบริบทควรใช้วงจรชีวิตเช่นเดียวกับกิจกรรมนั้น ๆ มิฉะนั้นจะจับบริบทเป็นตัวประกันทำให้หน่วยความจำรั่วไหล (ซึ่งเป็นเหตุผลเบื้องหลังคำเตือนผ้าสำลี)

แก้ไข:สำหรับผู้ชายที่ทุบตีตัวอย่างจากเอกสารด้านบนยังมีส่วนความคิดเห็นในโค้ดเกี่ยวกับสิ่งที่ฉันเพิ่งเขียนเกี่ยวกับ:

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

เป็นแค่คำเตือน ไม่ต้องกังวล. หากคุณต้องการใช้บริบทของแอปพลิเคชันคุณสามารถบันทึกในคลาส "singleton" ซึ่งใช้เพื่อบันทึกคลาสซิงเกิลตันทั้งหมดในโครงการของคุณ

ในกรณีของคุณมันไม่สมเหตุสมผลเท่าไหร่ที่จะให้มันเป็นฟิลด์คงที่ แต่ฉันไม่คิดว่ามันจะแย่ในทุกกรณี หากตอนนี้คุณกำลังทำอะไรอยู่คุณสามารถมีฟิลด์แบบคงที่ที่มีบริบทและเป็นโมฆะได้ในภายหลัง ฉันกำลังสร้างอินสแตนซ์แบบคงที่สำหรับคลาสโมเดลหลักของฉันที่มีบริบทอยู่ภายในบริบทของแอ็พพลิเคชันไม่ใช่บริบทกิจกรรมและฉันมีฟิลด์อินสแตนซ์แบบคงที่ของคลาสที่มีกิจกรรมที่ฉันว่างในการทำลาย ฉันไม่เห็นว่าฉันมีความจำรั่ว ดังนั้นถ้าผู้ชายฉลาด ๆ คิดว่าฉันผิดอย่าลังเลที่จะแสดงความคิดเห็น ...

การเรียกใช้ทันทียังทำงานได้ดีที่นี่ ...

ใช้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;
}

ตอนนี้คุณสามารถเข้าถึงบริบทเช่น

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

โดยทั่วไปหลีกเลี่ยงการกำหนดฟิลด์บริบทเป็นแบบคงที่ คำเตือนนั้นอธิบายว่าเหตุใด: มันเป็นความทรงจำรั่ว การทำลายทันทีอาจไม่ใช่ปัญหาใหญ่ที่สุดในโลก

ตอนนี้มีสองสถานการณ์ที่คุณจะได้รับคำเตือนนี้ สำหรับตัวอย่าง (สิ่งที่ชัดเจนที่สุด):

public static Context ctx;

จากนั้นก็มีสิ่งที่ยุ่งยากกว่าเล็กน้อยซึ่งบริบทถูกรวมไว้ในชั้นเรียน:

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

และคลาสนั้นถูกกำหนดให้เป็นแบบคงที่:

public static Example example;

และคุณจะได้รับคำเตือน

วิธีแก้ปัญหานั้นค่อนข้างง่าย: อย่าวางฟิลด์บริบทในอินสแตนซ์แบบคงที่ไม่ว่าจะเป็นคลาสการตัดหรือประกาศแบบคงที่โดยตรง

และวิธีแก้ไขคำเตือนนั้นง่ายมาก: อย่าวางฟิลด์ไว้นิ่ง ๆ ในกรณีของคุณส่งบริบทเป็นอินสแตนซ์ไปยังเมธอด สำหรับชั้นเรียนที่มีการเรียกบริบทหลายรายการให้ใช้ตัวสร้างเพื่อส่งผ่านบริบท (หรือกิจกรรมสำหรับเรื่องนั้น) ไปยังชั้นเรียน

_{โปรดทราบว่าเป็นการเตือนไม่ใช่ข้อผิดพลาด หากคุณต้องการบริบทคงที่ด้วยเหตุผลใดก็ตามคุณสามารถทำได้ แม้ว่าคุณจะสร้างความทรงจำรั่วไหลเมื่อคุณทำเช่นนั้น}

หากคุณแน่ใจว่าเป็น Application Context มันสำคัญที่สุด เพิ่มสิ่งนี้

@SuppressLint("StaticFieldLeak")