検証ボタンが正しくありませんか？

Question

だから、私は2つのラベルで2つの数字を生成するアプリケーションを作成する必要があり、ユーザーは答えとしてこれらの2つの数字の追加値を入力する必要があります。 次に、ユーザーがテキストボックスに正しい解決方法を入力したかどうかを確認するボタンを作成する必要があります。

ただし、私が作成したチェックボタンは毎回「間違った」と答えています。違う？

private void generatebutton_Click(object sender, EventArgs e 

    { 

      //Declaring Variables for Generate button 
      int answer = 0; 
      Random rand = new Random(); 
      var intNum1 = rand.Next(100, 500); 
      var intNum2 = rand.Next(100, 500); 
      answer = intNum1 + intNum2; 


      //Generate integer number 1 in first label 
      firstnumberlabel.Text = intNum1.ToString(); 

      //Generate integer number 2 in second label 
      secondNumberlabel.Text = intNum2.ToString(); 
    } 

    private void checkButton_Click(object sender, EventArgs e) 

     { 
      //Declaring variables for answer label 
      int userInput = 0; 
      int answer = 0; 
      Random rand = new Random(); 
      var intNum1 = rand.Next(100, 500); 
      var intNum2 = rand.Next(100, 500); 
      answer = intNum1 + intNum2; 

      if (int.TryParse(txtanswer.Text, out userInput)) 
      { 
       if (userInput == answer) 
       { 
        // Display success message 
        MessageBox.Show("Correct!"); 
       } 

       else 
       { 
        // Display wrong answer message 
        MessageBox.Show("Incorrect!"); 
       } 
     } 
     else 
     { 
      // show message that invalid input 
     } 
    } 

    private void exitButton_Click(object sender, EventArgs e) 
    { 
     //Closes Application 
     this.Close(); 
    } 

    private void button4_Click(object sender, EventArgs e) 
    { 
     //Clears text box 
     txtanswer.Text = ""; 
    } 
} 

}

Answer 1

あなたは、ランダムな数字ではなく、オリジナルの2個の乱数の新しいセットにユーザーの回答を比較しています。

checkButton_Clickイベントでは、int answer = 0の後に4行を置き換えます。 answer = int.Parse(firstnumberlabel.Text) + int.Parse(secondnumberlabel.Text);

ユーザが数字を入力せずに答えをチェックしようとした場合に例外を避けるために追加する必要がある入力チェックがまだあります。

Answer 2

次のように変更すると修正されます。あなたが間違った答えを見つけるために新しい乱数を作成しています

private void generatebutton_Click(object sender, EventArgs e 
{ 
     //Declaring Variables for Generate button 
     Random rand = new Random(); 
     var intNum1 = rand.Next(100, 500); 
     var intNum2 = rand.Next(100, 500); 

     //Generate integer number 1 in first label 
     firstnumberlabel.Text = intNum1.ToString(); 

     //Generate integer number 2 in second label 
     secondNumberlabel.Text = intNum2.ToString(); 
} 

private void checkButton_Click(object sender, EventArgs e) 
{ 
    int userInput = 0; 
    int answer = int.Parse(firstnumberlabel.Text) + int.Parse(secondnumberlabel.Text); 
    if (int.TryParse(txtanswer.Text, out userInput)) 
    { 
      if (userInput == answer) 
      { 
       // Display success message 
       MessageBox.Show("Correct!"); 
      } 
      else 
      { 
       // Display wrong answer message 
       MessageBox.Show("Incorrect!"); 
      } 
    } 
    else 
    { 
     MessageBox.Show("Please enter a valid answer!"); 
    } 
} 

private void exitButton_Click(object sender, EventArgs e) 
{ 
    //Closes Application 
    this.Close(); 
} 

private void button4_Click(object sender, EventArgs e) 
{ 
    //Clears text box 
    txtanswer.Text = ""; 
}