这是我更新的代码,它仍然无效。它返回2月29日所有案例的日期,如果它是闰年,它应该只返回一天,如果它不是闰年,则应返回。
public int checkDay (int day)
{
// For months with 30 days.
if ((month == 4 || month == 6 || month == 9 || month == 11) && day <= 30)
return day;
// For months with 31 days.
if ((month == 1 || month == 3 || month == 5 || month == 7 || month == 8 || month == 10 || month == 12) && (day <= 31))
return day;
// For leap years.
// If February 29th...
if (month == 2 && day == 29)
{
// Check if year is a leap year.
if ((year%4 == 0 && year%100!=0) || year%400 == 0)
{
// If year is a leap year return day as 29th.
return day;
}
// If not a leap year, return day as 1st.
else return 1;
}
// If Date if February 1st through 28th return day, as it is valid.
if (month == 2 && (day >= 1 && day <= 28))
return day;
// Return day as 1st for all other cases.
return 1;
}
答案 0 :(得分:5)
尝试GregorianCalendar http://docs.oracle.com/javase/6/docs/api/java/util/GregorianCalendar.html
GregorianCalendar gc = new GregorianCalendar();
if (gc.isLeapYear(year) )
答案 1 :(得分:1)
尝试将代码更改为
if (year%4==0&&(year%100!=0&&year%400==0))
答案 2 :(得分:0)
if ((year%4==0 && year%100!=0) || year%400==0)
这解决了你的问题,你的逻辑是错误的:)
尝试此代码:如果返回的布尔值为false,则可以将日期设置为1,因为日期无效。
public bool checkDay (int day, int month, int year){
bool valid = false;
if(day >=1){
// For months with 30 days.
if ((month == 4 || month == 6 || month == 9 || month == 11) && day <= 30){
valid = true;
}
// For months with 31 days.
if ((month == 1 || month == 3 || month == 5 || month == 7 || month == 8 || month == 10 || month == 12) && day <= 31){
valid = true;
}
// For February.
if (month == 2)
{
if(day <= 28){
valid = true;
} else if(day == 29){
if ((year%4 == 0 && year%100!=0) || year%400 == 0){
valid = true;
} //else invalid
}
}
} //else date is not valid
return valid;
}
最好在每个方法中只有一个return语句。这使得更容易理解代码,并通过它来调试它并找到可能的错误。如果您有任何问题,请随时提出。