android.os.NetworkOnMainThreadException in XmlPullParser [duplicate]

Question

I use google places api to return a list of interesting places nearby. Each place and its attributes are enclosed in result and /result tag. After encountering the first /result tag I get Network on main thread exception whenever there is parser.next() statement.

      try {
        int type = parser.getEventType();
         Locations obj = null;

        while (type != XmlPullParser.END_DOCUMENT) {

            String name = parser.getName();
            switch (type) {
                case XmlPullParser.START_TAG: {
                    if (name.equalsIgnoreCase("result"))
                    {obj = new Locations();}

                    else if (name.equalsIgnoreCase("name"))
                        obj.name = parser.nextText();

                    else if (name.equalsIgnoreCase("vicinity"))
                        obj.address = parser.nextText();
                    else if (name.equalsIgnoreCase("lat"))
                        lat = parser.nextText();
                    else if (name.equalsIgnoreCase("lng")) {
                        lng = parser.nextText();
                    LatLng latLng = new LatLng(Double.parseDouble(lat),Double.parseDouble(lng));
                        obj.latlng = latLng;
                    }
                    break;
                }

                case XmlPullParser.END_TAG: {
                    if (name.equalsIgnoreCase("result")) {
                        entries.add(obj);
                    }
                    break;
                }
            }//End Switch Case
            type = parser.next();

            }//End While Block



        } //End try block

      catch (IOException ioex)
      {
        ioex.printStackTrace();
      }
      catch (XmlPullParserException ioex)
      {
        ioex.printStackTrace();
      }
      catch (Exception ioex)
      {
        ioex.printStackTrace();
      }

Each object of Locations class represent a place. I have used a separate thread to perform httpurl connection activities and have also set read timeout to 3 mins. (below class is used to open http network connection)

    public class DownloadxmlTask  {

    public String status=null;
    public InputStream isi=null;
    private InputStream loadXmlfromnetwork(String url) {
    InputStream in = null;
    try {
        URL urlobj = new URL(url);
        HttpURLConnection conobj = (HttpURLConnection) urlobj.openConnection();
        conobj.setReadTimeout(3*60000);
        in = conobj.getInputStream();
        isi=in;
        status = in.toString();
        wait(3*60000);
        conobj.disconnect();
     } catch (Exception ex) {
        status =ex.getLocalizedMessage().toString();
      }
      return in;
      }
     protected InputStream doInBackground(String params) {

        final String url = params;

    Thread newthread=new Thread() {
        @Override
        public void run() {
            loadXmlfromnetwork(url);
        }
            };
        newthread.start();
      return isi;
      }
     }

@tyczj. I could see clearly that the Locations class may be a POJO class. Please look into the question and encourage forum newbie. — Nilanchala
– Nilanchala, Commented Aug 5, 2015 at 14:08
@Nilanchala he said he was getting the error on the first tag, how do you know that Locations does not do some network operation in the constructor? — tyczj
– tyczj, Commented Aug 5, 2015 at 14:11
@Bharaneedharan if you say you use a different thread to make the network connection you should show that because nothing in your question shows you are doing that — tyczj
– tyczj, Commented Aug 5, 2015 at 14:21

Nilanchala · Accepted Answer · 2015-08-05 14:06:40Z

The error is self explanatory. You need to perform your parser call inside a separate thread off from UI main thread. While there are various technique to achieve this, the simplest one is to create a AsyncTask.

I have pasted the simplest version of AsyncTask implementation. The doInBackground() method runs on its own separate thread and onPostExecute() executes on UI main thread.

Place all your parser logic inside doInBackground() method.

private final AsyncTask<Void, Void, Void> cacheInitializationTask = new AsyncTask<Void, Void, Void>() {
        @Override
        protected Void doInBackground(Void... params) {

            return null;
        }

        @Override
        protected void onPostExecute(Void aVoid) {
            //Runs on UI thread
            super.onPostExecute(aVoid);
        }
    };

For more information visit this link AsyncTask

Collectives™ on Stack Overflow

android.os.NetworkOnMainThreadException in XmlPullParser [duplicate]

1 Answer 1

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Linked

Related