下载后WebClient冻结

我面临另一个问题，我的应用程序从网上下载文件，提取它，删除它等，它运行良好的第一次运行，然后当涉及到下载下一个文件它只是冻结下载并永远挂在那里..这可能是尝试打开一个已经打开的连接的东西，但我不知道如何关闭它，这是我第一次与C＃联网，而且我是自学的。下载后WebClient冻结

我的代码：

public void start() { 
     if (File.Exists("Data/version.txt")) { File.Delete("Data/version.txt"); } 
     label1.Text = "Getting update information..."; 
     WebClient webClient = new WebClient(); 
     webClient.DownloadFileAsync(new Uri("http://127.0.0.1/update/version.txt"), @"Data/version.txt"); 
     webClient.DownloadFileCompleted += new AsyncCompletedEventHandler(versionCompleted); 
    } 

    private void versioncheck() { 
     if (File.Exists("Main.exe")) 
     {  
      label1.Text = "Contacting update server..."; 
       var versionInfo = FileVersionInfo.GetVersionInfo("Main.exe"); 
       string version = versionInfo.ProductVersion; 
       string[] nversion = version.Split('.'); 
       string updateversion = nversion[3]; 

       int version = Int32.Parse(updateversion); 
       /////////////////////////////////////////////////////////// 
       StreamReader sr = new StreamReader("Data/version.txt"); 
       /////////////////////////////////////////////////////////// 
       string readline = sr.ReadLine(); 
       sr.Dispose(); 
       int serverversion = Int32.Parse(readline); 

       if (serverversion > version) { 
        string filenumber = (version+1).ToString(); 
        downloadfile(filenumber); 
       } 
       else if(serverversion == version){ 
        label1.Text = "Game is up to date!"; 
        startButton.Enabled = true; 
       } 

     } 
     else { MessageBox.Show("Unexpected Error!", "Error!"); } 
    } 

    private void ProgressChanged(object sender, DownloadProgressChangedEventArgs e) 
    { 
     progressBar1.Value = e.ProgressPercentage; 
    } 

    private void Completed(object sender, AsyncCompletedEventArgs e) 
    { 
     label1.Text = "Extracting Files..."; 
     var versionInfo = FileVersionInfo.GetVersionInfo("Main.exe"); 
     string version = versionInfo.ProductVersion; 
     string[] nversion = version.Split('.'); 
     string updateversion = nversion[3]; 
     int version = Int32.Parse(updateversion); string nversion = (version + 1).ToString(); 

     Process proc = Process.Start("update"+nversion+".exe"); // extract in the silent mode 
     proc.WaitForExit(); 
     File.Delete("update" + nversion + ".exe"); 
     label1.Text = "Checking for more updates..."; 
     versioncheck(); 
    } 

    private void versionCompleted(object sender, AsyncCompletedEventArgs e) { 
     versioncheck(); 
    } 

    private void downloadfile(string filenumber) 
    { 
     try 
     { 
      //MessageBox.Show("Download working"); 
      System.Net.WebClient webClient = new System.Net.WebClient(); 
      webClient.OpenRead("http://127.0.0.1/Update/update" + filenumber + ".exe"); 
      Int64 bytes_total = Convert.ToInt64(webClient.ResponseHeaders["Content-Length"]); 
      string updatelength = Convert.ToString((bytes_total/1024).ToString()); 
      label2.Text = "File size:" + updatelength + "KB"; 
      //////////////////////////////////////////////////// 
      label1.Text = "Downloading Update..."; 
      webClient.DownloadProgressChanged += new DownloadProgressChangedEventHandler(ProgressChanged); 
      webClient.DownloadFileAsync(new Uri("http://127.0.0.1/Update/update" + filenumber + ".exe"), @"update"+filenumber+".exe"); 
      webClient.DownloadFileCompleted += new AsyncCompletedEventHandler(Completed); 
      webClient.Dispose(); 

     } 
     catch (WebException) 
     { 
      MessageBox.Show("Connection error!", "Error!"); 
      Application.Exit(); 
     } 
     catch (IOException) 
     { 
      MessageBox.Show("Unknown Error!", "Error!"); 
      Application.Exit() ; 
     }  
    }

来源

2014-01-08 Reousa Asteron

你可以断点到versionCheck（）吗？它到达那里了吗？第一次下载完成后，您的表单是否被锁定？我不记得了，但如果是这种情况，'DownloadFileCompleted'事件可能会回调到主窗体线程中。另外，尝试将'DownloadFileCompleted'调用移动到'DownloadFileAsync'上方的行。考虑到文件可能很大，这可能不会做任何事情，但无论如何这是一个好主意。 – Brandon

此外，您将重新声明变量作为不同类型的地方。 – Brandon

是的，我认为重新宣布它们会更容易，而不是用不同的名称创建新的，只是为了不会迷路，我相信当我重新声明它们时，它会摆脱旧的，是我错误？请纠正我的错误，如果你可能:) 而一切正常，事情是它应该下载3个文件连续，它只下载一个，当它开始下载第二个，它显示它的大小= 0字节在磁盘上，直到我重新启动它。 –

如果你使用异步操作，你需要使用async和await关键字。

当完成下载或失败时，您需要处理或关闭您的web客户端。

由于b1tsh1ft声明最好的事情是使用using语句。

string version = versionInfo.ProductVersion; 
string[] nversion = version.Split('.'); 

int version = Int32.Parse(updateversion); 
string nversion = (version + 1).ToString();

是不是你的VS没有在编辑器中给出一个矛盾的错误呢？

来源

2014-01-08 19:46:25 Jim

不是没有，这让我觉得它是像PHP ，新的价值超越了旧的价值，我知道现在你们都说它不行。我会编辑它，看看它是如何工作的。 –

首先，你应该把Web客户端在使用块，因为它实现IDisposable

using(var webClient = new WebClient()) 
{ 
    // do work here 
    webClient.DownloadFile(..) 
}

不要使用异步版本。一个异常可能被抛出，并在另一个线程上丢失。首先定期测试并定期开展工作。

还将您的StreamReader（或任何实现IDisposable的内容）放入using（）语句中。手动调用处置会更可靠，因为即使在失败时也能处理。

来源

2014-01-08 19:21:10 SlaterCodes

这是否意味着我应该把使用块之后代码的提取部分？我尝试做不同的事情的主要问题是提取在下载完成之前开始，这就是为什么我使用异步，谢谢你的信息，虽然，我会试试:) –

下载后WebClient冻结

回答

相关问题