NetworkOnMainThread

Question

Я получаю исключение NetworkOnMainThreadException при попытке реализовать следующий код:

public class HandlingXMLStuff extends ListActivity{
static final String URL = "xml_file";

 static final String ITEM = "item"; //parent
    static final String Id = "id";
    static final String Name = "name";
    static final String Desc = "desc";
    static final String Link = "Link";

@Override
public void onCreate(Bundle savedInstanceState){
    super.onCreate(savedInstanceState);
    setContentView(R.layout.xmllist);

ArrayList<HashMap<String, String>> menuItems = new ArrayList<HashMap<String, String>>(); 

xmlparser parser = new xmlparser();
String xml = parser.getXmlFromUrl(URL);
Document doc = parser.getDomElement(xml);


NodeList nl = doc.getElementsByTagName(ITEM);


//START: loop through all item nodes <item>
for (int i = 0;i<nl.getLength();i++){
    //lets create our HASHMAP!! (feeds items into our ArrayList)
    HashMap<String, String> map = new HashMap<String, String>();
    Element e = (Element) nl.item(i);
    //add each child node to the HashMap (key, value/<String, String>)
    map.put(Name, parser.getValue(e, Name));
    map.put(Desc, parser.getValue(e, Desc));
    map.put(Link, parser.getValue(e, Link));


    menuItems.add(map);

}//DONE 


ListAdapter adapter = new SimpleAdapter(this, menuItems, R.layout.list_item,
        new String[] {Name, Desc, Link}, new int []{R.id.name, R.id.description, R.id.link});

setListAdapter(adapter);
        }
}

и обработчик:

public class xmlparser{

public String getXmlFromUrl(String url) {
String xml = null;

try {

    DefaultHttpClient httpClient = new DefaultHttpClient();
    HttpPost httpPost = new HttpPost(url);

    HttpResponse httpResponse = httpClient.execute(httpPost);
    HttpEntity httpEntity = httpResponse.getEntity();
    xml = EntityUtils.toString(httpEntity);

} catch (UnsupportedEncodingException e) {
    e.printStackTrace();
} catch (ClientProtocolException e) {
    e.printStackTrace();
} catch (IOException e) {
    e.printStackTrace();
}

return xml;
}
public Document getDomElement(String xml){
    Document doc = null;
    DocumentBuilderFactory dbf = DocumentBuilderFactory.newInstance();
    try {

        DocumentBuilder db = dbf.newDocumentBuilder();

        InputSource is = new InputSource();
            is.setCharacterStream(new StringReader(xml));
            doc = db.parse(is); 

        } catch (ParserConfigurationException e) {
            Log.e("Error: ", e.getMessage());
            return null;
        } catch (SAXException e) {
            Log.e("Error: ", e.getMessage());
            return null;
        } catch (IOException e) {
            Log.e("Error: ", e.getMessage());
            return null;
        }

        return doc;
  }

public String getValue(Element item, String str) {      
        NodeList n = item.getElementsByTagName(str);        
        return this.getElementValue(n.item(0));
    }

public final String getElementValue( Node elem ) {
         Node child;
         if( elem != null){
             if (elem.hasChildNodes()){
                 for( child = elem.getFirstChild(); child != null; child = child.getNextSibling() ){
                     if( child.getNodeType() == Node.TEXT_NODE  ){
                         return child.getNodeValue();
                     }
                 }
             }
         }
         return "";
  }    

}

Есть идеи, почему? Это должно работать, все уроки, которые я прочитал, рассматривают это как рабочий код, но он не запускается и выдает только исключение. Я читал, что мне может понадобиться реализовать asynctask, но я новичок в этом и не уверен, какие части нуждаются в собственной нити. Спасибо за любую помощь, критику (конструктив), предложения и т. Д.

Answer 1

Есть идеи почему?

Потому что, если этот кусок кода выполняется в главном потоке приложения, вы выполняете сетевой ввод-вывод в главном потоке приложения.

Я читал, что мне может понадобиться реализовать asynctask, но я новичок в этом и не уверен, для каких частей нужен собственный поток.

Я бы поставил сетевой ввод-вывод и разбор в doInBackground(), а вызов setListAdapter() в onPostExecute() AsyncTask.

Answer 2

Если вы просто хотите проверить свой код и пока не хотите добавлять больше сложностей, вы можете добавить это к своему onCreate()

if (android.os.Build.VERSION.SDK_INT > 9) {
      StrictMode.ThreadPolicy policy = new StrictMode.ThreadPolicy.Builder().permitAll().build();
      StrictMode.setThreadPolicy(policy);
}

Вы не хотите, чтобы это былопостоянные, так как сетевые операции в потоке пользовательского интерфейса приводят к плохому опыту при использовании приложения, но могут быть полезны при тестировании.

Answer 3

Добавляя к ответу CommonsWare, исключение NetworkOnMainThreadException было добавлено между 2.3.3 (Gingerbread_MR1) и 3.0 (Honeycomb). Если вы посмотрите на

android.app.ActivityThread

вы найдете следующий фрагмент кода:

    /**
     * For apps targetting SDK Honeycomb or later, we don't allow
     * network usage on the main event loop / UI thread.
     *
     * Note to those grepping:  this is what ultimately throws
     * NetworkOnMainThreadException ...
     */
    if (data.appInfo.targetSdkVersion > 9) {
        StrictMode.enableDeathOnNetwork();
    }

Я думаю, что учебные пособия, которые вы читали, были написаны до того, как это было введено в действие, и поэтому не вызвали исключение NetworkOnMainThreadException. Следуйте инструкциям CommonsWare относительно AsyncTask, и вы исправите свою ошибку.