1

我的代码有递归——execute() 再次调用() 等等。我正在考虑在 main() 或 execute() 处使用 do/while 循环来消除这种情况,但如果我这样做了,则询问用户的字符串是否他们想再玩一次就会超出范围。我不能在循环之前或之后使用 console2 — 问他们是否想在开始时再次播放是没有意义的。随意指出我的初学者代码的任何缺陷,因为可能有很多。谢谢。

到目前为止,这是我的代码——带有递归:

import java.util.*;
import java.awt.*;
import java.io.*;


public class RockPaperScissors {


    public static int count = 0;

    public static void main(String[] args) {

            execute(); 
    }


    public static void execute(){       

        System.out.println("This program will allow you to play \n\"Rock, Paper, Scissors\" against a computer.");
        System.out.println();
        System.out.println("Enter 'r' for Rock, 'p' for Paper, or s for Scissors.");
        System.out.println("Enter 'w' to have an insta-win for that round. Enter '-1' at anytime to exit program.");

        String info = userInput();
        int value = guessCode();
        decideOutcome(value, info);
        again();
        }


        public static String userInput() {
        Scanner console = new Scanner (System.in);  
        String s = console.next();
        return s;
    }

        public static int guessCode() {
        Random r = new Random ();   
        return (r.nextInt(3)+1); // Random integer between 1 and 3;
    }

    public static void decideOutcome(int i, String j) {

        try {
            PrintStream output = new PrintStream(new FileOutputStream("records.txt", true));

            if (j.equalsIgnoreCase("rock")|| j.equalsIgnoreCase("r")) {
                count++;
                switch (i){
                    case 1:
                        System.out.println("You've won! Computer picked scissors.");
                        output.println(count + " Win ");
                        break;
                    case 2:
                        System.out.println("You've tied.... Computer also picked rock.");
                        output.println(count + " Tie ");
                        break;
                    case 3:
                        System.out.println("You've lost. Computer picked paper.");
                        output.println(count + " Loss ");
                        break;
                    }           
            } else if (j.equalsIgnoreCase("paper")|| j.equalsIgnoreCase("p")) {
                count++;
                switch (i){
                    case 1:
                        System.out.println("You've lost; Computer picked scissors.");
                        output.println(count + " Loss ");
                        break;
                    case 2:
                        System.out.println("You've won! Computer picked rock.");
                        output.println(count + " Win ");
                        break;
                    case 3:
                        System.out.println("You've tied.... Computer also picked paper.");
                        output.println(count + " Tie ");
                        break;
                    }
            } else if (j.equalsIgnoreCase("scissors")|| j.equalsIgnoreCase("s")) {
                count++;
                switch (i){
                    case 1:
                        System.out.println("You've tied.... Computer picked scissors.");
                        output.println(count + " Tie ");
                        break;
                    case 2:
                        System.out.println("You've lost; Computer picked rock.");
                        output.println(count + " Loss ");
                        break;
                    case 3:
                        System.out.println("You've won! Computer also picked paper.");
                        output.println(count + " Win ");
                        break;
                    }
            } else if (j.equalsIgnoreCase("w")) {
                count++;
                System.out.println("You've effortlessly defeated the computer!");
                output.println(count + " Win ");
            } else if (j.equals("-1")) {
                System.out.println("Thanks for playing!"); // need to find way to reach end.

                if (count == 1) { // If the user terminates after the first match.
                    System.out.println("You've played a single match.");        
                } else if (count > 1) { // Anything more than 1 match played upon termination.
                    System.out.println("You've played " + count + " matches total.");   
                } else { // This is for exceptions when user inputs gibberish for their sign and then 'no' for the second input.
                    System.out.println("No matches were played.");  
            }           
                System.out.println("Good Bye!");
                System.exit(0);
            } else {
                System.out.println("You didn't input the right thing.");
            }
        } catch (FileNotFoundException e) {
                System.out.println("File was not found; try again");
        }
    }

    public static void again() {
            System.out.println("Do you want to play again? (Type in 'y' for Yes or 'n' for No.)");

            Scanner console2 = new Scanner (System.in);

            String t = console2.next();

            while (t.equalsIgnoreCase("yes")||t.equalsIgnoreCase("y")) {
                System.out.println();
                System.out.println();
                execute(); // 
            }  
            if (t.equalsIgnoreCase("no") || t.equalsIgnoreCase("n") || t.equals("-1")) {
                System.out.println("Hope you had fun! I'm sure I've had just as much fun with making this program! Good Bye!");         

                if (count == 1) { // If the user terminates after the first match.
                        System.out.println("You've played a single match.");        
                } else if (count > 1) { // Anything more than 1 match played upon termination.
                        System.out.println("You've played " + count + " matches total.");   
                } else { // This is for exceptions when user inputs gibberish for their sign and then 'no' for the second input.
                        System.out.println("No matches were played.");              
                }   
                System.exit(0);

            } else { // If the user doesn't input 'yes' or 'no.'
                System.out.println("Not the proper response, but it's assumed that you don't want to continue.");   
                if (count == 1) { // If the user terminates after the first match.
                    System.out.println("You've completed a single match."); 
                } else if (count >= 2) { // Anything more than 1 match played upon termination.
                    System.out.println("You've completed " + count + " matches total.");
                } else { // The user haphazardly messes up both inputs.
                    System.out.println("No matches were finished.");
                }
                System.exit(0);
            }
        }
}
4

3 回答 3

0

这是您的代码的修改版本。有几行重复的代码可以重新用于差异条件。对它们进行了修改。将其与您的代码进行比较以检查已更改的内容。希望能帮助到你。

import java.util.*;
import java.awt.*;
import java.io.*;

public class Test {

public static int count = 0;
private static Scanner console;

public static void main(String[] args) {
    console = new Scanner(System.in);
    execute();
}

public static void execute() {
    String info = null;
    boolean accept;
    do {
        System.out
                .println("This program will allow you to play \n\"Rock, Paper, Scissors\" against a computer.");
        System.out.println();
        System.out
                .println("Enter 'r' for Rock, 'p' for Paper, or s for Scissors.");
        System.out
                .println("Enter 'w' to have an insta-win for that round. Enter '-1' at anytime to exit program.");

        info = userInput();
        int value = guessCode();
        decideOutcome(value, info);
        accept = again();
    } while (!info.equalsIgnoreCase("-1") && accept);
}

public static String userInput() {
    String s = console.next();
    return s;
}

public static int guessCode() {
    Random r = new Random();
    return (r.nextInt(3) + 1); // Random integer between 1 and 3;
}

public static void decideOutcome(int i, String j) {
    try {
        PrintStream output = new PrintStream(new FileOutputStream(
                "records.txt", true));
        if (j.equalsIgnoreCase("w")) {
            count++;
            System.out
                    .println("You've effortlessly defeated the computer!");
            output.println(count + " Win ");
        } else if (j.equals("-1")) {
            System.out.println("Thanks for playing!"); // need to find way
                                                        // to reach end.

            if (count == 1) { // If the user terminates after the first
                                // match.
                System.out.println("You've played a single match.");
            } else if (count > 1) { // Anything more than 1 match played
                                    // upon termination.
                System.out.println("You've played " + count
                        + " matches total.");
            } else { // This is for exceptions when user inputs gibberish
                        // for their sign and then 'no' for the second
                        // input.
                System.out.println("No matches were played.");
            }
            System.out.println("Good Bye!");
            System.exit(0);
        } else if (j.equalsIgnoreCase("rock") || j.equalsIgnoreCase("r")
                || j.equalsIgnoreCase("paper") || j.equalsIgnoreCase("p")
                || j.equalsIgnoreCase("scissors")
                || j.equalsIgnoreCase("s")) {
            count++;
            switch (i) {
            case 1:
                if (j.equalsIgnoreCase("rock") || j.equalsIgnoreCase("r")) {
                    System.out
                            .println("You've won! Computer picked scissors.");
                    output.println(count + " Win ");
                }
                if (j.equalsIgnoreCase("paper") || j.equalsIgnoreCase("p")) {
                    System.out
                            .println("You've lost; Computer picked scissors.");
                    output.println(count + " Loss ");
                }
                if (j.equalsIgnoreCase("scissors")
                        || j.equalsIgnoreCase("s")) {
                    System.out
                            .println("You've tied.... Computer also picked scissors.");
                    output.println(count + " Tie ");
                }

                break;
            case 2:
                if (j.equalsIgnoreCase("rock") || j.equalsIgnoreCase("r")) {
                    System.out
                            .println("You've tied.... Computer also picked rock.");
                    output.println(count + " Tie ");
                }
                if (j.equalsIgnoreCase("paper") || j.equalsIgnoreCase("p")) {
                    System.out.println("You've won! Computer picked rock.");
                    output.println(count + " Win ");
                }
                if (j.equalsIgnoreCase("scissors")
                        || j.equalsIgnoreCase("s")) {
                    System.out
                            .println("You've lost; Computer picked rock.");
                    output.println(count + " Loss ");
                }
                break;
            case 3:
                if (j.equalsIgnoreCase("rock") || j.equalsIgnoreCase("r")) {
                    System.out
                            .println("You've lost. Computer picked paper.");
                    output.println(count + " Loss ");
                }
                if (j.equalsIgnoreCase("paper") || j.equalsIgnoreCase("p")) {
                    System.out
                            .println("You've tied.... Computer also picked paper.");
                    output.println(count + " Tie ");
                }
                if (j.equalsIgnoreCase("scissors")
                        || j.equalsIgnoreCase("s")) {
                    System.out
                            .println("You've won! Computer picked paper.");
                    output.println(count + " Win ");
                    break;
                }
            }
        } else {
            System.out.println("You didn't input the right thing.");
        }
    } catch (FileNotFoundException e) {
        System.out.println("File was not found; try again");
    }
}

public static boolean again() {
    System.out
            .println("Do you want to play again? (Type in 'y' for Yes or 'n' for No.)");
    String t = console.next();
    if (t.equalsIgnoreCase("yes") || t.equalsIgnoreCase("y")) {
        System.out.println();
        System.out.println();
        return true; //w
    } else {
        if (t.equalsIgnoreCase("no") || t.equalsIgnoreCase("n")
                || t.equals("-1")) {
            System.out
                    .println("Hope you had fun! I'm sure I've had just as much fun with making this program! Good Bye!");
        } else {
            System.out
                    .println("Not the proper response, but it's assumed that you don't want to continue.");
        }
        if (count == 1) { // If the user terminates after the first match.
            System.out.println("You've played a single match.");
        } else if (count > 1) { // Anything more than 1 match played upon
                                // termination.
            System.out
                    .println("You've played " + count + " matches total.");
        } else { // This is for exceptions when user inputs gibberish for
                    // their sign and then 'no' for the second input.
            System.out.println("No matches were played.");
        }
        return false;
    }
}
 }
于 2013-05-26T19:56:51.143 回答
0

您可以创建一个新的布尔变量keepPlaying,将其初始化为true,然后一旦玩家希望退出游戏,将其设置为 false。

然后在主要方法中说:

public static void main(String[] args) {

System.out.println("This program will allow you to play \n\"Rock, Paper, Scissors\" against a computer.");
    System.out.println();
    System.out.println("Enter 'r' for Rock, 'p' for Paper, or s for Scissors.");
    System.out.println("Enter 'w' to have an insta-win for that round. Enter '-1' at anytime to exit program.");

    while (keepPlaying == true) {
        again();
    }
}

将输入代码放在again()方法的开头。

private static void again() { // Maybe rename to play()?
    String info = userInput();
    int value = guessCode();
    decideOutcome(value, info);
    .....
}

然后你就不需要了execute()。正如另一个答案所建议的那样,方法和变量只能是公共的,除非另一个类需要它们。

于 2013-05-26T19:14:57.030 回答
0

一个简单的更改是 main 方法中的 do-while execute,并在 while 中测试again(它可能是一个简单的布尔值)的结果,以了解用户是否想要继续。

顺便说一句,在我看来execute应该是唯一的公共方法,没有理由拥有一切public staticcount只是存在的候选人private。否则,例如,重新访问该类将无法确保其初始值为 0。

于 2013-05-26T19:21:19.297 回答