無限ループに巻き込まれる

Question

なぜ私のコードが無限ループに巻き込まれるのか分かりません。コードが間違って書かれているのでしょうか？ 出力がエラーに

public class HelloWorld 
{ 
    public static void main(String[] args) 
    { 
    //Not sure if it's a better way but I did this way: 
final int TRIES = 5; 
int heads = 0; 
for(int i=0; i<=TRIES; i++) 
{ 
    Scanner input = new Scanner(System.in); 
    System.out.print("Do you wanna continue? Y or N? "); 
    char c = input.next().charAt(0); 
    c = Character.toUpperCase(c); 
    if(c =='y'){ 
    int r = (int) Math.random * 2) + 1; //A number between 1 & 2 

     if(r == 1) 
     { 
     System.out.print("heads"); 
     heads++; 
     } 
     else{ 
     System.out.print("Lost"); 
     } 
    } 

    else{ 
     System.out.print(heads); 
    } 
    } 
} 

}

Answer 1

を与えていない私はどちらかわからないが、私はこのコードは正常に動作すべきだと思う：

import java.util.Scanner; 
import java.util.Random; 
public class HelloWorld { 
    public static void main(String[] args) { 
     //Not sure if it's a better way but I did this way: 
     final int TRIES = 5; 
     int heads = 0; 
     for(int i=0; i<=TRIES; i++) { 
      Scanner input = new Scanner(System.in); 
      System.out.print("Do you wanna continue? Y or N? "); 
      char c = input.next().charAt(0); 
      c = Character.toUpperCase(c); 
      if(c =='Y') { 
       int r = (int) (Math.random() * 2) + 1; //A number between 1 & 2 
       if(r == 1) { 
        System.out.println("heads"); 
        heads++; 
       } else { 
        System.out.println("Lost"); 
       } 
      } else { 
       System.out.print(heads); 
      } 
     } 
    } 
} 

Answer 2

public class HelloWorld 
{ 
    public static void main(String[] args) 
    { 
     //Not sure if it's a better way but I did this way: 
     final int TRIES = 5; 
     int heads = 0; 
     for(int i=0; i<=TRIES; i++) 
     { 
      Scanner input = new Scanner(System.in); 
      System.out.print("Do you wanna continue? Y or N? "); 
      char c = input.next().charAt(0); 
      c = Character.toUpperCase(c); 
      if(c =='Y'){ 
       int r = (int) (Math.random() * 2) + 1; //A number between 1 & 2 

       if(r == 1) 
       { 
        System.out.println("heads"); 
        heads++; 
       } 
       else{ 
        System.out.println("Lost"); 
       } 
      } 

      else{ 
       System.out.println(heads); 
       break; 
      } 
     } 
    } 

} 

私はわかりませんよあなたは達成しようとしていますが、このコードはコンパイルされ、さらに続行したくないと言ったときにループを停止します。 breakを使うのはベストプラクティスではありませんが、動作します。あなたのTRIES変数によって与えられた制限であるため、最大5回までしか聞こえません。あなたが入力した文字を回した

boolean stop = false; 
    while(!stop) 
    { 
     Scanner input = new Scanner(System.in); 
     System.out.print("Do you wanna continue? Y or N? "); 
     char c = input.next().charAt(0); 
     c = Character.toUpperCase(c); 
     if(c =='Y'){ 
      int r = (int) (Math.random() * 2) + 1; //A number between 1 & 2 

      if(r == 1) 
      { 
       System.out.println("heads"); 
       heads++; 
      } 
      else{ 
       System.out.println("Lost"); 
      } 
     } 

     else{ 
      System.out.println(heads); 
      stop=true; 
     } 

注：ユーザーが継続したくない場合、あなたはそのようなブール変数としばらく、使用したいmigth停止し、無限ループを実行する場合大文字に変換し、それを 'y'と比較すると、常にfalseになります。