11

I have a backup routine that copies everything from one folder to an external SD card which works perfectly. I'm trying to get an nice popup dialog box that shows when it's running but it just isn't showing. Doesn't even attempt to run (but the backup does complete).

Here's my code at the moment:

public void doBackup(View view) throws IOException{
    ProgressDialog pd = new ProgressDialog(this);
    pd.setProgressStyle(ProgressDialog.STYLE_SPINNER);
    pd.setMessage("Running backup. Do not unplug drive");
    pd.setIndeterminate(true);
    pd.setCancelable(false);
    pd.show();
    File source = new File("/mnt/extSdCard/DirectEnquiries"); 
    File dest = new File("/mnt/UsbDriveA/Backup");
    copyDirectory(source, dest);
    pd.dismiss();
}
Sufian
  • 6,405
  • 16
  • 66
  • 120
MissCoder87
  • 2,669
  • 10
  • 47
  • 82

3 Answers3

41

You run long running tasks in a Thread or with an AsyncTask. Then your ProgressDialog will show up.

Do something like:

public void doBackup(View view) throws IOException{
    final ProgressDialog pd = new ProgressDialog(this);
    pd.setProgressStyle(ProgressDialog.STYLE_SPINNER);
    pd.setMessage("Running backup. Do not unplug drive");
    pd.setIndeterminate(true);
    pd.setCancelable(false);
    pd.show();
    Thread mThread = new Thread() {
        @Override
        public void run() {
            File source = new File("/mnt/extSdCard/DirectEnquiries"); 
            File dest = new File("/mnt/UsbDriveA/Backup");
            copyDirectory(source, dest);
            pd.dismiss();
        }
    };
    mThread.start();
}
Sufian
  • 6,405
  • 16
  • 66
  • 120
maninder singh
  • 1,286
  • 13
  • 16
  • can you explain why we should put pd.dismiss(); inside the thread I used mThread.join() and I put the pd.dismiss() after it put it doesn't work ? – Mohammed Subhi Sheikh Quroush Apr 02 '14 at 11:30
  • The showing and dismissing of the ProgressDialog object should be done with a handler, since only the original thread can touch the view. – ılǝ Aug 20 '14 at 09:03
2

Create a asyntask and put your time consuming tasks

 public void doBackup(View view) throws IOException{
        ProgressDialog pd = new ProgressDialog(this);
       pd.setProgressStyle(ProgressDialog.STYLE_SPINNER);
        pd.setMessage("Running backup. Do not unplug drive");
       pd.setIndeterminate(true);
       pd.setCancelable(false);
        pd.show();
//create asyntask here
//Put below code in doInBackground mathod
        File source = new File("/mnt/extSdCard/DirectEnquiries"); 
        File dest = new File("/mnt/UsbDriveA/Backup");
        copyDirectory(source, dest);
//put this code in onPostExecute Method.
        pd.dismiss();
    }

You will get number of samples of Asyntask.

andy
  • 208
  • 1
  • 7
1

Instead of doing this in one function at once. Do the following steps and it will definately work for you. 1. Create one async class.(It will create one separate thread for your copy directory functionality and wont run on main UI.) 2. Show your progress dialog before you execute the async class. 3. On post execute method, dismiss your dialog.

omy
  • 21
  • 1