0

下面我有一个我不会工作的代码片段。我在我的主要方法中获得输入，然后将该输入传递给另一个方法以检查验证。但它并没有真正正确地检查。如果我输入99formonth并且day我希望它给我消息Check Month。

相反，我得到： THIS THIS

如果我输入02月份和99日期，我希望它会给我以下信息：Check day. 相反，我得到THIS THIS

如果我同时输入02两者，我希望它能够运行并继续运行其他方法。相反，我得到THIS THIS.

public class Date {

private Calendar parsedDate;

public static void main(String[] args) 
{
    Date main = new Date();
        System.out.println("Enter a date (use the format -> (MM/DD/YYYY)");

    //declare Scanner
    Scanner in = new Scanner (System.in);

    System.out.println("Enter a month (MM): ");
    String month = in.nextLine();

    System.out.println("Enter a day (DD): ");
    String day = in.nextLine();

    System.out.println("Enter a year (YYYY): ");
    String year = in.nextLine();

    if (main.isValidDate(month, day, year) == true) 
    {
        main.newFormat(month, day, year);
        main.isLeapYear(year);
        main.dayNumber(month, day);
    }
    else if (main.isValidDate(month, day, year) == false)
    {
        System.out.println("Invalid Input");
    }
}//end of main

private boolean isValidDate(String month, String day, String year) 
{
    //check month       
    if(month == "01" || month == "03" || month == "04" ||
       month == "05" || month == "06" || month == "07" || month == "08" ||
       month == "09" || month == "10" || month == "11" || month == "12")
    { 
        //check day
        if(day == "01" || day == "02" || day == "03" || day == "04" ||
           day == "05" || day == "06" || day == "07" || day == "08" ||
           day == "09" || day == "10" || day == "11" || day == "12" ||
           day == "13" || day == "14" || day == "15" || day == "16" ||
           day == "17" || day == "18" || day == "19" || day == "20" ||
           day == "21" || day == "22" || day == "23" || day == "24" ||
           day == "25" || day == "26" || day == "27" || day == "28" ||
           day == "29" || day == "30" || day == "31")
        {
            return true;
        }
        else
        {
            System.out.println("Check Day");
            return false;
        }
    }//end of check month
    else if (month == "02")
    {
        if (day == "28" || day == "29")
        {
            return true;
        }
    }//end of month 2
    else
    {
        System.out.println("THIS");
        return false;
    }

    parsedDate = null;// if it's valid set the parsed Calendar object up.
    return true;
}//end of isValidDate
4

5 回答 5

5

您应该使用 equals 方法来比较字符串。使用==运算符时，您比较的是字符串的地址，而不是它们的内容。

于 2013-04-03T05:55:14.510 回答
4

这month == "01"不是String在 Java 中比较 s 的正确方法。

您正在尝试比较两个变量的内存位置，这两个变量不太可能相等。

相反，您应该使用month.equals("01").

于 2013-04-03T05:54:20.253 回答
4

虽然答案解释了它目前不工作的原因 - 使用==而不是equals- 我建议你不要只是开始使用equals.

相反，首先将用户输入解析为数字，然后验证它们……用数字而不是字符串执行数字比较要容易得多。就像是：

private boolean isValidDate(int year, int month, int day) {
    // Adjust for whatever bounds you want
    if (year < 1900 || year > 2100) {
        System.out.println("Check year");
        return false;
    }

    if (month < 1 || month > 12) {
        System.out.println("Check month");
        return false;
    }

    Calendar calendar = Calendar.getInstance();
    calendar.clear();
    calendar.set(year, month - 1, 1);
    if (day < 1 || day > calendar.getActualMaximum(Calendar.DAY_OF_MONTH)) {
        System.out.println("Check day");
        return false;
    }
    calendar.set(Calendar.DAY_OF_MONTH, day);
    // Store calendar somewhere if you want...
    return true;
}

此外，如果可以的话，我强烈建议使用Joda Time作为更好的日期/时间 API。

编辑：你的下一个问题似乎是这个块：

if (main.isValidDate(month, day, year) == true) 
{
    main.newFormat(month, day, year);
    main.isLeapYear(year);
    main.dayNumber(month, day);
}
else if (main.isValidDate(month, day, year) == false)
{
    System.out.println("Invalid Input");
}

现在我提供的代码已经检查了所有内容并构造了 a Calendar- 不清楚if块中的方法是什么意思。但是你不需要isValidDate第二次打电话。您只需：

// Assuming you're using my new method...
if (isValidDate(year, month, day)) {
    // Do whatever you need to do
} else {
    System.out.println("Invalid input");
}
于 2013-04-03T05:57:28.183 回答
2

有两个大错误

  1. 您正在将字符串与 equals 进行比较，这将比较内存而不是内容，因此该行 if(month == "01")应该是if("01".equals(month))

  2. 检查有效日期的条件if (main.isValidDate(month, day, year) == true)应该是 if (main.isValidDate(month, day, year))

于 2013-04-03T06:04:40.923 回答
1

将字符串转换为整数然后检查（例如，然后您int monthVal = Integer.parseInt(month);可以检查（a）它是否正确解析，使用 try/catch 并（b）测试价值与if (monthVal > 0 && monthVal< 13) { /* month is good */}

于 2013-04-03T05:59:31.300 回答