Скамья, бумага, ножницы-BlueJ сумасшедшая петля

Question

следующий мой код для игры рок, бумага, ножницы в BlueJ. Когда я компилирую и пользователь вводит вход, компьютер немедленно печатает многочисленные выходы из playerWins(). Игра заканчивается, когда пользователь набирает «quit». Может ли кто-нибудь помочь мне, чтобы мой экран не был затоплен? (и если есть какой-либо способ сконденсировать мой код, который также будет большим).

import java.util.Random; 
import java.util.Scanner; 

public class RockPaperScissors 
{ 

    public static void main(String[] args) 
    { 
     int wins = 0, losses = 0, ties = 0; 
     boolean output; 
     Scanner scan = new Scanner(System.in); 
     System.out.print("(R)ock, (P)aper, (S)cissors, or quit: "); 
     String playerChoice = scan.nextLine(); 
     while (playerChoice.equals("quit") == false) 
     {  
      playerChoice = playerChoice.toUpperCase(); 
      String computerChoice = getComputerChoice(); 
      if(playerChoice.equals(computerChoice) != true) 
      { 
       output = playerWins(playerChoice, computerChoice); 
       if (output == true) 
       { 
        wins++; 
       } 
       else if (output == false) 
       { 
        losses++; 
       } 
      } 
      else 
      { 
       ties++; 
       System.out.println("Tie!"); 
      } 
     } 
     System.out.println("QUIT"); 
     System.out.println("Wins: " + wins); 
     System.out.println("Losses: " + losses);    
     System.out.println("Ties: " + ties); 
     scan.close(); 
    } 
    public static String getComputerChoice() 
    { 
     Random gen = new Random(); 
     int num = gen.nextInt(30) + 1; 
     if (num % 3 == 2) 
     { 
      return "R"; 
     } 
     else if (num % 3 == 1) 
     { 
      return "P"; 
     } 
     else 
     { 
      return "S"; 
     } 
    } 
    public static boolean playerWins(String playerChoice, String computerChoice) 
    { 
     if (playerChoice.equals("R") == true) 
     { 
      if (computerChoice.equals("P") == true) 
      { 
       System.out.println("My Point! \nP beats R"); // Rock is beaten by paper 
       return false; 
      } 
      else if (computerChoice.equals("S") == true) 
      { 
       System.out.println("Your Point! \nR beats S"); // Rock beats scissors 
       return true; 
      } 
     } 
     else if (playerChoice.equals("P") == true) 
     { 
      if (computerChoice.equals("R") == true) 
      { 
       System.out.println("Your Point! \nP beats R"); //Paper beats rock 
       return true; 
      } 
      else if (computerChoice.equals("S") == true) 
      { 
       System.out.println("My Point! \nS beats P"); //Paper is beaten by scissors 
       return false; 
      } 
     } 
     else if (playerChoice.equals("S") == true) 
     { 
      if (computerChoice.equals("P") == true) 
      { 
       System.out.println("Your Point! \nS beats P"); //Scissor beats paper 
       return true; 
      } 
      else if (computerChoice.equals("R") == true) 
      { 
       System.out.println("My Point! \nR beats S"); //Scissors is beaten by rock 
       return false; 
      } 
     } 
     return false; 
    } 
} 

Answer 1

Эти две линии являются проблемой:

String playerChoice = scan.nextLine(); 
while (playerChoice.equals("quit") == false) { 

Вы читаете одну строку, а затем проверить эту линию снова и снова. Вам необходимо прочитать новую строку внутри цикл. То, как вы это делаете сейчас, просто пытается сделать один и тот же ход бесконечно.

Try:

String playerChoice = scan.nextLine(); 
while (playerChoice.equals("quit") == false) { 

    //do all of the stuff that is already inside your loop 

    playerChoice = scan.nextLine(); 
} 

Это позволит получить пользовательский ввод снова каждый раз после обработки последнего ввода.

Answer 2

Вы проверяете ввод пользователя только один раз, то есть перед циклом:

String playerChoice = scan.nextLine(); 

Один очень хороший способ, чтобы решить, что является следующим:

String playerChoice; 
while((playerChoice = scan.nextLine()).equals("quit") == false) 
{ 
    //more code 
} 

Вышеприведенный код делает , то, что playerChoice установлен прямо перед .equals("quit") называется. Если вы используете этот метод, вам не нужна еще одна строка в конце вашего цикла, чтобы установить playerChoice, но просто сделайте это в голове вашего цикла.