我对编程很陌生。我很难理解这段代码:
public static boolean onCampus(String name)
{
boolean invalidResponse = true;
do{
System.out.println("Is "+name+ " living on campus? (yes/no)");
String response = in.nextLine();
if(response.equalsIgnoreCase("yes"))
return true;
else if (response.equalsIgnoreCase("no"))
return false;
else
System.out.println("Invalid response.");
} while(invalidResponse);
return false;
}我不明白的是,如果用户输入可接受的响应,invalidResponse在哪里分配false?最后一个return false;是什么?为什么是在“待办”声明之后?
发布于 2015-10-12 22:11:38
这在某种程度上是一种反模式--循环是无限的,永远不会“正确”终止。相反,获得有效响应后,只需调用return并退出该方法,从而终止循环。
循环后的return false是编译限制。在java中,如果方法返回一个值,那么每个可以想到的分支都必须包含一个return语句(确切地说,是一个throw语句)。在编译时,Java并不“知道”循环可能永远不会终止,因此它迫使您在该分支中有一个返回语句(即假设循环终止而不执行if或else if分支)。
发布于 2015-10-12 22:15:54
最后一个“返回false;”就在那里,这样编译器就不会抱怨,它是无法到达的代码。因为您的返回类型是布尔型,所以您的代码必须返回所有情况下的true或false。"invalidResponse“从未分配给false,因此循环将无限地运行,直到用户输入是或否,在这种情况下,它返回布尔值并退出该函数。
发布于 2015-10-12 22:07:08
假设in表示静态Scanner变量。
循环是有意无限的;invalidResponse永远不会被设置为false。
相反,假设输入了"yes"或"no"的值,则"yes"行是退出循环的方法。
对于编译目的,最终的return false;是必要的,但永远不会到达。
但是,值得注意的是,我是单个return的粉丝,使用常量作为等号的前半部分,并删除一次大小写敏感性,并将其重新编码如下:
public static boolean onCampus(String name) {
boolean response = false;
do {
System.out.println("Is " + name + " living on campus? (yes/no)");
String input = in.nextLine().toLowerCase();
response = "yes".equals(input);
if (response || "no".equals(input)) {
break; // exit the loop
} else {
System.out.println("Invalid response.");
}
} while (!response);
return response;
}https://stackoverflow.com/questions/33090969
复制相似问题