2016-06-13 4 views
1

コードを修正するのに手伝ってくれる人はいますか?2つのASyncTasksを実行するAndroidスタジオ

明らかに問題は、メインスレッドで複数のASyncTaskを実行できないということです。私のコードを修正する方法について誰かに助言を与えることができますか? ありがとうございました!

私のコードにコメントしていないことをお詫び申し上げます。読んでいる間に混乱しているかどうかを説明することができます。

public class MainActivity extends AppCompatActivity { 

    ListView listView; 
    private SQLiteDatabase myDatabase; 
    private Cursor cursor; 
    boolean finished = false; 

    @Override 
    protected void onCreate(Bundle savedInstanceState) { 
     super.onCreate(savedInstanceState); 
     setContentView(R.layout.activity_main); 

     listView = (ListView) findViewById(R.id.listView); 
     myDatabase = this.openOrCreateDatabase("HackerNews", MODE_PRIVATE, null); 

     Cursor cursor = myDatabase.rawQuery("SELECT * FROM ids", null); 
     int index = cursor.getColumnIndex("urlID"); 
     cursor.moveToFirst(); 

     DownloadContent content = new DownloadContent(); 

     while(cursor != null){ 
       String newUrl = "https://hacker-news.firebaseio.com/v0/item/" + cursor.getString(index) + ".json?print=pretty"; 
       content.execute(newUrl); 
       cursor.moveToNext(); 
     } 
    } 

    public class DownloadIDs extends AsyncTask<String, Void, String> { 

     @Override 
     protected String doInBackground(String... params) { 

      String result = ""; 
      URL url; 
      HttpURLConnection urlConnection = null; 
      try { 
       url = new URL(params[0]); 
       urlConnection = (HttpURLConnection) url.openConnection(); 
       InputStream inputStream = urlConnection.getInputStream(); 
       InputStreamReader reader = new InputStreamReader(inputStream); 
       int data = reader.read(); 

       while (data >= 0) { 
        char current = (char) data; 
        result += current; 
        data = reader.read(); 
       } 

       return result; 

      } catch (Exception e) { 
       e.printStackTrace(); 
       return "Fail"; 
      } 
     } 

     @Override 
     protected void onPostExecute(String s) { 
      super.onPostExecute(s); 

      myDatabase.execSQL("CREATE TABLE IF NOT EXISTS ids (id INTEGER PRIMARY KEY, urlID VARCHAR)"); 
      cursor = myDatabase.rawQuery("SELECT COUNT(*) FROM ids", null); 
      cursor.moveToFirst(); 
      int count = cursor.getInt(0); 

      if (!(count > 0)) { 

       try { 
        JSONArray ids = new JSONArray(s); 
        for (int i = 0; i < ids.length(); i++) { 
         myDatabase.execSQL("INSERT INTO ids (urlID) VALUES ('" + ids.getString(i) + "')"); 
        } 

       } catch (JSONException e) { 
        e.printStackTrace(); 
       } 
      } else { 
       Log.i("message", "TABLE1 IS NOT EMPTY"); 
      } 
     } 
    } 

    public class DownloadContent extends AsyncTask<String, Void, String> { 

     @Override 
     protected String doInBackground(String... params) { 
      String result = ""; 
      URL url; 
      HttpURLConnection urlConnection = null; 
      try { 
       url = new URL(params[0]); 
       urlConnection = (HttpURLConnection) url.openConnection(); 
       InputStream inputStream = urlConnection.getInputStream(); 
       InputStreamReader reader = new InputStreamReader(inputStream); 
       int data = reader.read(); 

       while (data >= 0) { 
        char current = (char) data; 
        result += current; 
        data = reader.read(); 
       } 

       return result; 

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

     @Override 
     protected void onPostExecute(String s) { 
      super.onPostExecute(s); 

      myDatabase.execSQL("CREATE TABLE IF NOT EXISTS content(id INTEGER PRIMARY KEY, title VARCHAR, url VARCHAR)"); 
      cursor = myDatabase.rawQuery("SELECT COUNT(*) FROM content", null); 
      cursor.moveToFirst(); 
      int count = cursor.getInt(0); 

      if (!(count > 0)) { 

       try { 
        JSONObject jsonObject = new JSONObject(s); 
        String title = jsonObject.getString("title"); 
        String url = jsonObject.getString("url"); 

        myDatabase.execSQL("INSERT INTO content (title, url) VALUES('" + title +"','" + url + "')"); 

       } catch (JSONException e) { 
        e.printStackTrace(); 
       } 
      } else { 
       Log.i("mess2", "table 2 is NOT EMPTY"); 
      } 
     } 
    } 
} 
+0

すべてのAsyncTasksは同じバックグラウンドスレッドを共有するため、2番目のネットワーク要求は最初の競合がブロックされるまでブロックされます。 –

+0

非同期タスクはデフォルトでシリアルエグゼキュータを使用しますが、plsはこれを参照しています[複数のAsyncTasksを同時に実行する - 不可能ですか?](http://stackoverflow.com/questions/4068984/running-multiple-asynctasks-at-the-sameタイムアウト不可/ 13800208#13800208) –

答えて

0

AsyncTaskインスタンスは1回しか実行できません。これを解決する最も簡単な方法は、実行する必要があるときにはいつでも新しいインスタンスを作成することです。

while(cursor != null) { 
    String newUrl = "https://hacker-news.firebaseio.com/v0/item/" + cursor.getString(index) + ".json?print=pretty"; 
    new DownloadContent().execute(newUrl); 
    cursor.moveToNext(); 
} 
+0

ありがとう!それはそれを修正した。私はあなたの答えを受け入れることを試みたが、ウェブサイトは私をさせません。 –

関連する問題