How to properly implement a progressBar in Android?

Question

Now I am doing an Android application.In my application I have to get the data from json page.This operation is taking time delay.So I have to show a progressbar until the fetching process is completed.I used the following code to show progressbar.

 public void onCreate(Bundle savedInstanceState) {
//somecode
ProgressDialog progressBar = new ProgressDialog(this);
     progressBar.setCancelable(true);
      progressBar.setMessage("Loading");
      progressBar.show();
      Thread thread = new Thread(this);
      thread.start();

}
 public void run() {
    flag=GetFixtureDetailsJsonFunction();
    handler.sendEmptyMessage(0);
}
protected boolean GetFixtureDetailsJsonFunction() {
//json parsing code
return true
}
private Handler handler = new Handler() {
    @Override
    public void handleMessage(Message msg) {
        if (flag==true) {
            progressBar.dismiss();
        }


    }
};

Using this code I am getting exception.android.view.ViewRoot$CalledFromWrongThreadException: Only the original thread that created a view hierarchy can touch its views.

sarath, see this [Updating the progress bar](http://huuah.com/android-progress-bar-and-thread-updating/) update progress bar using `Handler`.. — ρяσѕρєя K, Mar 19 '12 at 07:38

Hasmukh · Accepted Answer · 2013-03-20T12:04:48.163

AsyncTask is best way for getting response from xml or database. try like this,

private class DownloadQuestion extends AsyncTask<String, Void, String> 
    {           

        @Override
        protected void onPreExecute() 
        {

            pd = ProgressDialog.show(Activity_SplashScreen.this, "","Please wait...", true,false);                    

            super.onPreExecute();
        }

        @Override
        protected String doInBackground(String... urls) 
        {
            //Write background code here Code 
            return "";
        }

        @Override
        protected void onPostExecute(String response1) 
        {
            //Some Code.....

            if (pd != null && pd.isShowing())
                pd.dismiss();

        }
    }

score 1 · Answer 2 · answered Mar 19 '12 at 07:29

Instead i would suggest you to implement AsyncTask, which is known as Painless Threading in android.

Using this AsyncTask, you don't need to bother about managing Threads. And its easy!!

FYI, do as follows:

Display ProgressBar in onPreExecute() method.
Do long running tasks inside the doInBackground() method.
Dismiss the ProgressBar inside the onPostExecute() method. You can also do display kinds of operation in this method.

score 1 · Answer 3 · edited May 23 '17 at 12:11

1

This is a bit strange way to implement this functionality. Instead of fixing that code I suggest you using AsyncTask, which was implemented right for that purpose. See here.

edited May 23 '17 at 12:11

Community

1
1

answered Mar 19 '12 at 07:32

Vladimir Ivanov

42,730
18
77
103

score 1 · Answer 4 · answered Mar 19 '12 at 07:32

You are trying to access the UI View from another thread which is not eligible.In this case this is your handler.

Instead of trying to access UI thread like this you should use an AsyncTask and do your progressDialog logic in it.

Start showing the progress bar onPreExecute
doInBackground() jobs while progressBar showing
And finallly dismiss your progressBar after your doInBackground() is complete onPostExecute()

How to properly implement a progressBar in Android?

4 Answers4