Показать прогресс загрузки внутри активности с помощью DownloadManager

90

Я пытаюсь воспроизвести тот же прогресс, который DownloadManager показывает на панели уведомлений в моем приложении, но мой прогресс никогда не публикуется. Пытаюсь обновить с помощью runOnUiThread (), но почему-то не обновляется.

моя загрузка:

String urlDownload = "https://dl.dropbox.com/s/ex4clsfmiu142dy/test.zip?token_hash=AAGD-XcBL8C3flflkmxjbzdr7_2W_i6CZ_3rM5zQpUCYaw&dl=1";
DownloadManager.Request request = new DownloadManager.Request(Uri.parse(urlDownload));

request.setDescription("Testando");
request.setTitle("Download");
request.allowScanningByMediaScanner();
request.setNotificationVisibility(DownloadManager.Request.VISIBILITY_VISIBLE_NOTIFY_COMPLETED);
request.setDestinationInExternalPublicDir(Environment.DIRECTORY_DOWNLOADS, "teste.zip");

final DownloadManager manager = (DownloadManager) getSystemService(Context.DOWNLOAD_SERVICE);

final long downloadId = manager.enqueue(request);

final ProgressBar mProgressBar = (ProgressBar) findViewById(R.id.progressBar1);

new Thread(new Runnable() {

    @Override
    public void run() {

        boolean downloading = true;

        while (downloading) {

            DownloadManager.Query q = new DownloadManager.Query();
            q.setFilterById(downloadId);

            Cursor cursor = manager.query(q);
            cursor.moveToFirst();
            int bytes_downloaded = cursor.getInt(cursor
                    .getColumnIndex(DownloadManager.COLUMN_BYTES_DOWNLOADED_SO_FAR));
            int bytes_total = cursor.getInt(cursor.getColumnIndex(DownloadManager.COLUMN_TOTAL_SIZE_BYTES));

            if (cursor.getInt(cursor.getColumnIndex(DownloadManager.COLUMN_STATUS)) == DownloadManager.STATUS_SUCCESSFUL) {
                downloading = false;
            }

            final double dl_progress = (bytes_downloaded / bytes_total) * 100;

            runOnUiThread(new Runnable() {

                @Override
                public void run() {

                    mProgressBar.setProgress((int) dl_progress);

                }
            });

            Log.d(Constants.MAIN_VIEW_ACTIVITY, statusMessage(cursor));
            cursor.close();
        }

    }
}).start();

мой метод statusMessage:

private String statusMessage(Cursor c) {
    String msg = "???";

    switch (c.getInt(c.getColumnIndex(DownloadManager.COLUMN_STATUS))) {
    case DownloadManager.STATUS_FAILED:
        msg = "Download failed!";
        break;

    case DownloadManager.STATUS_PAUSED:
        msg = "Download paused!";
        break;

    case DownloadManager.STATUS_PENDING:
        msg = "Download pending!";
        break;

    case DownloadManager.STATUS_RUNNING:
        msg = "Download in progress!";
        break;

    case DownloadManager.STATUS_SUCCESSFUL:
        msg = "Download complete!";
        break;

    default:
        msg = "Download is nowhere in sight";
        break;
    }

    return (msg);
}

Мой журнал работает отлично, в то время как у меня идет загрузка, говорит "Загрузка идет!" и когда он завершает «Загрузка завершена!», но этого не происходит с моим прогрессом, почему? Мне действительно нужна помощь, другие логики, которые я действительно ценю

Виктор Лаэрт
источник
Может быть, ваш файл слишком мал и загрузка завершается до публикации хода выполнения? Что возвращает запрос на загрузку в вашей задаче? Если задача выполняется только через некоторое время, у вас может быть другая длительная операция в вашем основном потоке.
Paul Lammertsma
Я обновил код, можете теперь взглянуть? Что касается длины файла, то он не так уж и мал, я могу видеть прогресс загрузки на панели уведомлений
Виктор Лаэрт

Ответы:

62

Вы делите два целых числа:

final double dl_progress = (bytes_downloaded / bytes_total) * 100;

Как bytes_downloadedменьше bytes_total, (bytes_downloaded / bytes_total)будет 0, и поэтому ваш прогресс всегда будет 0.

Измените свой расчет на

final int dl_progress = (int) ((bytes_downloaded * 100l) / bytes_total);

для получения прогресса в целых (хотя и минимальных) процентилях.

Пол Ламмерцма
источник
@AZ_ Спасибо за ваш вклад. Я предлагаю вам добавить свой собственный ответ с более сложным решением.
Пол Ламмертсма
Ничего страшного, я не хочу ставить еще один ответ из уже принятого ответа, потому что это будет сложно для пользователей. Вы можете отказаться от принятия моих
правок
1
Если ваша активность завершится и вы захотите отменить загрузку, вы получите division by zeroфатальную ошибку. Вот почему мне это понравилось final int dl_progress = ( bytes_total > 0 ? (int) ((bytes_downloaded * 100L) / bytes_total) : 0 );
KaHa6uc
17

Ответ Пола правильный, но с большими загрузками вы довольно быстро достигнете max int и начнете получать отрицательный прогресс. Я использовал это для решения проблемы:

final int dl_progress = (int) ((bytes_downloaded * 100l) / bytes_total);
Джастин Моррис
источник
Ты прав; Я изменил свой ответ, чтобы убедиться, что другие не повторяют ту же ошибку.
Paul Lammertsma
5

В случае, если кому-то нужна реализация извлечения прогресса загрузки из вопроса @Victor Laerte в Kotlin с RxJava, вот вам:

DownloadStateRetriever.kt

class DownloadStateRetriever(private val downloadManager: DownloadManager) {

    fun retrieve(id: Long) {
        var downloading = AtomicBoolean(true)

        val disposable = Observable.fromCallable {
            val query = DownloadManager.Query().setFilterById(id)
            val cursor = downloadManager.query(query)

            cursor.moveToFirst()

            val bytesDownloaded = cursor.intValue(DownloadManager.COLUMN_BYTES_DOWNLOADED_SO_FAR)
            val bytesTotal = cursor.intValue(DownloadManager.COLUMN_TOTAL_SIZE_BYTES)

            if (isSuccessful(cursor)) downloading.set(false)
            cursor.close()

            if (bytesTotal == 0) 0.toInt() else ((bytesDownloaded * 100F) / bytesTotal).toInt()
        }
                .subscribeOn(Schedulers.newThread())
                .delay(1, TimeUnit.SECONDS)
                .repeatUntil { !downloading.get() }
                .subscribe {
                    Timber.i("Subscribed to $id. progress: $it")
                }
    }

    private fun isSuccessful(cursor: Cursor) = status(cursor) == DownloadManager.STATUS_SUCCESSFUL

    private fun status(cursor: Cursor) = cursor.intValue(DownloadManager.COLUMN_STATUS)
}

Я добавил расширения к курсору для большей ясности кода:

CursorExtensions.kt

import android.database.Cursor

fun Cursor.column(which: String) = this.getColumnIndex(which)
fun Cursor.intValue(which: String): Int = this.getInt(column(which))
fun Cursor.floatValue(which: String): Float = this.getFloat(column(which))
fun Cursor.stringValue(which: String): String = this.getString(column(which))
fun Cursor.doubleValue(which: String): Double = this.getDouble(column(which))

Рахимли
источник
4

Как сказал Пол, вы делите два целых числа, и результат всегда <1.

Всегда приводите свое число перед делением, которое вычисляется и возвращается в виде с плавающей запятой.

Не забудьте обработать DivByZero.

final int dl_progress = (int) ((double)bytes_downloaded / (double)bytes_total * 100f);
Деннис С
источник