Variabler Wert anders außerhalb der if-Anweisung

Question

In diesem Codeabschnitt, in Zeile 13, habe ich correctAnswers um 1 inkrementiert. Sobald die if-Anweisung jedoch abgebrochen wurde, ist der Wert nur eins (oder null), wenn der Prozentsatz ausgegeben wird . Kann mir jemand sagen, was mit meinem Code nicht stimmt?

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

Platz int correctAnswers = 0; vor der while Linie.

Wenn es in der While-Schleife ist, wird es die Gesamtpunktzahl bei jedem Lauf zurückgesetzt.

Das bedeutet, die Gäste wirklich jedes Mal aus einer ist, nicht 10.

Answer 2

UserF40 Ihre Fehler hingewiesen hat. Für den Rest werde ich einen kurzen Code-Review machen.

1. Sie verwenden eine while Schleife von 1 bis 10 zu zählen, ist es viel einfacher, eine for Schleife zu verwenden, das zu tun: for (int i = 1; i < 11; ++i). Verwenden Sie eine while-Schleife, wenn eine nicht-numerische boolesche Bedingung zum Beenden der Schleife vorliegt.

2. Ihre zweite if ist verlegt, und kann weggelassen werden. Da nur beim letzten Durchlauf der Schleife eine Aktion ausgeführt wird, können Sie die Aktion nach dem Beenden der Schleife einfügen, ohne dass if erforderlich ist.

3. Lernen Sie die Java-Bibliotheken. Sie Math.random() ist gut für reelle Zahlen, aber nicht gut für ganze Zahlen. Für zufällige Ganzzahlen verwenden Sie Random.nextInt(), die Integer direkt liefert.

4. Vermeiden Sie mehrere Deklarationen in einer Zeile, es ist unnötig und kann verwirrend sein. Eine Deklaration pro Zeile ist klarer.