1 回答
TA贡献1995条经验 获得超2个赞
“我希望该方法运行一次并且输出是用户想要的。”
如果您的真正意思是,基本上这正是代码的作用:
“我希望该方法运行一次,如果他或她输入yes,输出将是用户希望的结果”。
如果用户在classes()方法中对选择输入否,则要求用户选择另一个类,直到他或她对选择感到满意为止。我相信这不是本意吗?
我认为您的问题是,当用户确实进行了类选择时,它停留在classes()方法中。我相信您真正想要做的是返回该选择并在最初调用 classes() 方法之后利用该选择,以便更深入地进行游戏。
为了做到这一点,你需要你的classes()方法返回一些东西,最好是选择的类。由于不同的类(等级)包含在一个字符串数组中:
static String[] Class = {"Farmer", "Wanderer", "Trader", "Nothing"};
您需要从classes()方法返回的只是所选类的实际索引值。您当然可以让另一个类成员变量保存用户选择....也许是一个名为:userClass的变量。然而,在我们开始之前,我想至少提出两个建议:
不要命名变量Class。你可以做到,但对我来说(我相信还有很多其他人)这很令人困惑而且不是一个好主意,因为class本身就是一个Java Keyword。作为建议,也许将其更改为:Ranks。
赋予您的变量和方法名称含义,以便可以轻松理解变量可能包含的内容或方法将执行的操作,例如:getRank()。
// Class Member Variables
static Scanner keyBoardInput = new Scanner(System.in);
static String userInput;
static String userClass = "";
static String[] RANKS = {"Farmer", "Wanderer", "Trader", "Nothing"};
static String LS = System.lineSeparator();
public static void main(String args[]) {
System.out.println("\nNow, before we start you need to choose a class." + LS
+ "Each Class will offer diffrent benefits.");
int rank = getRank();
userClass = RANKS[rank];
System.out.println("Your chosen Class is: " + userClass);
}
public static int getRank() {
int maxRank = (RANKS.length - 1);
// Displays the RANKS array
for (int a = 0; a < RANKS.length; a++) {
System.out.println("[" + a + "]- " + RANKS[a]);
}
// Allow User to select a Rank
int rank = -1;
do {
System.out.println("What would you like to be? (0 to " + maxRank +
" or q to quit)"); //ask for input
userInput = keyBoardInput.nextLine();
// Quit game if desired.
if (userInput.equalsIgnoreCase("q")) {
System.out.println(LS + "Thanks for playing...Bye Bye.");
System.exit(0);
}
// Make sure a valid numerical value was supplied!
if (!userInput.matches("\\d") || Integer.parseInt(userInput) < 0 ||
Integer.parseInt(userInput) > maxRank) {
System.out.println("Invalid input! You can only supply a value "
+ "from 0 to " + maxRank);
userInput = "no";
continue;
}
rank = Integer.parseInt(userInput); // Store for for array index use and returning
String message = "";
// Farmer (0)
switch (rank) {
case 0:
message = LS + "Your health will be increased by 1 permanently.";
break;
case 1:
//if input is this then...
message = LS + "You will have a dagger.";
break;
case 2:
message = LS + "You will have 1 health potion. Drink it to regain 3 HP.";
break;
case 3:
message = LS + "You are just a normal human. You get nothing.";
break;
default:
break;
}
System.out.println(message);
System.out.println("Are you sure you want to be " + RANKS[rank] + "?");
userInput = keyBoardInput.nextLine();
} while (userInput.equalsIgnoreCase("no")); //loops while user types no
return rank; // Return the index value for the chosen Rank.
}
注意到变量名称和getRank()方法名称了吗?遵循代码并理解正在发生的事情要容易得多。不,我个人并不真正关心静态变量和方法。静态的确实有一席之地,但我更喜欢运行比这更严格的代码。
您还会注意到上述代码中的其他更改。特别是用于显示类菜单选项的for循环:
[0]- Farmer
[1]- Wanderer
[2]- Trader
[3]- Nothing
您使用一个名为counter的变量并将其用于索引。这正是您的for循环的用途。您可以摆脱counter变量并简单地使用在for循环本身内声明的变量。
注意switch/case块是如何使用的。它如何减少代码。您还会注意到getRank()方法返回用户选择的类的索引值。当从main()方法调用时, getRank()方法返回类索引值,我们将选择的类放入成员变量userClass中,您现在可以在代码中的任何地方使用它。是的,您也可以直接在getRank()方法中设置userClass变量。
添加回答
举报