Индикатор выполнения в виде списка при загрузке файла AsyncTask - PullRequest
6 голосов
/ 17 августа 2011

У меня странная проблема, у меня запускается новая асинхронная задача всякий раз, когда я нахожу файл MP3 в веб-браузере, и запускаю индикатор выполнения в списке просмотра для каждой AsyncTask. Так что количество скачиваний может быть более 1 и одновременно. Но теперь, когда запускается Asynctask, ProgressBar перемещается одинаково для всех и не различается для разных AsyncTask, PLZ направляет меня .......

LISTVIEW WITH PROGRESS BAR

    public class CopyOfDownloadsListActivity extends ListActivity {
    /** Called when the activity is first created. */

//  static ArrayList<String> pthreads = new ArrayList<String>();
ImageView bt;
ProgressBar pb;
ListView allList;
TextView tv;
String fileName;
String mp3URL;
    URL url2;
    int filecount  = 0;

private class DownloadFile extends AsyncTask<String, Integer, Void>{
    MyCustomAdapter adapter;

    int count = 0;
    ProgressDialog dialog;
    ProgressBar progressBar;
    int myProgress;

   @Override        
    protected Void doInBackground(String... u) {        
        try {                   
            URL ul = new URL(u[0]);
            Log.i("UI",ul.toString());
       //   int len = CopyOfMusicDownloader.mp3urls.size();
       //   URL url2 = new URL(CopyOfMusicDownloader.mp3urls.get(len-1));
            HttpURLConnection c = (HttpURLConnection) ul.openConnection();
            c.setRequestMethod("GET");
            c.setDoOutput(true);
            c.connect();

            int lengthOfFile = c.getContentLength();

            String PATH = Environment.getExternalStorageDirectory()
                    + "/download/";
            Log.v("", "PATH: " + PATH);
            File file = new File(PATH);
            file.mkdirs();

            fileName = "Track";
            filecount++;

            fileName =  fileName + Integer.toString(filecount) + ".mp3";


            File outputFile = new File(file, fileName);
            FileOutputStream fos = new FileOutputStream(outputFile);
            InputStream is = c.getInputStream();

            byte[] buffer = new byte[1024];
            int len1 = 0;      
            while ((len1 = is.read(buffer)) != -1) {
                myProgress = (int)((len1/lengthOfFile)*100);
                myProgress = myProgress + myProgress;
                Log.i("lengthOfFile", Integer.toString(lengthOfFile));
                Log.i("My Progress", Integer.toString(myProgress));
                publishProgress(myProgress);
                fos.write(buffer, 0, len1);
            }
            fos.close();
            is.close();

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

    protected void onPostExecute() {
    }

    @Override
    protected void onPreExecute() {         
          adapter = new MyCustomAdapter(CopyOfDownloadsListActivity.this, R.layout.row, CopyOfMusicDownloader.mp3urls);
          setListAdapter(adapter);
    }



    @Override
    protected void onProgressUpdate(Integer... values) {
         Log.i("Value", values[0].toString());
         count++;
         adapter.notifyDataSetChanged();
    }


    public class MyCustomAdapter extends ArrayAdapter<String> {     
        public MyCustomAdapter(Context context, int textViewResourceId, ArrayList<String> pthreads) {
        super(context, textViewResourceId, pthreads);
        }

        @Override
        public View getView(int position, View convertView, ViewGroup parent) {
                LayoutInflater inflater = getLayoutInflater();
                View row = inflater.inflate(R.layout.row, parent, false);
                bt =(ImageView)row.findViewById(R.id.cancel_btn);
                tv =(TextView)row.findViewById(R.id.filetext);
                pb = (ProgressBar)row.findViewById(R.id.progressbar_Horizontal);
                pb.setProgress(count);  
                return row;
            }
        }       
 }  

Это запуск AsyncTask и его создание новый DownloadFile (). execute (url2.toString ());

Ответы [ 2 ]

4 голосов
/ 17 августа 2011

Также, прочитайте примечания по execute () здесь:

http://developer.android.com/reference/android/os/AsyncTask.html#execute%28Params...%29

После Honeycomb будет только один поток, выполняющий все ASyncTasks, поэтому вы, вероятно, не сможете запустить более одного одновременно. После первого, остальные будут стоять в очереди и не будут выполняться вообще, пока не закончится первый. Вам, вероятно, потребуется выполнить каждую загрузку в потоке и использовать один ASyncTask для мониторинга всех из них. (Вы также можете отслеживать из потока, но вам нужно предпринять дополнительные шаги для безопасной публикации обновлений в пользовательском интерфейсе в потоке пользовательского интерфейса).

3 голосов
/ 17 августа 2011

Ваша проблема в этом static ProgressBar pb;

Вы не можете иметь одну статическую ссылку и ожидать управления несколькими индикаторами выполнения. Полностью инкапсулируйте ProgressBar внутри AsyncTask, сделайте его переменной экземпляра.

EDIT

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

@Override
    protected void onProgressUpdate(Integer... values) {
         Log.i("Value", values[0].toString());
         count++;
         progressBar.setProgress(count);
    }

Вы также никогда не назначаете значение progressBar, вы продолжаете оценивать pb, избавьтесь от этой переменной! В вашей PreExecute вы продолжаете переназначать адаптер Списка.

Вам нужно МНОГО модификаций. Создайте один адаптер, который управляет списком. Каждая строка может иметь AsyncTask, которая поддерживает свой собственный индикатор выполнения, представление изображения и представление текста.

...