0

ProgressBar in AsyncTask causes crash. This code essentially loads an image from a URL, and while doing so shows a progress spinner. The first image in the group loads fine, but following that the app crashes.

public class LoadImageTask extends AsyncTask<String, Void, Bitmap> {
ImageView bmImage;
ProgressBar progressBar;

public LoadImageTask(ImageView bmImage, ProgressBar progressBar) {
    this.bmImage = bmImage;
    this.progressBar = progressBar;
}

protected Bitmap doInBackground(String... urls) {
    progressBar.setVisibility(View.VISIBLE);
    String urldisplay = urls[0];
    Bitmap scaledImage = null;
    try {
        InputStream in = new java.net.URL(urldisplay).openStream();
        Bitmap image = BitmapFactory.decodeStream(in);

        scaledImage = Bitmap.createScaledBitmap(image, 380, 250, false);
    } catch (Exception e) {
        Log.e("Error", e.getMessage());
        e.printStackTrace();
    }
    return scaledImage;
}

protected void onPostExecute(Bitmap result) {
    progressBar.setVisibility(View.GONE);
    bmImage.setImageBitmap(result);
}

}

Console printout:

 02-03 02:45:44.014  24125-24375/simplewall.ryandushane.com.simplewall E/AndroidRuntime﹕ FATAL EXCEPTION: AsyncTask #5
    Process: simplewall.ryandushane.com.simplewall, PID: 24125
    java.lang.RuntimeException: An error occured while executing doInBackground()
            at android.os.AsyncTask$3.done(AsyncTask.java:300)
            at java.util.concurrent.FutureTask.finishCompletion(FutureTask.java:355)
            at java.util.concurrent.FutureTask.setException(FutureTask.java:222)
            at java.util.concurrent.FutureTask.run(FutureTask.java:242)
            at android.os.AsyncTask$SerialExecutor$1.run(AsyncTask.java:231)
            at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)
            at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)
            at java.lang.Thread.run(Thread.java:818)
     Caused by: android.view.ViewRootImpl$CalledFromWrongThreadException: Only the original thread that created a view hierarchy can touch its views.
            at android.view.ViewRootImpl.checkThread(ViewRootImpl.java:6247)
            at android.view.ViewRootImpl.requestLayout(ViewRootImpl.java:867)
            at android.view.View.requestLayout(View.java:17364)
            at android.view.View.requestLayout(View.java:17364)
            at android.view.View.requestLayout(View.java:17364)
            at android.view.View.requestLayout(View.java:17364)
            at android.view.View.requestLayout(View.java:17364)
            at android.view.View.requestLayout(View.java:17364)
            at android.widget.RelativeLayout.requestLayout(RelativeLayout.java:360)
            at android.view.View.requestLayout(View.java:17364)
            at android.widget.AbsListView.requestLayout(AbsListView.java:1975)
            at android.view.View.requestLayout(View.java:17364)
            at android.widget.RelativeLayout.requestLayout(RelativeLayout.java:360)
            at android.view.View.requestLayout(View.java:17364)
            at android.view.View.setFlags(View.java:9633)
            at android.view.View.setVisibility(View.java:6663)
            at android.widget.ProgressBar.setVisibility(ProgressBar.java:1563)
            at simplewall.ryandushane.com.simplewall.LoadImageTask.doInBackground(LoadImageTask.java:27)
            at simplewall.ryandushane.com.simplewall.LoadImageTask.doInBackground(LoadImageTask.java:17)
            at android.os.AsyncTask$2.call(AsyncTask.java:288)
            at java.util.concurrent.FutureTask.run(FutureTask.java:237)
            at android.os.AsyncTask$SerialExecutor$1.run(AsyncTask.java:231)
            at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)
            at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)
            at java.lang.Thread.run(Thread.java:818)

4 Answers 4

2

CalledFromWrongThreadException: Only the original thread that created a view hierarchy can touch its views

Because you are calling progressBar.setVisibility method from doInBackground which run on non-ui thread.

Override onPreExecute() method of AsyncTask which run on Main UI Thread to change Visibility of progress bar:

@Override
protected void onPreExecute() {
     progressBar.setVisibility(View.VISIBLE);
}
1

You cannot set progressBar.setVisibility(View.VISIBLE); in doInBackground. its a different thread

or

protected Bitmap doInBackground(String... urls) {
    runOnUiThread (new Runnable () {

                        @Override
                        public void run () {

                            progressBar.setVisibility(View.VISIBLE);

                        }
                    });

    String urldisplay = urls[0];
    Bitmap scaledImage = null;
    try {
        InputStream in = new java.net.URL(urldisplay).openStream();
        Bitmap image = BitmapFactory.decodeStream(in);

        scaledImage = Bitmap.createScaledBitmap(image, 380, 250, false);
    } catch (Exception e) {
        Log.e("Error", e.getMessage());
        e.printStackTrace();
    }
    return scaledImage;
}
1

Move progressBar.setVisibility(View.VISIBLE); in onPreExecute

0

Put this progressBar.setVisibility(View.VISIBLE) in onPreExecute() method of AsyncTask and remove this line from doInBackground method.

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.