1

forループを使用してdoInBackgroundの配列リストから画像のURLを読み取っていますが、ループはインクリメントされていません。最初のURLのみが画像の読み込みと保存を行っています。ここにコードがあります：

class DownloadImageTask extends AsyncTask<Void, Void, Bitmap> {
    // This class definition states that DownloadImageTask will take String
    // parameters, publish Integer progress updates, and return a Bitmap
    @SuppressWarnings("unused")
    protected Bitmap doInBackground(Void... paths) {
        //URL url;
        try {
              for(int j=0; j<List.size();j++)
            {
                     reviewImageLink =List.get(j).get(TAG_Image);
                     URL     url = new URL(reviewImageLink);
                    // URL reviewImageURL;
                    String name = reviewImageLink.substring(reviewImageLink .lastIndexOf("/") + 1,reviewImageLink.length());
                        //try {

                            if (!hasExternalStoragePublicPicture(name)) {
                                isImage = false;
                                //new DownloadImageTask().execute();
                                Log.v("log_tag", "if");
                                isImage = true;
                                File sdImageMainDirectory = new File(Environment.getExternalStorageDirectory(), getResources().getString(R.string.directory));
                            //if(!sdImageMainDirectory.exists()){
                                sdImageMainDirectory.mkdirs();
                                File file = new File(sdImageMainDirectory, name);
                                Log.v("log_tag", "Directory created");}
                    //}
                    //  catch (MalformedURLException e) {
                        //  Log.v(TAG, e.toString());   }
              //  }

           // }//try 
           // catch (Exception e) {
               //     e.printStackTrace();}
            //url = new URL(List.get(j).get(TAG_Image));
            HttpURLConnection connection = (HttpURLConnection) url.openConnection();
            int length = connection.getContentLength();
            InputStream is = (InputStream) url.getContent();
            byte[] imageData = new byte[length];
            int buffersize = (int) Math.ceil(length / (double) 100);
            int downloaded = 0;
            int read;
            while (downloaded < length) {
                if (length < buffersize) {
                    read = is.read(imageData, downloaded, length);
                } else if ((length - downloaded) <= buffersize) {
                    read = is.read(imageData, downloaded, length- downloaded);
                } else {
                    read = is.read(imageData, downloaded, buffersize);
                }
                downloaded += read;
                setProgress((downloaded * 100) / length);
            }
            Bitmap bitmap = BitmapFactory.decodeByteArray(imageData, 0,length);
            if (bitmap != null) {
                Log.i(TAG, "Bitmap created");
            } else {
                Log.i(TAG, "Bitmap not created");
            }
            is.close();
            return bitmap;

        }
        }catch (MalformedURLException e) {
            Log.e(TAG, "Malformed exception: " + e.toString());
        } catch (IOException e) {
            Log.e(TAG, "IOException: " + e.toString());
        } catch (Exception e) {
            Log.e(TAG, "Exception: " + e.toString());
        }
        return null;

    }

    protected void onPostExecute(Bitmap result) {
        String name = reviewImageLink.substring(reviewImageLink.lastIndexOf("/") + 1,reviewImageLink.length());
        if (result != null) {
            hasExternalStoragePublicPicture(name);
            saveToSDCard(result, name);
            isImage = true;

        } else {
            isImage = false;

        }
    }
}
4

1 に答える 1

1

Listはインターフェイスです。次のようには使用できませんList.size()。ArrayList などの List サブタイプを 1 つ定義します。for ループに問題はありません。例:

List<Integer> li = new ArrayList<Integer>();
li.add(10);
li.add(20)';
for(int i = 0; i <= li.size(); ++i) {
//your stuff
}
于 2013-02-25T07:11:06.690 に答える