1
public void StartDownloading()
{
    File file = null;

    for(int i=0; i<FileNames.length; i++) 
    {
         file = new File("/sdcard/Sounds/"+FileNames[i]);
         boolean exists = file.exists();
         if(exists)
         {
             continue;
         }
         else 
         {
             new DownloadFileAsync().execute(Path+FileNames[i],FileNames[i]);               
         }
         file = null;
     }               
}

@Override
protected Dialog onCreateDialog(int id) 
{
    switch (id) 
    {
        case DIALOG_DOWNLOAD_PROGRESS:
            WaitDialog = new ProgressDialog(this);
            WaitDialog.setMessage("Downloading files...");
            WaitDialog.setProgressStyle(ProgressDialog.STYLE_SPINNER);
            WaitDialog.setCancelable(false);
            WaitDialog.show();
            return WaitDialog;
        default:
            return null;
    }
}

class DownloadFileAsync extends AsyncTask<String, String, String> {

    @Override
    protected void onPreExecute() 
    {
        super.onPreExecute();
        showDialog(DIALOG_DOWNLOAD_PROGRESS);
    }

    @Override
    protected String doInBackground(String... aurl) {
        int count;

        try 
        {
            URL url = new URL(aurl[0]);
            URLConnection conexion = url.openConnection();
            conexion.connect();

            int lenghtOfFile = conexion.getContentLength();

            InputStream input = new BufferedInputStream(url.openStream());
            File file = new File("/sdcard/Sounds/"+aurl[1]);
            OutputStream output = new FileOutputStream(file);
            byte data[] = new byte[4046848];

            long total = 0;

            while ((count = input.read(data)) != -1) {
                total += count;
                publishProgress(""+(int)((total*100)/lenghtOfFile));
                output.write(data, 0, count);
            }

            output.flush();
            output.close();
            input.close();

        } catch (Exception e) {}

        return null;
    }

    @Override
    protected void onPostExecute(String unused) 
    {
        dismissDialog(DIALOG_DOWNLOAD_PROGRESS);         
    }
}

При этом я пытаюсь загрузить 4 файла суммарным весом 2.71 мб, прогресс бар появляется, но обычно быстро пропадает, и файлы остаются загруженные по 5-30кб, иногда даже появляются 1-2 файла тоже по 5-30кб.

16
  • Подозреваю, что в doInBackground() выбрасывается исключение, которое успешно отлавливается catch'ем. Попробуйте в блоке catch сделать вывод исключения в лог и просмотрите его потом. 28 мар 2011 в 15:45
  • Была ошибка Out of memory on a 10046864-byte allocation. уменьшил. Эффект тотже, кроме как ошибки нету.
    – AndroidDev
    28 мар 2011 в 16:03
  • А разрешение на использование INTERNET есть?
    – Dex
    28 мар 2011 в 16:09
  • Если он хоть маленькие кусочки грузит. Полюбому есть
    – AndroidDev
    28 мар 2011 в 16:13
  • 1
    Зачем вы вообще такой гигантский буфер делаете? В этом нет никакого смысла: вы никогда не прочтёте за раз больше, чем размер буффера BufferedInputStream (по умолчанию он 8192). Кроме того, вообще нет смысла делать его таким гигантским: при чтении из сети достаточно 4096 или 8192. Больше нет вообще смысла, это пустая трата ресурсов.
    – cy6erGn0m
    28 мар 2011 в 16:13

1 ответ 1

1

Чтобы отловить завершение всех загрузок я бы написал что-то типа такого

List<File> filesToBeDownloaded = getNonExistFiles();

CountDownLatch latch = new CountDownLatch(filesToBeDownloaded.size());
for(File f : filesToBeDownloaded) {
    new FileDownloadTask().execute(f.getName(), latch);
}

new DlAwaitTask().execute(latch);

/// ...

class FileDownloadTask extends AsyncTask<Void, String, CountDownLatch> {
    // ...
    @Override
    protected Void doInBackground(String fileName, CountDownLatch latch) {
         try {
              // perform download
         } ...
         } finally {
             latch.countDown();
         }
         }
     }
}

class DLAwaitTask exetends AsyncTask<Void, CountDownLatch> {

     //.....
     @Override
    protected Void doInBackground(String fileName, CountDownLatch latch) {
         try {
             if(latch.await(MY_DEFAULT_TIMEOUT, TimeUnit.SECONDS)) 
                 // download complete.. notify UI
             else
                 // timeout.. show error, etc
         } ....
         } finally {
              // destroy all anyaway
         }
}

Разумеется это шаблон, куда вы можете вписать свой код. Кроме того, не мешает копнуть в андроидное API. Возможно найдётся способ запустить эти таски через общий пул потоков и подождать завершения всех задач каким-то более халявным способом.

12
  • Неразобрался со 2 частью
    – AndroidDev
    28 мар 2011 в 20:41
  • С DLAwaitTask? Мы запускаем эту таску.. она ждёт latch пока он не сработает (это случится, когда счётчик сойдёт на ноль). Пока он не дойдёт до нуля, поток будет заблокирован в latch.await и будет спать. Он снимется с блокировки, если лотч обнулится, случится таймаут или interrupt. Каждая задача на загрузку заканчивается latch.countDown тем самым сообщая что на одну работающую задачу меньше.
    – cy6erGn0m
    28 мар 2011 в 20:50
  • Помойму слишком заморочено, нам всеголиш нужно определить когда все файлы загружены и убрать с экрана прогресбар
    – AndroidDev
    28 мар 2011 в 20:56
  • Всего лишь? У вас N параллельных потоков. Их надо так или иначе синхронизовать. У вас почти нет вариантов. Разве что запускать загрузки в порядке очереди, а не все сразу. К тому же, где вы тут увидели что-то замороченное? Тут добавилась совершенно крошечная таска и добавилось буквально три строки в остальном коде. Вот создавать буффер для простой сетевой операции размером в 4 мегабайта - вот где замороченно ;)
    – cy6erGn0m
    28 мар 2011 в 21:13
  • Хм ок попробывал, есть проблемы... Incorrect number of arguments for type AsyncTask<Params,Progress,Result>; it cannot be parameterized with arguments <Void, CountDownLatch> И насчет этого List<File> filesToBeDownloaded = getNonExistFiles(); CountDownLatch latch = new CountDownLatch(filesToBeDownloaded.size()); Интересно как их получить? Может быть просто заранее указать известный мне размер?
    – AndroidDev
    28 мар 2011 в 21:18

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.