Win8アプリのタイマーカウント

Question

3つのボタン（Button1、Button2、StartButton）とタイマーがあるwin8アプリをビルドします。 Button1とButton2は無効です。 StartButtonをクリックすると、Button1が有効になり、20秒以内のクリック数がカウントされ、テキストブロック1に表示されます。タイマーが終了するとButton1が無効になり、Button2が有効になり、クリック数がカウントされてtextblock2に表示されます。私の問題は、Button1ではなくButton2でタイマーが正しく動くことです。 button2を有効にすると、タイマーが速くなります。誰か助けてくれますか？ 私のコードは以下の通りです：

private int count1=0; 
    private int count2=0; 
    private int clickCounter = 0; 
    private int timeLeft; 
    private DispatcherTimer timer; 

    private void StartTimer() 
    { 
     if (this.timer != null) 
     { 
      this.StopTimer(); 
     } 
     DispatcherTimer timer = new DispatcherTimer(); 
     timer.Interval = new TimeSpan(0,0,0,1); 
     timer.Tick += timer_Tick; 
     timer.Start(); 

    } 

    private void StopTimer() 
    { 
     if (this.timer != null) 
     { 
      this.timer.Stop(); 
      this.timer = null; 
     } 
    } 

    public void timer_Tick(object sender, object args) 
    { 

     if (timeLeft > 0) 
     { 
      timeLeft = timeLeft - 1; 
      timerTextBlock.Text = Convert.ToString(timeLeft); 
      this.StartButton.IsEnabled = false; 
     } 
     else 
     { 
      StopTimer(); 
      if (clickCounter==2) 
      { 
       ShowResult(); 
       this.Button2.IsEnabled = false; 
       this.StartButton.IsEnabled = false; 
      } 
      else 
      { 
       myMsg.Text = "Time's up!"; 
       this.Button1.IsEnabled = false; 
       this.StartButton.IsEnabled = true; 
      } 
     } 
    } 

    private void Button1_Click(object sender, Windows.UI.Xaml.RoutedEventArgs e) 
    { 
     // TODO: Add event handler implementation here. 
     count1++; 
     this.textblock1.Text=count1.ToString(); 

    } 

    private void Button2_Click(object sender, Windows.UI.Xaml.RoutedEventArgs e) 
    { 
     count2++; 
     this.textblock2.Text=count2.ToString(); 
    } 
    public void ResetTimer() 
    { 
      timeLeft = 20; 
    } 
    private void StartButton_Click(object sender, RoutedEventArgs e) 
    { 
     clickCounter++; 
     if (textblock1.Text == "0") 
     { 
      ResetTimer(); 
      StartTimer(); 
      this.Button1.IsEnabled = true; 
     } 
     else 
     { 
      ResetTimer(); 
      StartTimer(); 
      this.Button2.IsEnabled = true; 
     } 
    } 

Answer 1

あなたはStartTimer方法timer.Tick += timer_Tick;呼び出すたびに実行されます。つまり、StartTimerを2回目に呼び出すと、すべてのティックで2つのイベントが呼び出されます。

private void InitTimer() 
{ 
    timer = new DispatcherTimer(); 
    timer.Interval = new TimeSpan(0,0,0,1); 
    timer.Tick += timer_Tick; 
} 

private void StartTimer() 
{ 
    timer.Start(); 
} 

のようにコードを分割し、InitTimerのみを呼び出します。もう1つのオプションは、if (this.timer != null)のイベントをコードtimer.Tick -= timer_Tick; に登録解除することです。もう1つ目の問題は命名の競合です。StartTimerメソッドにプライベートグローバル変数タイマーと1つの可変タイマーがあります。

//編集：フル更新されたコード：

private int count1=0; 
private int count2=0; 
private int clickCounter = 0; 
private int timeLeft; 
private DispatcherTimer timer; 

private void StartTimer() { 
    if (timer == null) { 
     timer = new DispatcherTimer(); 
     timer.Interval = new TimeSpan(0,0,0,1); 
     timer.Tick += timer_Tick; 
    } 
    timer.Stop(); 
    timeLeft = 20; 
    timer.Start(); 
} 

public void timer_Tick(object sender, object args) { 
    if (timeLeft > 0) { 
     timeLeft = timeLeft - 1; 
     timerTextBlock.Text = Convert.ToString(timeLeft); 
    } else { 
     timer.Stop(); 
     if (clickCounter==2) { 
      ShowResult(); 
      Button2.IsEnabled = false; 
      StartButton.IsEnabled = false; 
     } else { 
      myMsg.Text = "Time's up!"; 
      Button1.IsEnabled = false; 
      StartButton.IsEnabled = true; 
     } 
    } 
} 

private void Button1_Click(object sender, Windows.UI.Xaml.RoutedEventArgs e) { 
    count1++; 
    textblock1.Text=count1.ToString(); 
} 

private void Button2_Click(object sender, Windows.UI.Xaml.RoutedEventArgs e) { 
    count2++; 
    textblock2.Text=count2.ToString(); 
} 

private void StartButton_Click(object sender, RoutedEventArgs e) { 
    clickCounter++; 
    StartButton.IsEnabled = false; 
    if (textblock1.Text == "0"){ 
     Button1.IsEnabled = true; 
     StartTimer(); 
    } else { 
     Button2.IsEnabled = true; 
     StartTimer(); 
    } 
}