0

我有一个正在运行的下载功能。但是当我运行它时，就像 80% 的时间它让我的手机滞后，强制关闭，很长一段时间没有响应，比如 1~2 分钟。这种情况非常随机发生，我无法真正追踪到问题所在。下载完成后设备将恢复正常。我尝试过各种设备，例如galaxy S2、galaxy note、SE xperia Arc S 和几张桌子。问题依旧。谁能建议我如何改进我的代码？以下是我现有的代码：

public void onClickDownload(View view){
        String url = "http://www.mydomain.com./" + fileURL;
        url = url.replaceAll(" ","%20");
        String sourceUrl = url;
        new DownloadFileAsync().execute(sourceUrl);
    }   

public class DownloadFileAsync extends AsyncTask<String, Integer, Void> {
        private boolean run_do_in_background = true;

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

    @Override
    protected Void doInBackground(String... aurl) {
        int count;
        try {
            URL url = new URL(aurl[0]);
            URLConnection conexion = url.openConnection();
            conexion.connect();

            int lengthOfFile = conexion.getContentLength();
            Log.d("ANDRO_ASYNC", "Lenght of file: " + lengthOfFile);

            File folder = new File(Environment.getExternalStorageDirectory() + "/MaxApps");
            boolean success = false;
            if (!folder.exists()) {
                success = folder.mkdirs();
            }
            if (!success) {
            } else {
            }

            InputStream input = new BufferedInputStream(url.openStream());
            OutputStream output = new FileOutputStream("/sdcard/MaxApps/" + apkURL);

            byte data[] = new byte[100*1024];

            long total = 0;

            while ((count = input.read(data)) != -1) {
            total += count;             
            int progressPercent = (int) ((total*100)/lengthOfFile);
            if(progressPercent % 5 == 0){  
                publishProgress(progressPercent);
                }
            output.write(data, 0, count);
            }

            output.flush();
            output.close();
            input.close();
            } catch (Exception e) {

                notificationManager.cancel(Integer.parseInt(ID.toString()));

                Notification MyN = new Notification(); MyN.icon = R.drawable.logo1;
                MyN.tickerText = "Download Failed";
                MyN.number = 1;
                MyN.setLatestEventInfo (getApplicationContext(), apkURL + " Download Failed.", "Please try again", MyPI);
                MyN.flags |= Notification.FLAG_AUTO_CANCEL;
                MyNM.notify(1, MyN);    
                run_do_in_background = false;
            }

        return null;
    }

    @Override
    protected void onProgressUpdate(Integer... progress) {          
        notification.contentView.setProgressBar(R.id.pbStatus, 100, progress[0], false);
        notificationManager.notify(Integer.parseInt(ID.toString()), notification);
        }

    @Override
    protected void onPostExecute(Void unused) {
        if(run_do_in_background) {
        notificationManager.cancel(Integer.parseInt(ID.toString()));

        Notification MyN = new Notification(); MyN.icon = R.drawable.logo1;
        MyN.tickerText = "Download Complete";
        MyN.number = 1;
        MyN.setLatestEventInfo (getApplicationContext(), "Download Complete, Click to install.", apkURL, MyPI);
        MyN.flags |= Notification.FLAG_AUTO_CANCEL;
        MyNM.notify(Integer.parseInt(ID.toString()) , MyN);
        }
    }
}
4

1 回答 1

0

可能是更新 UI 需要很长时间？也许您可以尝试像这样进行测试，看看它是否有所作为：

@Override
protected void onProgressUpdate(Integer... progress) {          
    Log.d("ANDRO_ASYNC", "Progress: " + progress[0]);
}

顺便说一句，这与您的要求无关，但我认为您在处理进度的代码中存在问题：

int progressPercent = (int) ((total*100)/lengthOfFile);
if(progressPercent % 5 == 0){  
     publishProgress(progressPercent);
}

只有当进度正好是 5、10、15、20% 等时才会更新进度……我猜你实际上想在进度比以前至少 %5 时更新进度。

int previousProgress = 0;
while ((count = input.read(data)) != -1) {
    total += count;             
    int progressPercent = (int) ((total*100)/lengthOfFile);
    if(progressPercent - previousProgress >= 5) {  
        previousProgress = progressPercent;
        publishProgress(progressPercent);
    }
    output.write(data, 0, count);
}
于 2012-07-05T10:19:39.583 回答