扫描仪收集的输入未被正确处理

Question

作为一项自我分配的练习，我正在尝试编写一个简单的程序，该程序以生日为准，并确定给定日期所在的一周中的哪一天。计划的第一部分决定了年份。我使用扫描仪收集用户输入，然后使用do while循环测试输入是否可用（必须是数字，必须有四位数字），如果输入不符合标准，则重复该循环。我试图进入1998年，这显然是一个数字和四位数，它重复循环，好像我的输入是无效的。我假设do while循环中的某个地方有问题，但我无法确定它是什么。下面是有问题的代码。

public static boolean isInteger(String input)
{
   try
   {
      Integer.parseInt(input);
      return true;
   }
   catch(Exception e)
   {
      return false;
   }
}
public static void main(String[] args)
{
    String year = "";
    String yearcorrect = "";
    int ylength = year.length();
    boolean yearIsYear = true;
    Scanner scan = new Scanner(System.in);

    do 
    {
        System.out.println("Please enter the year of your birth.");
        year = scan.nextLine();
        boolean yearIsNum = isInteger(year);

        if (yearIsNum = false || ylength != 3)
        {
            System.out.println("Input is not a valid year.");
            yearIsYear = false;
        }
        else 
        {
            System.out.println("You entered the year " + year + ". Is this correct?");
            yearIsYear = true;
            yearcorrect = System.console().readLine();
        }
    }
    while (yearcorrect != "yes" && yearIsYear != true);

Answer 1

第一个错误：

    if (yearIsNum = false || ylength != 3)

应该是的

    if (yearIsNum = false || ylength != 4)

第二个错误：

正如@ChiefTwoPencils所指出的，ynIsNum=false会将false分配给ynIsNum并且条件永远不会通过。通常java会保护我们免受除...布尔值之外的所有类型的此类错误。为了避免将来出现此类错误（并使您的代码更具可读性），您可以简单地使用！ ynIsNum而不是ynIsNum==false或颠倒值的顺序-

第三个错误

您错误地比较了字符串。必须使用str1。等于（str2）。对对象（字符串是对象）使用==比较它们的引用，而不是它们的值。

例如“a”==新字符串（“a”）将等于false