이벤트 외부에서 이벤트 인수를 사용하는 방법

Question

먼저 프로그래밍에 대한 첫 번째 접근 방식으로 C#을 배우기 시작 했으므로 2 개월 전부터 코딩을 시작한 절대 초보자 인 hobbiest를 알려 주시기 바랍니다. 나랑.

그래서 무엇을 하려는지 사용자 정의 다운로드 응용 프로그램을 만드는 것입니다. 다운로드 버튼을 제외한 모든 버튼은 "DownloadProgressChangedEventArgs"에서 "percent1"변수를 선택할 수 없습니다. mainForm 생성자보다 먼저 인스턴스화했지만 변경된 값은 읽지 않습니다. 여기

는 대부분의 질문에 관련 밤은 이후 부분적으로 제거 코드,이다 :

public partial class Main : Form 
{ 
//Variables (not all, just the one im having issues with) 
    private double percentage1; 

//Main form constructor 
    public Main(){...} 

//Download File Async custom method 
    public void DldFile(string url, string fileName, string localPath, AsyncCompletedEventHandler completedName, DownloadProgressChangedEventHandler progressName) 
    { 
      WebClient webClient = new WebClient(); 
      webClient.DownloadFileAsync(new Uri(url), localPath + "\\" + fileName); 
      webClient.DownloadFileCompleted += new AsyncCompletedEventHandler(completedName); 
      webClient.DownloadProgressChanged += new DownloadProgressChangedEventHandler(progressName); 
    } 

//Button 1 click event to start download 
    private void btnDld1_Click(object sender, EventArgs e) 
    { 
     if (url1 != "" && Directory.Exists(localPath1)) 
     { 
      _startDate1 = DateTime.Now; 
      DldFile(url1, fileName1, localPath1, completed1, progress1); 
     } 
     //took out the try/catch, other ifs to try and cut it down 
    } 

//Download Progress Changed event for Download 1 
    public void progress1(object sender, DownloadProgressChangedEventArgs e) 
    { 
     percentage1 = e.ProgressPercentage; //THIS IS WHERE I WAS EXPECTING TO UPDATE "percentage1" 
     progressBar1.Value = int.Parse(Math.Truncate(percentage1).ToString()); 
    } 

//Button that starts all downloads click event where all my problems are at the moment 
    private void btnDldAll_Click(object sender, EventArgs e) 
    { 
     //The progress bar that should let me know the global status for all webClients 
     progressBarAll.Value = (
      int.Parse(Math.Truncate(percentage1).ToString()) + //HERE IS MY PROBLEM 
      int.Parse(Math.Truncate(percentage2).ToString()) + //HERE IS MY PROBLEM 
      int.Parse(Math.Truncate(percentage3).ToString()) + //HERE IS MY PROBLEM 
      int.Parse(Math.Truncate(percentage4).ToString()) + //HERE IS MY PROBLEM 
      int.Parse(Math.Truncate(percentage5).ToString()))/5; //HERE IS MY PROBLEM 

     //Checks if the link exists and starts it from the download button click event 
     if (url1 != "") 
     { 
      btnDld1.PerformClick(); 
     } 
     //Continues for url2, 3, 4, 5 and else 
    } 
} 

그래서 내가 당신을시키는의 발견 가장 짧은 방법입니다 메신저 누락 된 뭔가가 있다면, 해 내려고 알고 알려 주시기 바랍니다, 최대한 빨리 모든 정보를 추가하려고합니다.

"progress1"을 인스턴스화하여 percentage1 변수에 액세스하려고 시도했지만 작동하지 않았습니다. 나는 webClient와 똑같은 일을 시도했지만 시도도하지 않았다. 나는 구글과 stackflow 검색을 아무 소용이 사용했습니다. 그래서 질문이 너무 멍청하거나, 내 사고 방식에서 완전히 그 문제를 보는 다른 방법이 있는지 확실하지 않습니다.

그래서 주된 문제는 "percentage1"변수를 업데이트하고 사용하는 것입니다. "progressBarAll.Value"계산과 관련하여 다른 문제가 있습니다. 올바른 값으로 손을 잡을 수있을 때 해결됩니다. 그렇게 보더라도 걱정할 필요가 없습니다.

긴 게시물에 대한 미안하지만, 문제가 어디에 있는지 잘 모르겠다. 그래서 나는 다른 곳에서 엉망이되지 않도록 더 많은 정보를 제공해야했다.

해피 코딩 D

Answer 1

'이벤트 외부의 이벤트 인수 사용'에 대해 생각하지 마십시오. 양식의 상태를 업데이트하는 방법에 대해 생각해보십시오. 나는 더욱 코드를 리팩토링 것

public partial class Main : Form 
{ 
    private double percentage1; 
    private double percentage2; 
    private double percentage3; 
    private double percentage4; 
    private double percentage5; 

    private double Percentage1 
    { 
    get 
    { 
     return this.percentage1; 
    } 
    set 
    { 
     this.percentage1 = value; 
     this.UpdatePercentageAll(); // this will update overall progress whenever the first one changes 

     progressBar1.Value = GetValueFromPercentage(value); 
    } 
    } 
    private double Percentage2 
    // same code as for Percentage1 

    void UpdatePercentageAll() 
    { 
    this.PercentageAll = (this.Percentage1 + this.Percentage2 + this.Percentage3 + this.Percentage4 + this.Percentage5)/5; 
    } 

    static int GetValueFromPercentage(double percentage) 
    { 
    return (int)Math.Truncate(percentage); 
    } 

    double percentageAll; 
    private double PercentageAll 
    { 
    get 
    { 
     return this.percentageAll; 
    } 
    set 
    { 
     this.percentageAll = value; 

     progressBarAll.Value = GetValueFromPercentage(value); 
    } 
    } 

    //Download File Async custom method 
    public void DldFile(string url, string fileName, string localPath, AsyncCompletedEventHandler completedName, DownloadProgressChangedEventHandler progressName) 
    { 
    WebClient webClient = new WebClient(); 
    webClient.DownloadFileAsync(new Uri(url), localPath + "\\" + fileName); 
    webClient.DownloadFileCompleted += new AsyncCompletedEventHandler(completedName); 
    webClient.DownloadProgressChanged += new DownloadProgressChangedEventHandler(progressName); 
    } 

    //Button 1 click event to start download 
    private void btnDld1_Click(object sender, EventArgs e) 
    { 
    if (url1 != "" && Directory.Exists(localPath1)) 
    { 
     this.StartDownloadFile1(); 
    } 
    //took out the try/catch, other ifs to try and cut it down 
    } 
    void StartDownloadFile1() 
    { 
     this.Percentage1 = 0; 
     _startDate1 = DateTime.Now; 
     DldFile(url1, fileName1, localPath1, completed1, progress1); 
    } 
    //Download Progress Changed event for Download 1 
    public void progress1(object sender, DownloadProgressChangedEventArgs e) 
    { 
    this.Percentage1 = e.ProgressPercentage; // update property, not field 

    //this will be done in property setters 
    //progressBar1.Value = int.Parse(Math.Truncate(percentage1).ToString()); 
    } 
    // then add similar code for other download buttons 

    //Button that starts all downloads click event where all my problems are at the moment 
    private void btnDldAll_Click(object sender, EventArgs e) 
    { 
    //Checks if the link exists and starts it from the download button click event 
    if (url1 != "") 
    { 
     this.StartDownloadFile1(); 
    } 
    //Continues for url2, 3, 4, 5 and else 
    } 
} 

,하지만 난 그것을 코드가 원본에 가까운 경우에 당신이 이해하기 쉽게 될 것이라고 생각 :

를 사용하여 속성을 업데이트 논리를 단순화합니다.

주요 아이디어는 수학적 기능처럼 작동하는 일련의 연결된 속성을 만드는 것입니다. PercentageX 속성을 작성할 때 'PercentageAll을 모든 백분율의 평균으로합시다.'라고 말하는 것이 좋습니다. 그런 다음 각 다운로드가 자체 진행 상태로 업데이트됩니다. 진행률이 업데이트되면 평균값이 업데이트되고 이고 진행 변경 이벤트 핸들러 내에을 기억할 필요가 없습니다.

마지막으로 백분율 속성에서 진행률 막대를 업데이트하고 있습니다. 아주 간단합니다. 비율이 변경되면 바를 업데이트해야합니다. 그렇다면, 왜 사방이 Percentage1을 변경하면 내가 막대를 업데이트 할 필요가 있음을 기억해야합니다이 경우

this.Percentage1 = x; 
this.progressBar1.Value = (int)Math.Truncate(x); 

같은 것을 쓰고 귀찮게. 그리고 나의 예에서는 단지 한 곳에서만 사용되고 매번 작동하는 엄격한 규칙을 만듭니다. 그래서 나는 그것을 잊을 수 없다. 규칙을 변경해야하는 경우 한 곳만 변경해야하므로 다시 실수를 할 수 없습니다.

내가 보여주는 기술은 잘 알려진 규칙으로 표현할 수 있습니다. '하나의 규칙 - 한 곳'. 프로그램에 존재하는 각 논리 규칙을 나타내는 코드에서 한 곳만 사용해야합니다. 그것은 매우 중요한 아이디어입니다, 당신이 배우고 그것을 사용하는 것이 좋습니다.