使用WebClient进行文件下载时,用户界面冻结

时间:2015-11-04 09:45:44

标签: c# download webclient

我正在尝试使用WebClient.DownloadFileAsync-Method下载一些文件。 只要没有显示UI,它就能正常工作。

UI是一个带有Label和ProgressBar的表单。

在DownloadProgressChanged-Event中,我想显示当前的进度。 为了做到这一点,我调用一个带有int参数的方法。 这些是下载方法:

private void DownloadProgressChanged(object sender, DownloadProgressChangedEventArgs e)
{
    if(progressDialog!=null){
        progressDialog.setFileProgress(e.ProgressPercentage);
    }
    Trace.WriteLine(String.Format("downloaded {0} of {1} bytes. {2} % complete...", 
        e.BytesReceived, 
        e.TotalBytesToReceive,
        e.ProgressPercentage));
}


private void DownloadFileCompleted(object sender, AsyncCompletedEventArgs e)
{               
    if(progressDialog!=null){               
        progressDialog.setFileProgress(100);
    }
    are.Set();
}


private AutoResetEvent are = new AutoResetEvent(false);
public void DownloadFiles(List<DownloadObject> objects){
    Trace.WriteLine("Start Download");
    wc.DownloadProgressChanged += DownloadProgressChanged;
    wc.DownloadFileCompleted += DownloadFileCompleted;
    try{ 
        foreach(DownloadObject dlo in objects){                 
            currentFile = dlo;
            String url = dlo.DownloadURL;
            String path = dlo.LocalPath;
            Uri uri = new Uri(url);
            //GET
            Thread thread = new Thread(() => wc.DownloadFileAsync(uri,path));
            //thread.SetApartmentState(ApartmentState.STA);
            thread.Start();                 
            are.WaitOne();
            DeleteFile(dlo.ID);
        }               
        Trace.WriteLine("FileDownload finished");
    }catch(Exception ex){
        Trace.WriteLine("FileDownload failed: "+ex.Message);
    }finally{
        wc.Dispose();   
    }           
}

这些是ProgressDialog-Form中的相关方法:

public delegate void dummy();
public void setFileProgress(int progress){          
    if(prgFile.InvokeRequired){
        Trace.WriteLine("Invoke required");
        prgFile.Invoke(new dummy(() => prgFile.Value = progress));
    }else{              
        Trace.WriteLine("Invoke not required");             
        prgFile.Value = progress;
    }           
}

public static ProgressDialog getInstance(IWin32Window owner){
    ProgressDialog pd = new ProgressDialog();
    //pd.Show(owner); //
    return pd;
}

现在发生的是: 如果没有调用pd.Show(),一切都很好。进度发生了变化,我得到输出“不需要调用”以及下载的每一步。

但是如果调用pd.Show(),我会多次获得输出“Invoke required”,而不会有任何下载消息。

所以我通过代码的那一部分进行了调试,似乎调用了progressDialog.setFileProgress(),但是在调用prgFile.Invoke方法之后直接再次触发了DownloadProgressChanged-Event。

如果我将Invoke-call切换到BeginInvoke,我会再次获得所有正确的消息,但ProgressDialog将冻结,直到所有下载完成并且我没有显示任何进度。

我在那里错过了什么?我阅读了大量关于此的问题和线索,但无法使其运行。

我正在使用SharpDevelop和.Net-Framework 4.0

2 个答案:

答案 0 :(得分:0)

因为你的'DownloadFiles'方法是同步的,它会阻止UI线程。你开始一个新的线程下载文件,但

are.WaitOne();

阻止当前线程(UI线程),直到设置are为止(在下载完成事件中)。

正确的解决方案是在完成事件后下载(例如DeleteFile)后的那些工作。

答案 1 :(得分:0)

因为DownloadFiles - 方法是同步的并且在UI线程中运行。因此,如果您执行该方法,您的UI线程将被阻止,直到方法完成。

有两种方法:

<强>多线程

让Method在不同的任务中运行将解决你的UI-Freezing。但是,正如您所提到的,在您的情况下,这将是一种可怕的方法,因为您无法操纵UI跨线程。如果你想进行状态更新,你不应该这样做。

异步/等待

这里有更好的方法。虽然看起来很简单,但async / await实际上很难做到。互联网上有很多资源,我建议从dotnetperls开始,这是对async / await的详细介绍。这需要一些时间,但绝对是最好的方法。