代码:
package minecraft;
import java.util.Scanner;
class minecraft {
public static void main(String[] argv) {
Scanner sc = new Scanner(System.in);
boolean i = sc.nextBoolean();
boolean isMCagoodgame = i;
if (isMCagoodgame == true) {
System.out.println("Minecraft is a good game!");
} else {
{
System.out.println("Minecraft is a NOT good game!");
}
}
}
}
与标题一样,我正在寻找此代码的一些改进。感谢您的阅读,祝您有美好的一天!
我建议您阅读有关Java代码约定的Oracle教程,这对于初学者至关重要。
说到您的示例,前面建议的单行(称为三元运算符(可能很好,但它往往会降低更复杂的代码块中的可读性。
就我而言,您不需要为您尝试执行的任务声明两个变量。您需要理性地创建变量。他们应该为代码带来一个目的,否则就不需要他们了。并坚持使用更有意义的名字,因为您肯定会对您给他们起的名字以及他们应该做什么感到困惑。
一般来说,这些提示可能会有所帮助:
- 始终保持类名(首字母大写(和变量驼峰大小写。(CamelCase 是一种命名约定,其中名称由多个单词组成,这些单词连接在一起为一个单词,多个单词中每个单词的第一个字母大写,以便组成名称的每个单词都可以轻松阅读。来源: https://searchmicroservices.techtarget.com/definition/CamelCase(
- 将变量命名为有意义的变量。不要使用单个字母。任何查看您的代码的人都应该了解您要做什么。
- 正确的缩进
- 避免不必要的代码。保持代码简短。例如,如果可执行代码是单行的,则可以使用三元运算符代替 if else。
-
有时,如果额外的变量使您的代码更具可读性,则可以使用额外的变量。
class Minecraft { public static void main(String[]argv) { Scanner input = new Scanner(System.in); String message; boolean isGoodGame = input.nextBoolean(); message = isGoodGame ? "Minecraft is a good game!" : "Minecraft is a NOT good game!"; System.out.println(message); } }