2

I want to display simple progress dialog with rotating wheel, while deleting folder from SD card. I have a following piece of code:

  ProgressDialog dialog = ProgressDialog.show(this, "",
            "Please wait for few seconds...", true);

private void deleteCache() {

    File f = new File(Environment.getExternalStorageDirectory()
            .getAbsoluteFile() + Constants.DATA_DIR);
    deleteDirectory(f);
    dialog.dismiss();
}

 private void deleteDirectory(File path) {
        if (path.exists()) {
            File[] files = path.listFiles();
            for (int i = 0; i < files.length; i++) {
                if (files[i].isDirectory()) {
                    deleteDirectory(files[i]);
                } else {
                    files[i].delete();
                }
            }
        }
        return (path.delete());
    }

Which is supposed to show dialog before deleteDirectory(f); and dissmis it after it ends. But I never see any dialog, event though the folder is being deleted.

Andro Selva
  • 53,910
  • 52
  • 193
  • 240
Sver
  • 3,349
  • 6
  • 32
  • 53

3 Answers3

4

This answer is also all over StakcOverflow. Use AsyncTask which will run on a different thread and has three stages... One pre which you will load the wheel in and the post which you will dismiss it when done... And then the background which is the actual work.

svick
  • 236,525
  • 50
  • 385
  • 514
Anthony Graglia
  • 5,355
  • 5
  • 46
  • 75
  • Not sure the edit was necessary but oh well. The important thing here is that ASyncTask is the intended function for this. It has three functions, pre, during, and post which allow for a lot of power. Take a look at the link. You will learn a lot. – Anthony Graglia Aug 01 '11 at 14:29
2
private void deleteCache() {
  ProgressDialog dialog = ProgressDialog.show(this, "",
    "Please wait for few seconds...", true);

  Runnable myRun = new Runnable() {
    public void run() {
      File f = new File(Environment.getExternalStorageDirectory()
        .getAbsoluteFile() + Constants.DATA_DIR);
      deleteDirectory(f);

      runOnUiThread(new Runnable() {
        @Override
        public void run() {
          dialog.dismiss();
        }
      });
    }
  }
}

private void deleteDirectory(File path) {
  if (path.exists()) {
    File[] files = path.listFiles();
    for (int i = 0; i < files.length; i++) {
      if (files[i].isDirectory()) {
        deleteDirectory(files[i]);
      } else {
        files[i].delete();
      }
    }
  }

  return (path.delete());
}
Anthony Graglia
  • 5,355
  • 5
  • 46
  • 75
Sherif elKhatib
  • 45,786
  • 16
  • 89
  • 106
1

Modify your code like this,

 ProgressDialog dialog = ProgressDialog.show(this, "",
        "Please wait for few seconds...", true);
        new Thread(new Runnable() {
        public void run() {
              File f = new File(Environment.getExternalStorageDirectory()
        .getAbsoluteFile() + Constants.DATA_DIR);
deleteDirectory(f);
   private void deleteDirectory(File path) {
    if (path.exists()) {
        File[] files = path.listFiles();
        for (int i = 0; i < files.length; i++) {
            if (files[i].isDirectory()) {
                deleteDirectory(files[i]);
            } else {
                files[i].delete();
            }
        }
    }
    return (path.delete());
}


            handler.sendEmptyMessage(0);

        }
    }).start();

    handler = new Handler() {
        @Override
        public void handleMessage(Message msg) {

            try {



                progressDialog.dismiss();
            } catch (IllegalArgumentException e) {
                e.printStackTrace();
            }
        }
    };

I am not sure why this happens. Since your code does everything in a single thread, progressdialog will not show up quickly. So instead trying to handle other things in a separate thread handles this problem.

Andro Selva
  • 53,910
  • 52
  • 193
  • 240