ifステートメント以外の変数値

Question

このコードブロックでは、13行目でcorrectAnswersを1つ増やしました。ただし、if文が壊れた後は、パーセンテージを出力するときに値が1になります。誰か私のコードに間違っていることを教えてもらえますか？

private static void multiplicationTest(int maxNumber, int minNumber) { 
    int i = 1;               

    while(i != 11) {              
     int firstNumber = (minNumber + (int)(Math.random() * ((maxNumber - minNumber) + 1))), secondNumber = (minNumber + (int)(Math.random() * ((maxNumber - minNumber) + 1))); 
     int inputAnswer, answer = (firstNumber * secondNumber); 
     int correctAnswers = 0; 

     System.out.print("Question " + i + ".)\t" + firstNumber + " * " + secondNumber + " = "); 
     inputAnswer = input.nextInt(); 

     if(inputAnswer == answer) { 
      correctAnswers++; 
      System.out.print("\tcorrect\n"); 

     } else { 
      System.out.print("\tincorrect --- " + firstNumber + " * " + secondNumber + " = " + answer + "\n"); 

     } if(i == 10) { 
      System.out.println("\nYou scored " + correctAnswers + " out of 10 - " + (correctAnswers * 10) + "%."); 

     } 

     i++; 
    } 

} 

Answer 1

while行の前に置きint correctAnswers = 0;。

whileループでは、すべての実行で総合スコアをリセットしたままにします。

これは、スコアは1の外に実際にあるたびに意味はなく、10

Answer 2

UserF40はあなたの誤りを指摘しています。残りの部分については、私は短いコードレビューを行います。

1. あなたは、それを行うためにforループを使用することがはるかに簡単である1から10までカウントするwhileループを使用している：for (int i = 1; i < 11; ++i)を。ループを終了する数値でないブール条件がある場合は、whileループを使用します。

2. お客様の2番目のifは置き忘れられている可能性があります。ループの最後のパスでのみアクションが発生するので、ループが終了した後でアクションを実行することができます。ifは必要ありません。

3. Javaライブラリを学んでください。あなたはMath.random()は実数には良いが、整数には適していない。ランダムな整数の場合は、整数を直接供給するRandom.nextInt()を使用します。

4. 1行に複数の宣言を避けてください。これは不要で、混乱する可能性があります。 1行につき1つの宣言が明確になります。