1

私は Jsoup を使用する Android アプリに取り組んでいます。開発の早い段階で、スレッド化に取り組む前にコードの大部分を完成させたかったので、あらゆる種類のスレッド化を実装しなければならないことを「回避」しました。AsyncTask を使用しようとしていますが、まだ NetworkOnMainThreadException エラーが発生します。AsyncTask に関する多くのチュートリアルと SO の投稿を読みましたが、それでも問題を特定できるようです。StrictMode... コードを追加すると、Jsoup を使用してデータをロードするときの UI のロックアップを除いて、アプリは希望どおりに動作します。AsyncTask に関して私が間違っていることを誰かが教えてくれたら、ありがたいです。(PS クリーンアップする必要のあるコードの冗長性がたくさんあることは知っていますが、最初に AsyncTask を機能させたいと思います)

public void onCreate(Bundle savedInstanceState)  {
            super.onCreate(savedInstanceState);
            setContentView(R.layout.activity_main);  

            /***This is the work around used***/
            StrictMode.ThreadPolicy policy = new     StrictMode.ThreadPolicy.Builder().permitAll().build();
            StrictMode.setThreadPolicy(policy);
            /******/  



            up = new TreeMap<Double, String[]>(); 

            c1 = "example.com/1";
            //instansiate textviews (6)
                    doc1 = doc;
            c2 = "example.com/2";
            //instansiate textviews (6)
                    doc2 = doc;
            c3 = "example.com/3";
            //instansiate textviews (6)
            doc3 = doc;

            // instansiate textviews(16)

            new Download().execute(c1,c2,c3);

   } 
        private class Download extends AsyncTask<String, Integer, String[][]> {
            @Override
            protected String[][] doInBackground(String... urls){
                out = new String[7][3];
                try {
                    doc = Jsoup.connect(urls[0]).data().get();          
                    //days, times, and cs arrays created and filled

                    String[] out1arr = {days[0], times[0], cs[0]};
                    //...all 7
                    String[] out7arr = {days[6], times[6], cs[6]};
                    String[][] outarrs = {out1arr,out2arr,out3arr,out4arr,out5arr,out6arr,out7arr};
                    for (int i= 0; i < out.length; i++){
                        out[i] = outarrs[i];
                    }
                } catch (IOException e1) {
                    e1.printStackTrace();
                }
                return (out);
            }
            @Override
            protected void onProgressUpdate(Integer... progress){
            }
            @Override
            protected void onPostExecute(String[][] result){

                Do(/*textviews(6)*/, c1, a, outa, "example1"); //a is previously instantiated double array, outa is preiously instantiated string array
                Do(/*textviews(6)*/, c2, b, outb, "example2");
                Do(/*textviews(6)*/, c3, c, outc, "example3");


                upc00.setText(getUpc()[0][0]);
                //setText for all 16
                upc32.setText(getUpc()[3][2]);

            }

            private void Do(TextView t, TextView u, TextView v, TextView w, TextView x, TextView y,String webpage, double[] darr, String[] sarr, String show){


                t.setText(doInBackground(webpage)[0][0]);
                //...all 6
                y.setText(doInBackground(webpage)[1][2]);       
                for (int i =0; i < darr.length; i++){
                    darr[i] = tis[i];
                    up.put(darr[i], out[i]);
                }
            }
        }
        private ArrayList<String[]> getMap(){
                //...
            return s;
        }
        private String[][] getUpc(){
            //...
            return upc;
        }
4

1 に答える 1

0

フレームワークはdoInBackground、自分で呼び出すべきではありません。コードはonPostExecute、UI スレッドでフレームワークによって呼び出される呼び出しを行います。したがって、呼び出しは UI スレッドで効果的に実行されます。

フェッチ ロジックをすべてdoInBackgoundメソッドに移動します。このonPostExecuteメソッドは、呼び出し元に結果を配信するために使用する必要があります。

于 2012-12-29T07:56:08.803 に答える