2013-09-30 5 views
0

Я хочу позвонить в новый AsyncTask от onPostExecute от AsyncTask.Вызов другой AsyncTask из метода onPostExecute

  protected void onPostExecute(Integer feed) { 
     dialog1.dismiss();   
     super.onPostExecute(feed); 
     new SendsequenceofRequest().execute(); 
    } 

Но я получаю следующее исключение:

 android.os.NetworkOnMainThreadException 

Мой выход LogCat является:

09-30 16:08:19.517: W/System.err(3706): android.os.NetworkOnMainThreadException 
09-30 16:08:19.517: W/System.err(3706):  at android.os.StrictMode$AndroidBlockGuardPolicy.onNetwork(StrictMode.java:1077) 
09-30 16:08:19.517: W/System.err(3706):  at dalvik.system.BlockGuard$WrappedNetworkSystem.read(BlockGuard.java:279) 
09-30 16:08:19.517: W/System.err(3706):  at org.apache.harmony.luni.net.PlainSocketImpl.read(PlainSocketImpl.java:448) 
09-30 16:08:19.517: W/System.err(3706):  at org.apache.harmony.luni.net.SocketInputStream.read(SocketInputStream.java:75) 
09-30 16:08:19.517: W/System.err(3706):  at org.apache.http.impl.io.AbstractSessionInputBuffer.fillBuffer(AbstractSessionInputBuffer.java:103) 
09-30 16:08:19.517: W/System.err(3706):  at org.apache.http.impl.io.SocketInputBuffer.isStale(SocketInputBuffer.java:132) 
09-30 16:08:19.517: W/System.err(3706):  at org.apache.http.impl.AbstractHttpClientConnection.isStale(AbstractHttpClientConnection.java:205) 
09-30 16:08:19.517: W/System.err(3706):  at org.apache.http.impl.conn.AbstractClientConnAdapter.isStale(AbstractClientConnAdapter.java:185) 
09-30 16:08:19.577: W/System.err(3706):  at org.apache.http.impl.client.DefaultRequestDirector.execute(DefaultRequestDirector.java:336) 
09-30 16:08:19.577: W/System.err(3706):  at org.apache.http.impl.client.AbstractHttpClient.execute(AbstractHttpClient.java:555) 
09-30 16:08:19.577: W/System.err(3706):  at org.apache.http.impl.client.AbstractHttpClient.execute(AbstractHttpClient.java:487) 
09-30 16:08:19.597: W/System.err(3706):  at org.apache.http.impl.client.AbstractHttpClient.execute(AbstractHttpClient.java:465) 
09-30 16:08:19.597: W/System.err(3706):  at com.dm.ekot.reqres.SimpleHttpClient.sendresponseSequReqRes(SimpleHttpClient.java:715) 
09-30 16:08:19.607: W/System.err(3706):  at com.dm.ekot.MainScreen$SendsequenceofRequest$1$1.run(MainScreen.java:1020) 
09-30 16:08:19.620: W/System.err(3706):  at android.os.Handler.handleCallback(Handler.java:587) 
09-30 16:08:19.620: W/System.err(3706):  at android.os.Handler.dispatchMessage(Handler.java:92) 
09-30 16:08:19.620: W/System.err(3706):  at android.os.Looper.loop(Looper.java:132) 
09-30 16:08:19.620: W/System.err(3706):  at android.app.ActivityThread.main(ActivityThread.java:4025) 
09-30 16:08:19.627: W/System.err(3706):  at java.lang.reflect.Method.invokeNative(Native Method) 
09-30 16:08:19.627: W/System.err(3706):  at java.lang.reflect.Method.invoke(Method.java:491) 
09-30 16:08:19.627: W/System.err(3706):  at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:841) 
09-30 16:08:19.627: W/System.err(3706):  at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:599) 
09-30 16:08:19.642: W/System.err(3706):  at dalvik.system.NativeStart.main(Native Method) 

В моей SendsequenceofRequestAsyncTask я начать другую нить в doInBackground() методом.

  class SendsequenceofRequest extends AsyncTask<String, Void, Integer> { 

// private ProgressDialog dialog1; 

    protected void onPreExecute() { 
    // dialog1 = ProgressDialog.show(MainScreen.this, "", "Loading..."); 
    } 

    protected void onPostExecute(Integer feed) { 
    // dialog1.dismiss();   
     super.onPostExecute(feed); 
    }   

    @Override 
    protected Integer doInBackground(String... arg0) {  

        String str_stater_menu = "Starters"; 
        String response_starter = ""; 
       try { 

        new Thread(new Runnable() { 
         @Override 
         public void run() { 
          // TODO Auto-generated method stub 
          while (isThreadPaused1) { 
           try { 
            Thread.sleep(3000); 
            mHandler.post(new Runnable() {  

             @Override 
             public void run() { 
              // TODO Auto-generated method stub 
              try { 
               final String response_req_sequence = SimpleHttpClient.sendresponseSequReqRes(response_send_order); 
               System.out.println(" i ma in thread"); 
               if(response_req_sequence != null) 
    { 
                onPause(); 
     runOnUiThread(new Runnable() {  
       @Override 
       public void run() {  
        // TODO Auto-generated method stub 
        Toast.makeText(getApplicationContext(), "Your Order will be delivered in"+response_req_sequence, 10000).show(); 
       } 
      }); 
    } 
} catch (Exception e) { 
    // TODO Auto-generated catch block 
    e.printStackTrace(); 
} 
             } 
            }); 
           } catch (Exception e) { 
            // TODO: handle exception 
           } 
          } 
         } 
        }).start(); 
        //response_starter = SimpleHttpClient.sendFirstStarter(str_stater_menu); 
       } catch (Exception e1) { 
        // TODO Auto-generated catch block 
        e1.printStackTrace(); 
       }  
        System.out.println("i am getting response for starter"+response_starter); 
       // str_reponse_request_tableid_ip = SimpleHttpClient.sendIpTabletId(url , str_ip); 

       /** 
       *  
       */ 
       // res = response_starter.toString(); 
        System.out.println("response TEST: "+res); 
      // Object obj = response_starter.toString(); 
       System.out.println("after object creation "); 

       // getJsonStringForStarter(response_starter); 

       System.out.println("response :"+res);  

      /** Inside the new thread we cannot update the main thread 
      So updating the main thread outside the new thread */ 
      try { 

      }catch (Exception e) { 
     e.printStackTrace(); 
       // e.printStackTrace(); 
      } 
     return null; 
      }  


    } 

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

+1

Почему вы создаете темы в AsyncTask? Все дело в том, что вам не нужно управлять своими потоками. – Geobits

+0

Спасибо за ваш ответ. Я должен отправлять запрос на сервер до тех пор, пока я не получу никакого ответа. И этот запрос будет выполнен после завершения первого запроса. Вот почему я использую его в методе onPostExecute. Если у вас есть другое решение или предложение, пожалуйста, помогите мне .. – DJhon

+2

Вы можете сделать это в 'doInBackground'. Просто добавьте 'while (true)' loop с 'Thread.sleep (1000)', и как только он будет успешным, выполните 'break' – Tseng

ответ

2

Я хотел бы предложить использовать publishProgress публиковать текущий прогресс и переопределить onProgressUpdate который выполняется в основном потоке. При этом вам не придется использовать обработчики и потоки в AsyncTask.

Обратите внимание, что первая линия изменилась с extends AsyncTask<String, Void, Integer> на extends AsyncTask<String, String, Integer>. Средний параметр определяет тип объекта параметра обновления. Если вам нужно отправить сложные данные методу обновления, измените его на класс, содержащий всю необходимую информацию.

class SendsequenceofRequest extends AsyncTask<String, String, Integer> { 
    @Override 
    protected Integer doInBackground(String... arg0) {  
     try { 
      // This will run indefinitely, unless stopped by break. So make sure you have an exit condition, see below 
      String response_req_sequence = null; 
      do { 
       response_req_sequence = SimpleHttpClient.sendresponseSequReqRes(response_send_order); 
       if(response_req_sequence != null) { 
        // Instead of calling runOnUiThread, call progress update 
        publishProgress(response_req_sequence); 
       } 

       // Wait before retry 
       Thread.sleep(3000); 

      // Keep in mind, that if the connection is successful, the string may be empty, 
      // so TextUtils.isEmpty(response_req_sequence); may be better exit condition as it checks if string is null or has size of 0 
      } while(response_req_sequence!=null); 
     } catch (Exception e) { 
      e.printStackTrace(); 
     } 

     return null; 
    } 

    @Override 
    void onProgressUpdate(String... string) { 
     // This will always be executed in main thread 
     onPause(); 
     Toast.makeText(getApplicationContext(), "Your Order will be delivered in"+response_req_sequence, 10000).show(); 
    } 

    protected void onPostExecute(Integer feed) { 
     // Notify user the task has bene completed 
     Toast.makeText(getApplicationContext(), "Task completed.", 10000).show(); 
    }   
} 
+0

Спасибо .... Позвольте мне проверить ваше предложение. – DJhon

+0

Я получил ваше предложение, но как я отправлю запрос снова и снова, пока я не получил ответ. – DJhon

+1

Это делается в 'doInBackground'. 'While (true)' будет вызываться снова и снова, пока вы не назовете 'break' внутри него. Я не знаю вашего фактического условия выхода, поэтому я использовал 'if (somecondition == true)' в качестве владельца места. Если 'null' будет вашим условием выхода, вы можете сделать' if (response_req_sequence == null) break; 'для выхода из цикла while – Tseng

0

Если mHandler был создан из основной/пользовательской нити, то любые Runnable s, отправленные в него, также будут работать на основной теме. Поскольку вы пытаетесь выполнить сетевой ввод-вывод внутри такого Runnable, приложение бросает NetworkOnMainThreadException.

+0

Спасибо за быстрый ответ .. Так что я должен решить мою проблему. – DJhon

+0

Я предлагаю удалить код 'Handler' и' Thread' из вашего метода 'doInBackground()', поскольку по определению вы уже работаете в фоновом потоке. Пожалуйста, см. Ответ Ценга для более полного решения. – acj

0

Runnable начато

mHandler.post(Runnable runnable); 

выполняется на основном потоке, и вы пытаетесь использовать сети в том, что работоспособной.

Чтобы избежать этой проблемы просто создать новую нить и запустить его, как это:

Thread t = new Thread(new Runnable { 
... 
}); 
t.start(); 
+0

Я уже пробовал это, но это для меня такое же исключение. – DJhon

+2

Нет необходимости работать с Threads, если вы собираетесь использовать 'AsyncTask'. «AsyncTask» был сделан как удобство, чтобы упростить работу и не работать с обработчиками и потоками напрямую. См. Мой ответ ниже – Tseng

+0

Вы пытались использовать сетевые функции внутри 'doInBackground'? Зачем создавать внутреннюю нить? Вы можете использовать 'Thread.sleep (...)' wrapped by 'while' внутри' doInBackground'. – XeNoN

Смежные вопросы