AsyncTask onPostExecute called before doInBackground

Question

I've this problem with this AsyncTask class:

public class DownloadPDFTask extends AsyncTask<String, Void, File> {

    private Context mContext;
    private String mDownloadDIR = Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_DOWNLOADS).toString();

    public DownloadPDFTask(Context mContext) {
        this.mContext = mContext;
    }

    @Override
    protected synchronized void onPreExecute() {
        super.onPreExecute();
        removeOldFiles();
    }

    @Override
    protected synchronized File doInBackground(String... args) {
        File retFile = null;
        downloadFilePDF(args[0]);
        File directory = new File(mDownloadDIR);
        File[] files = directory.listFiles();
        for (File file : files) {
            if (file.getName().contains(".pdf")) {
                retFile = file;
                break;
            }
        }
        return retFile;

    }

    @Override
    protected synchronized void onPostExecute(File file) {
        super.onPostExecute(file);
        NotificationManager mNotificationManager = (NotificationManager) mContext.getSystemService(Context.NOTIFICATION_SERVICE);
        mNotificationManager.cancelAll();
        Intent mGoToPDFView = new Intent(mContext, PDFViewerActivity.class);
        mGoToPDFView.putExtra("FILE", file);
        mGoToPDFView.setFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
        mContext.startActivity(mGoToPDFView);
    }

    private void downloadFilePDF(String url) {
        DownloadManager mDownloadManager = (DownloadManager) mContext.getSystemService(Context.DOWNLOAD_SERVICE);
        Uri mUri = Uri.parse(url);
        DownloadManager.Request mReq = new DownloadManager.Request(mUri);
        mReq.setTitle("Develop Test");
        mReq.setDescription("PDF downloading....");
        mReq.setNotificationVisibility(DownloadManager.Request.VISIBILITY_VISIBLE);
        mReq.setDestinationInExternalPublicDir(Environment.DIRECTORY_DOWNLOADS, "temp.pdf");
        mReq.setMimeType("application/pdf");
        mDownloadManager.enqueue(mReq);
    }


    private void removeOldFiles() {
        File mDirectory = new File(mDownloadDIR);
        if (mDirectory.isDirectory()) {
            String[] children = mDirectory.list();
            for (String child : children) {
                if (child.contains(".pdf")) {
                    new File(mDirectory, child).delete();
                }
            }
        }
    }
}

I want to clear download folder, download PDF and the open other activity with file founded passed via intent, if I use debug, and go slowly the process work, but at runtime, onPostExecute method it's called before doInBackground method.

Can you help me?

Answer 1

Thanks to @CommonsWare for input!

I solve with this solution, I change AsyncTask:

public class DownloadPDFTask extends AsyncTask<String, Void, Void> {

    @Override
    protected synchronized void onPreExecute() {
        String mDownloadDIR = Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_DOWNLOADS).toString();
        super.onPreExecute();
        File mDirectory = new File(mDownloadDIR);
        if (mDirectory.isDirectory()) {
            String[] children = mDirectory.list();
            for (String child : children) {
                if (child.contains(".pdf")) {
                    new File(mDirectory, child).delete();
                }
            }
        }
    }

    @Override
    private void downloadFilePDF(String url) {
        DownloadManager mDownloadManager = (DownloadManager) mContext.getSystemService(Context.DOWNLOAD_SERVICE);
        Uri mUri = Uri.parse(url);
        DownloadManager.Request mReq = new DownloadManager.Request(mUri);
        mReq.setTitle("Develop Test");
        mReq.setDescription("PDF downloading....");
        mReq.setNotificationVisibility(DownloadManager.Request.VISIBILITY_VISIBLE);
        mReq.setDestinationInExternalPublicDir(Environment.DIRECTORY_DOWNLOADS, "temp.pdf");
        mReq.setMimeType("application/pdf");
        mDownloadManager.enqueue(mReq);
        return null;
    }

    @Override
    protected synchronized void onPostExecute(Void aVoid) {
        super.onPostExecute(aVoid);
        NotificationManager mNotificationManager = (NotificationManager) BaseApplication.getAppContext().getSystemService(Context.NOTIFICATION_SERVICE);
        mNotificationManager.cancelAll();
    }
}

And I'm to use onPostExecute, but I'm wait DownloadManager with a BroadcastReceiver:

public class DownloadReceiver extends BroadcastReceiver {

    private String mDownloadDIR = Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_DOWNLOADS).toString();
    private Context mContext = BaseApplication.getAppContext();

    @Override
    public void onReceive(Context context, Intent intent) {
        String action = intent.getAction();
        if (DownloadManager.ACTION_DOWNLOAD_COMPLETE.equals(action)) {
            Log.i("DownloadReceiver", "DownloadCompleted!");
            File retFile = null;
            File directory = new File(mDownloadDIR);
            File[] files = directory.listFiles();
            for (File file : files) {
                Log.i("DownloadReceiver", "Searching file in DOWNLOADS");
                if (file.getName().contains(".pdf")) {
                    Log.i("DownloadReceiver", "File found!");
                    retFile = file;
                    break;
                }
            }

            if (retFile != null) {
                Log.i("DownloadReceiver", "Go to PDF viewing...");
                Intent mGoToPDFView = new Intent(mContext, PDFViewerActivity.class);
                mGoToPDFView.putExtra("FILE", retFile);
                mGoToPDFView.setFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
                mContext.startActivity(mGoToPDFView);
            }

        }
    }
}

And after AndroidManifest.xml updating... it works!