android 4.1 image reading from url

Question

Hi I am trying to get an image from url to bitmap. I have android 4.1 device. When I run this code on new URL(). open connection().getInputStream()); app freezes then force close. Any idea?

   runOnUiThread(new Runnable() {
            public void run() {
                String url = "http://netmera.com/cdn/app/file/netmera.com/series/img-48/1372262272227_89/medium";
                try {
                    Bitmap bmp = BitmapFactory.decodeStream(new URL(url)
                            .openConnection().getInputStream());
                } catch (MalformedURLException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                } catch (IOException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                } 
              }

Try with Bitmap bmp = BitmapFactory.decodeStream(new URL(url).openStream()); Usually I'm using BitmapFactory.decodeStream(new URL(url).openStream(), null, BitmapOptions); — Kamen Stoykov, Jun 27 '13 at 09:23
check your logcat if you have any exceptions `NetworkOnMainThreadException`. — Raghunandan, Jun 27 '13 at 09:26
it says Channel is unrecoverably broken and will be disposed! Error opening directory '/storage/sdcard1/', skipping: Permission denied. @KamenStoykov I trid it that has same problem — user2527366, Jun 27 '13 at 09:31
have you set Internet permissions in manifest or you getting NetworkOnMainTrhreadException, in those case refer [this link](http://stackoverflow.com/a/9288544/2345913) — CRUSADER, Jun 27 '13 at 09:31
post the stack here and you should use asyntask for network related operation — Raghunandan, Jun 27 '13 at 09:41
I have an async task and this runOnUiThread in this async task @Raghunandan — user2527366, Jun 27 '13 at 09:45
@user2527366 check the updated answer and post the stack trace also. — Raghunandan, Jun 27 '13 at 09:48

score 0 · Answer 1 · answered Jun 27 '13 at 09:37

0

bitmap = BitmapFactory.decodeStream((InputStream) new URL("your url").getContent());

answered Jun 27 '13 at 09:37

user2462737

208
2
11

score 0 · Answer 2 · answered Jun 27 '13 at 09:38

The reason for the Crash can be two things

Network operation should not be done on the UI thread. Kindly use AsyncTask for this. //NetworkOnMainThreadException
Do not use strong referneces for Bitmaps. Use WeakReference<Bitmap> objects //OutOfMemoryException

Raghunandan · Accepted Answer · 2013-06-27T10:02:44.167

You are running network related operation on the ui thread using runOnUiThread.

You should use a Thread or use Asynctask.

http://developer.android.com/reference/android/os/AsyncTask.html

You are probably getting NetworkOnMainThreadException

http://developer.android.com/reference/android/os/NetworkOnMainThreadException.html

Load asynctask on ui thread.

   new TheTask().execute().

AsyncTask

  class TheTask extends AsyncTask<Void,Void,Void>
  {

@Override
protected Void doInBackground(Void... params) {
    // TODO Auto-generated method stub
             String url = "http://netmera.com/cdn/app/file/netmera.com/series/img-48/1372262272227_89/medium"; 
    try {
                Bitmap bmp = BitmapFactory.decodeStream(new URL(url)
                        .openConnection().getInputStream());

            } catch (MalformedURLException e) {
                // TODO Auto-generated catch block
                e.printStackTrace();
            } catch (IOException e) {
                // TODO Auto-generated catch block
                e.printStackTrace();
            } 
    return null;
}

@Override
protected void onPostExecute(Void result) {
    // TODO Auto-generated method stub
    super.onPostExecute(result);
}

@Override
protected void onPreExecute() {
    // TODO Auto-generated method stub
    super.onPreExecute();
}
}

Use runOnUiThread to update ui and do your netowrk related operation in doInbackground().

  runOnUiThread(new Runnable() //run on ui thread
                 {
                  public void run() 
                  { 
                     // update ui
                  }
                 });

@user2527366 remove the `runonuithread` then. this bitmap = `BitmapFactory.decodeStream((InputStream) new URL( doctor.getPhoto()).getContent())` should be outside `runonuithread` in `doInbackground()`. you can update ui in `runonuithread`. — Raghunandan, Jun 27 '13 at 09:43

score 0 · Answer 4 · answered Jun 27 '13 at 09:42

I am doing it like this

final Handler handler = new Handler(){
            @Override
            public void handleMessage(Message msg) {
                // TODO Auto-generated method stub

                if(msg.what==1){
                    doctoImage.setImageBitmap(bitmap);// doctoImage you image view
                }
            }
        };

Thread thread = new Thread(new Runnable() {

            public void run() {
                // TODO Auto-generated method stub

                try {
                    bitmap = BitmapFactory.decodeStream((InputStream) new URL(
                            doctor.getPhoto()).getContent());

                    handler.sendEmptyMessage(1);
                } catch (MalformedURLException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                } catch (IOException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }
            }
        });thread.start();

android 4.1 image reading from url

4 Answers4