Question

代码：

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!");
   }
  }
 }
}

与标题一样，我正在寻找对此代码的一些改进。感谢您阅读并度过美好的一天！

Answer 1

我建议你阅读关于Java代码约定的Oracle tutorial，这对初学者来说至关重要。

说到你的例子，前面提到的单行（称为三元运算符）可能是好的，但它往往会降低更复杂代码块的可读性。

就我而言，您不需要为您尝试执行的任务声明两个变量。你会想要理性地创建变量。他们应该为代码带来一个目的，否则他们就没有必要。并坚持使用更有意义的名字，因为你肯定会回答他们对你给他们的名字感到困惑以及他们应该做些什么。

Answer 2

一般来说，这些提示可能有所帮助：

始终保持班级名称（首字母大写）和变量Camel案例。（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);
   }
}

您好，我是Java的新手，有人可以告诉我如何改进这个吗？有小费吗？

2 个答案: