ProgressDialog: как предотвратить утечку окна

Question

ProgressDialog: как предотвратить утечку окна

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

все работало нормально, пока моему клиенту не удалось произвести эту ошибку:

"07-06 17:10:50.363: ERROR/WindowManager(8821): Activity android.pixelrain.framework.PixelRainActivity has leaked window com.android.internal.policy.impl.PhoneWindow$DecorView@463f3e50 that was originally added here
07-06 17:10:50.363: ERROR/WindowManager(8821): android.view.WindowLeaked: Activity android.pixelrain.framework.PixelRainActivity has leaked window com.android.internal.policy.impl.PhoneWindow$DecorView@463f3e50 that was originally added here
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.view.ViewRoot.<init>(ViewRoot.java:251)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.view.WindowManagerImpl.addView(WindowManagerImpl.java:148)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.view.WindowManagerImpl.addView(WindowManagerImpl.java:91)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.view.Window$LocalWindowManager.addView(Window.java:424)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.app.Dialog.show(Dialog.java:241)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.app.ProgressDialog.show(ProgressDialog.java:107)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.app.ProgressDialog.show(ProgressDialog.java:90)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.pixelrain.HTTPHelper.DraftHelper.getDraft(DraftHelper.java:70)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.pixelrain.online.OnlineRetriver.getDraft(OnlineRetriver.java:312)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.pixelrain.HTTPHelper.DraftButtonGL.loadDraft(DraftButtonGL.java:72)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.pixelrain.HTTPHelper.DraftButtonGL.isTouched(DraftButtonGL.java:89)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.pixelrain.opengl.views.game.QuickStartGL.touchEnded(QuickStartGL.java:160)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.pixelrain.game.GameHandler.onTouchEvent(GameHandler.java:277)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.pixelrain.opengl.GLSurfaceViewChipmunk.onTouchEvent(GLSurfaceViewChipmunk.java:27)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.view.View.dispatchTouchEvent(View.java:3765)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:944)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:944)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:944)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at com.android.internal.policy.impl.PhoneWindow$DecorView.superDispatchTouchEvent(PhoneWindow.java:1701)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at com.android.internal.policy.impl.PhoneWindow.superDispatchTouchEvent(PhoneWindow.java:1116)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.app.Activity.dispatchTouchEvent(Activity.java:2093)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at com.android.internal.policy.impl.PhoneWindow$DecorView.dispatchTouchEvent(PhoneWindow.java:1685)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.view.ViewRoot.handleMessage(ViewRoot.java:1802)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.os.Handler.dispatchMessage(Handler.java:99)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.os.Looper.loop(Looper.java:144)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at android.app.ActivityThread.main(ActivityThread.java:4937)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at java.lang.reflect.Method.invokeNative(Native Method)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at java.lang.reflect.Method.invoke(Method.java:521)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:868)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:626)
07-06 17:10:50.363: ERROR/WindowManager(8821):     at dalvik.system.NativeStart.main(Native Method)"

и я понятия не имею как это исправить.

есть идеи, что вызывает это и как его решить ?

журнал отслеживает ошибку до этой строки:

    progressDialog = ProgressDialog.show(PixelRainActivity.staticThis, "",PixelRainActivity.staticThis.getResources().getString( R.string.draftProgressMessage), true);

решило бы это проблему, если бы я изменил ее на это :

this.runOnUiThread(new Runnable() {
            public void run() {
                progressDialog = ProgressDialog.show(PixelRainActivity.staticThis, "",PixelRainActivity.staticThis.getResources().getString( R.string.draftProgressMessage), true);
            }
        });

42

android memory-leaks progressdialog

автор: Jason Rogers

9 ответов

автор: Guillaume Brunerie · Accepted Answer · 2011-07-13 00:31:04

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

самый простой способ исправить это использоватьContext вместо. Измените свой staticThis = this строка в методе onCreate() to staticThis = this.getApplicationContext() и он должен работать (и изменение типа staticThis to Context если это еще не так)

автор: Jimmy Collazos · Accepted Answer · 2011-12-22 11:47:11

использование:

progressDialog.dismiss();

в конце работы

109

автор: Jimmy Collazos

автор: Diaconu Nicu · Accepted Answer · 2017-02-22 22:33:32

есть ситуации, когда вы должны проверить в onDetach или onDestroy если диалог выполнения все еще виден. Вот так:

@Override
public void onDetach() {
    if (mProgressDialog != null && mProgressDialog.isShowing())
        mProgressDialog.dismiss();
    super.onDetach();
}

автор: Melinda Green · Accepted Answer · 2011-12-01 03:29:52

cygnus имеет хорошую идею использовать showDialog (MY_INT), где MY_INT-это просто некоторое постоянное значение, которое вы выбираете, чтобы отличить его от любых других подобных диалогов, которые вы запускаете таким образом. Вы берете его таким же образом с dismissDialog(MY_INT). Просто не запускайте его из своего метода onPause. Вы можете сделать это вместо метода onResume действия, которое собирается выполнить пользователь. Затем вы переопределяете метод onCreateDialog этого действия следующим образом:

@Override
protected Dialog onCreateDialog(int id) {
    if(id == MY_INT) {
        ProgressDialog progressDialog = new ProgressDialog(this);
        progressDialog.setMessage("Your message string");
        return progressDialog;
    }
    return super.onCreateDialog(id);
}

автор: cyngus · Accepted Answer · 2017-01-18 12:20:36

вместо ProgressDialog.show(), попробуйте использовать

Activity.showDialog() который должен автоматически управлять Dialog для вас и предотвращения утечек.

EDIT: когда вы вызываете showDialog(int), это вызовет Activity.onCreateDialog(int) где вы можете создать Dialog вы хотите, и вернуть Dialog вы хотите отобразить.

автор: Andy Dyer · Accepted Answer · 2014-12-16 11:26:42

Я столкнулся с аналогичной проблемой с диалогом и фоновой задачи. AsyncTask (http://android-developers.blogspot.de/2009/05/painless-threading.html) позволил мне сделать оба гораздо более чисто и без протекающего окна.

автор: Xar E Ahmer · Accepted Answer · 2016-05-02 06:14:33

лучше использовать AsyncTask чтобы получить что-то из интернета в фоновом режиме. И нет необходимости передавать статический контекст. И активность

new YourAsyncTask(context).execute();

вызов AsyncTask, как указано выше

private class YourAsynTask extends AsyncTask<String,Void,String>
{
 private Context context;
 private ProgressDialog progressDialog;

 //pass context in constructor
  public YourAsynTask(Context context)
  {
     this.context = context;
  }

  //show progress in onPre 
  @Override
  protected void onPreExecute()
  {
    //show Progress code here.
    progressDialog = ProgressDialog.show(context, "", "Loading. Please wait...", true);
  }

  //dismiss Progress dialog in onPost
  @Override 
  protected void OnPostExecute(String response)
  {
    if(progressDialog!=null)
     progressDialog.dismiss();
     progressDialog = null;
  }
}

автор: Mahesh · Accepted Answer · 2017-01-18 13:31:37

если вы используете thread или AsyncTask и загрузка материала из интернета и показ progress bar , вы должны использовать DialogFragment или отменить , когда Activity get stop также, если вы показываете прогресс в Asynctask кулак отменить Asynctask и переопределить oncancel метод обратного вызова и dismiss есть диалог прогресса.

Window leak на Activity или fragment на самом деле происходит из-за того, что вы пытаетесь добавить окно, и пока оно появляется, оно находится на foreground,но когда вы нажимаете на дом, он останавливается, а затем останавливается через onStop(). Так ваш CustomView остается прикрепленным к окну, которое теперь исчезло.Следовательно, в соответствии с системой ваш customView занимал пространство, которое он не освобождал.

автор: Zeeshan Khan · Accepted Answer · 2017-07-28 17:13:04

позвонить progressDialog.dismiss() прежде чем активность будет убита. Я свою починил вот так.