Pokaż postęp pobierania w aktywności za pomocą DownloadManagera

90

Próbuję odtworzyć ten sam postęp, jaki pokazuje DownloadManager na pasku powiadomień w mojej aplikacji, ale mój postęp nigdy nie jest publikowany. Próbuję zaktualizować go za pomocą runOnUiThread (), ale z jakiegoś powodu nie został zaktualizowany.

moje pobieranie:

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();

my statusMessage metoda:

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);
}

Mój dziennik działa idealnie, podczas gdy moje pobieranie jest uruchomione, wyświetla komunikat „Trwa pobieranie!” a kiedy zakończy się „Pobieranie zakończone!”, ale to samo nie dzieje się z moim postępem, dlaczego? Naprawdę potrzebuję pomocy, inne logiki są naprawdę doceniane

Victor Laerte
źródło
Czy to możliwe, że plik jest po prostu za mały, a pobieranie kończy się przed opublikowaniem postępu? Co zwraca zapytanie o pobranie w Twoim zadaniu? Jeśli zadanie zostanie wykonane dopiero po pewnym czasie, możesz mieć inną długotrwałą operację w głównym wątku.
Paul Lammertsma
Zaktualizowałem kod, możesz teraz rzucić okiem? A co do długości pliku nie jest za mały, postęp pobierania widzę na pasku powiadomień
Victor Laerte

Odpowiedzi:

62

Dzielisz dwie liczby całkowite:

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

Ponieważ bytes_downloadedjest mniejsze niż bytes_total, (bytes_downloaded / bytes_total)będzie równe 0, a zatem Twój postęp zawsze będzie wynosił 0.

Zmień obliczenia na

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

aby uzyskać postęp w całych percentylach (aczkolwiek pozbawionych).

Paul Lammertsma
źródło
@AZ_ Dzięki za Twój wkład. Proponuję dodać własną odpowiedź z bardziej rozbudowanym rozwiązaniem.
Paul Lammertsma
W porządku Nie chcę umieszczać kolejnej odpowiedzi już zaakceptowanej odpowiedzi, ponieważ będzie to trudne dla użytkowników. Możesz zrezygnować z akceptacji mojej zmiany :)
AZ_
1
Jeśli Twoja aktywność zakończy się i zechcesz anulować pobieranie, pojawi się division by zerobłąd krytyczny. Dlatego zrobiłem to tak, jak final int dl_progress = ( bytes_total > 0 ? (int) ((bytes_downloaded * 100L) / bytes_total) : 0 );
KaHa6uc
17

Odpowiedź Paula jest poprawna, ale przy większych pobraniach osiągniesz maksymalny int dość szybko i zaczniesz uzyskiwać negatywny postęp. Użyłem tego do rozwiązania problemu:

final int dl_progress = (int) ((bytes_downloaded * 100l) / bytes_total);
JustinMorris
źródło
Masz rację; Zmodyfikowałem odpowiedź, aby upewnić się, że inni nie popełnią tego samego błędu.
Paul Lammertsma,
4

Jak powiedział Paweł, dzielisz dwie liczby całkowite, z wynikiem zawsze <1.

Zawsze rzucaj swoją liczbę przed dzieleniem, które oblicza i zwraca w postaci zmiennoprzecinkowej.

Nie zapomnij obsłużyć DivByZero.

final int dl_progress = (int) ((double)bytes_downloaded / (double)bytes_total * 100f);
Dennis C
źródło
4

W przypadku, gdy ktoś potrzebuje zaimplementowania narzędzia pobierania postępu pobierania z pytania @Victor Laerte w Kotlin z RxJava, tutaj:

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)
}

Dodałem rozszerzenia do kursora dla większej przejrzystości kodu:

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))

rahimli
źródło