Утечка памяти Android в коде

Question

Утечка памяти Android в коде

1

Я пытаюсь понять концепцию утечек памяти. Я пробовал этот код и пробовал несколько способов найти из соответствующих сообщений, но я не мог решить проблему. Нужна помощь в понимании утечки памяти в этом коде. Мое приложение имеет только 2 действия

//Первая активность

package com.pace.mat;

import android.app.Activity;
import android.app.Dialog;
import android.content.Context;
import android.content.Intent;
import android.os.Bundle;
import android.view.View;
import android.view.Window;
import android.view.View.OnClickListener;
import android.widget.Button;
import android.widget.ImageView;

public class MATDemoActivity extends Activity implements OnClickListener {

    private Dialog dialog1;
    private Button btnSubmit;
    private Context myClassContext;
    private ImageView RedImage,BlueImage,Yellow,Orange,Green;

    /** Called when the activity is first created. */
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);
        myClassContext = this;

        RedImage = (ImageView) findViewById(R.id.Red);
        BlueImage = (ImageView) findViewById(R.id.Blue);
        Yellow = (ImageView) findViewById(R.id.Yellow);
        Orange = (ImageView) findViewById(R.id.Orange);
        Green = (ImageView) findViewById(R.id.Green);

       RedImage.setImageResource(R.drawable.red);
       BlueImage.setImageResource(R.drawable.blue);
       Yellow.setImageResource(R.drawable.yellow);
       Orange.setImageResource(R.drawable.orange);
       Green.setImageResource(R.drawable.green);

        btnSubmit = (Button)findViewById(R.id.btnSubmitAtFirst);
        btnSubmit.setOnClickListener(this);

    }

    public void onClick(View arg0) {
        // TODO Auto-generated method stub
        if(arg0 == (View)btnSubmit)
        {
            dialog1=new Dialog(myClassContext); 
            Window window = dialog1.getWindow(); 
            window.setBackgroundDrawableResource(android.R.color.transparent); 
            window.requestFeature(window.FEATURE_NO_TITLE);                     
            dialog1.setContentView(R.layout.progress_indicator); 
            dialog1.show();     

            // Doing a network intensive task

            if(dialog1 !=null)
            {
                dialog1 = null;
                myClassContext =  null;
                window = null;
            }

            Intent i = new Intent(MATDemoActivity.this,SecondActivity.class);
            startActivity(i);
        }
    }

    @Override
    public void onStop() {    
        super.onStop();  
        myClassContext =  null;
        dialog1 = null;
        RedImage = null;
        BlueImage = null;
        Yellow = null;
        Orange = null;

        Green=null;
        this.finish();   
    }       

    @Override
    public void onPause() {
        super.onPause();
         myClassContext =  null;
            dialog1 = null;
            RedImage = null;
            BlueImage = null;
            Yellow = null;

            Orange = null;
            Green=null;
         this.finish(); 
    }           

    @Override
    public void onDestroy() {
        super.onDestroy();
         myClassContext =  null;
            dialog1 = null;
            RedImage = null;
            BlueImage = null;

            Yellow = null;
            Orange = null;
            Green=null;
        this.finish();
    }

}

//Вторая активность

package com.pace.mat;

import android.app.Activity;
import android.app.Dialog;
import android.content.Context;
import android.content.Intent;
import android.os.Bundle;
import android.view.View;
import android.view.Window;
import android.view.View.OnClickListener;
import android.widget.Button;
import android.widget.ImageView;

public class SecondActivity extends Activity implements OnClickListener {

    private Dialog dialog1;
    private Button btnSubmit;
    private Context myClassContext1;
    private ImageView RedImage,BlueImage,Yellow,Orange,Green;

     /** Called when the activity is first created. */
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.secondactivity);

        myClassContext1 = this;

        RedImage = (ImageView) findViewById(R.id.Red);
        BlueImage = (ImageView) findViewById(R.id.Blue);
        Yellow = (ImageView) findViewById(R.id.Yellow);
        Orange = (ImageView) findViewById(R.id.Orange);
        Green = (ImageView) findViewById(R.id.Green);

       RedImage.setImageResource(R.drawable.red);
       BlueImage.setImageResource(R.drawable.blue);
       Yellow.setImageResource(R.drawable.yellow);
       Orange.setImageResource(R.drawable.orange);
       Green.setImageResource(R.drawable.green);

        btnSubmit = (Button)findViewById(R.id.btnSubmitAtFirst);
        btnSubmit.setOnClickListener(this);
    }

    public void onClick(View v) {
        // TODO Auto-generated method stub
        if(v == (View)btnSubmit)
        {
            dialog1=new Dialog(myClassContext1); 
            Window window = dialog1.getWindow(); 
            window.setBackgroundDrawableResource(android.R.color.transparent); 
            window.requestFeature(window.FEATURE_NO_TITLE);                     
            dialog1.setContentView(R.layout.progress_indicator); 
            dialog1.show();     

            // Uploading an Image to network 

            if(dialog1 !=null)
            {
                dialog1 = null;
                myClassContext1 =  null;
                window = null;
            }

            Intent i = new Intent(this,MATDemoActivity.class);
            startActivity(i);
        }
    }

    @Override
    public void onStop() {    
        super.onStop();  

        this.finish();   
    }       

    @Override
    public void onPause() {
        super.onPause();

         this.finish(); 
    }           

    @Override
    public void onDestroy() {
        super.onDestroy();
         myClassContext1 =  null;
            dialog1 = null;
            RedImage = null;
            BlueImage = null;

            Yellow = null;
            Orange = null;
            Green=null;
        this.finish();
    }

}

//LOG CAT DATA, КОГДА Я ПЕРВУЮ ПЕРВЫЙ К ВТОРОЙ ДЕЯТЕЛЬНОСТИ

05-17 12:12:43.323: E/WindowManager(2264): Activity com.pace.mat.SecondActivity has leaked window com.android.internal.policy.impl.PhoneWindow$DecorView@44f63b88 that was originally added here
05-17 12:12:43.323: E/WindowManager(2264): android.view.WindowLeaked: Activity com.pace.mat.SecondActivity has leaked window com.android.internal.policy.impl.PhoneWindow$DecorView@44f63b88 that was originally added here
05-17 12:12:43.323: E/WindowManager(2264):  at android.view.ViewRoot.<init>(ViewRoot.java:247)
05-17 12:12:43.323: E/WindowManager(2264):  at android.view.WindowManagerImpl.addView(WindowManagerImpl.java:148)
05-17 12:12:43.323: E/WindowManager(2264):  at android.view.WindowManagerImpl.addView(WindowManagerImpl.java:91)
05-17 12:12:43.323: E/WindowManager(2264):  at android.view.Window$LocalWindowManager.addView(Window.java:424)
05-17 12:12:43.323: E/WindowManager(2264):  at android.app.Dialog.show(Dialog.java:241)
05-17 12:12:43.323: E/WindowManager(2264):  at com.pace.mat.SecondActivity.onClick(SecondActivity.java:54)
05-17 12:12:43.323: E/WindowManager(2264):  at android.view.View.performClick(View.java:2408)
05-17 12:12:43.323: E/WindowManager(2264):  at android.view.View$PerformClick.run(View.java:8816)
05-17 12:12:43.323: E/WindowManager(2264):  at android.os.Handler.handleCallback(Handler.java:587)
05-17 12:12:43.323: E/WindowManager(2264):  at android.os.Handler.dispatchMessage(Handler.java:92)
05-17 12:12:43.323: E/WindowManager(2264):  at android.os.Looper.loop(Looper.java:123)
05-17 12:12:43.323: E/WindowManager(2264):  at android.app.ActivityThread.main(ActivityThread.java:4627)
05-17 12:12:43.323: E/WindowManager(2264):  at java.lang.reflect.Method.invokeNative(Native Method)
05-17 12:12:43.323: E/WindowManager(2264):  at java.lang.reflect.Method.invoke(Method.java:521)
05-17 12:12:43.323: E/WindowManager(2264):  at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:868)
05-17 12:12:43.323: E/WindowManager(2264):  at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:626)
05-17 12:12:43.323: E/WindowManager(2264):  at dalvik.system.NativeStart.main(Native Method)

Prem 17 май 2012, в 09:34

Источник

0

не похоже, что в этом коде есть утечка памяти, возможно, в какой-то другой части кода есть утечка памяти. также вам не нужно вызывать null для ваших объектов в onStop() , onPause() . Это приведет к задержке, когда приложение возобновит работу.
Mayank 17 май 2012, в 06:39
0

Благодарю. Да, я попробовал это только на метод ondestroy первоначально, но даже после этого я получил утечку памяти. Так что добавили и другие методы тоже. я выложу полный код сейчас.
Prem 17 май 2012, в 06:40
0

Пример: 'RedImage = (ImageView) findViewById (R.id.Red); `я вижу, что вы создаете одинаковые объекты изображения в обоих действиях.
Mayank 17 май 2012, в 07:27

Показать ещё 1 комментарий

Теги:

java

android

memory-leaks

memory

3 ответа

0

Из журнала я считаю, что причиной проблемы является то, что вы показываете диалог, но не отклоняете его, тогда вы назначаете диалог null и начинаете другое действие.

if(dialog1 !=null)
{
    dialog1 = null;
    myClassContext1 =  null;
    window = null;
}

Таким образом, активность протекает в диалоговом окне (диалоговое окно - это окно). В любом случае, если вам не нужен диалог, вы должны отклонить его.

dreamtale 17 май 2012, в 06:23

0

Похоже, вы вызываете одну активность из другой, повторяете ее, как будто вы зацикливаетесь на действия и переключаете их.

MATDemoActivity вызывает

Intent i = new Intent(MATDemoActivity.this,SecondActivity.class); 
            startActivity(i);

а затем из SecondActivity вы вызываете

Intent i = new Intent(this,MATDemoActivity.class);   
        startActivity(i);

также вы воссоздаете все объекты в каждом действии, а не делите их между двумя действиями.

попытайтесь устранить эти проблемы и посмотрите, работает ли это.

Mayank 17 май 2012, в 05:42

0

«Вы воссоздаете все объекты в каждом упражнении» - как вы это скажете? Можете ли вы предоставить более глубокое понимание?
Prem 17 май 2012, в 07:24

Ещё вопросы

не похоже, что в этом коде есть утечка памяти, возможно, в какой-то другой части кода есть утечка памяти. также вам не нужно вызывать null для ваших объектов в onStop() , onPause() . Это приведет к задержке, когда приложение возобновит работу.
Благодарю. Да, я попробовал это только на метод ondestroy первоначально, но даже после этого я получил утечку памяти. Так что добавили и другие методы тоже. я выложу полный код сейчас.
Пример: 'RedImage = (ImageView) findViewById (R.id.Red); `я вижу, что вы создаете одинаковые объекты изображения в обоих действиях.
«Вы воссоздаете все объекты в каждом упражнении» - как вы это скажете? Можете ли вы предоставить более глубокое понимание?

N-JOY · Accepted Answer · 2012-05-17T05-39-00.000Z

ваш код, похоже, не имеет утечки памяти. и большую часть времени утечки памяти связаны с сохранением долговечной ссылки на Context.

Вот хорошая статья Ромена Гая, которая поможет вам понять утечки памяти в деталях. проверить.

Надеюсь, это поможет.

РЕДАКТИРОВАТЬ: согласно вашему обновленному вопросу

Посмотрев на ваш журнал, кажется, что утечки памяти нет. закрыть или закрыть диалоговое окно, прежде чем завершить свою деятельность. это решит вашу проблему.

Пожалуйста, обратитесь к моим данным журнала, где я вижу утечку памяти. Кроме того, у меня нет статической ссылки в моем файле класса, которая может содержать постоянную ссылку
это не утечка памяти, это окно утечка исключений. закрыть или закрыть свой диалог, прежде чем вы закончите свою деятельность. это решит вашу проблему.
Привет спасибо После того, как я закрыл диалоговое окно, я не получил исключение в log cat.