将多个if-else语句格式化为方法

Question

我有一个检查用户日期输入并显示下一个日期输出的程序。 但是，我在程序中使用了多个if-else语句。

我想对其进行更改，以便它将使用方法而不是重复的代码来进行计算。

下面是我的代码示例，以3月和4月为例，一个30天，一个31天：

public class calDate{

    public static void main(String[] args) {
        Scanner scan = new Scanner(System.in);
        boolean dateValid = false;
        int day = 0;
        int month = 0;
        int year = 0;
        int nextDay = 0
        int nextYear = 0;
        String nextMonth = "";
        boolean run=true;
        char again='Y';

while(run)
        {

        System.out.print("Day: ");
        day = scan.nextInt();

        System.out.print("Month: ");
        month = scan.nextInt();

        System.out.print("Year: ");
        year = scan.nextInt();

while (month < 1 && month > 12) //check if month is within 1 to 12
        { 
            dateValid = false;
        }

if ((month == 3) && (day >= 1 && day <= 31)) 
        {
            dateValid = true;
            nextDay = day + 1;
            nextMonth = " March ";
            nextYear = year;

            if (day == 31) {
                nextDay = 1;
                nextMonth = " April ";
                nextYear = year;
            }
        } 

        else if ((month == 4) && (day >= 1 && day <= 30)) 
        {
            dateValid = true;
            nextDay = day + 1;
            nextMonth = " April ";
            nextYear = year;

            if (day == 30) 
            {
                nextDay = 1;
                nextMonth = " May ";
                nextYear = year;
            }  
        }
if (dateValid){
        System.out.println("Tomorrow's date: " + nextDay + nextMonth + nextYear);
        System.out.print("Continue?(Y/N)  ");
        again = scan.next().charAt(0);
        if (again=='Y')
        {
        run=true;
        System.out.println("");
        }
        if (again=='N')
        {
        run=false;
        }
        if ((again != 'N')&&(again!='Y'))
        {
        System.out.println("Invalid input. Ending program."); 
            run=false;
        }   
        }

        else{
        System.out.println("Invalid input. Continue?(Y/N)  ");
        again = scan.next().charAt(0);
        if (again=='Y')
        {
        run=true;
        System.out.println("");
        }
        if (again=='N')
        {
        run=false;
        }
        if ((again != 'N')&&(again!='Y'))
        {
        System.out.println("Invalid input. Ending program.");
        run=false;
        }
    }
   }

有许多重复的值，例如dateValid ， nextDay ， nextMonth ， nextYear 。 如何格式化代码，以便可以将语句放入单独的方法中？ 谢谢。

Answer 1

我的第一个建议是将几个月变成枚举。 这样，您可以将与月份相关的所有逻辑封装在一个类中：

enum Month {
    JAN ("Jan", 31),
    FEB ("Feb", 28), 
    ....
    DEC ("Dec", 31);

    private final String monthName;
    private final int daysInMonth;

    private Month(String monthName, int daysInMonth) {
        this.monthName = monthName;
        this.daysInMonth = daysInMonth;
    }

    public String getName() {
        return monthName;
    }

    public static Month monthWithNumber(int monthNumber) {
        if (monthNumber < 1 || monthNumber > 12) 
            throw new IllegalArgumentException();
        return values()[monthNumber - 1];
    }

    public isLegalDay(int day) {
        return day > 0 && day <= daysInMonth;
    }

    public isLastDayInMonth(int day) {
        return day == daysInMonth;
    }

    public Month nextMonth() {
        return values[(ordinal() + 1) % 12];
    }
}

这将立即删除您的大多数if语句，并将它们替换为：

Month month = Month.monthWithNumber(monthNumber);
if (month.isLegalDay(dayNumber)) {
    ....
    if (month.isLastDayInMonth(dayNumber)) {
        nextMonth = month.nextMonth().getName();
        nextDay = 0;
    }
}

Answer 2

我建议您使用给定的输入值构建一个String，使用SimpleDateFormat解析生成的String，然后捕获ParseException以更新dateValid布尔值。

这也使您可以更轻松地添加一天。

Answer 3

我建议您使用类似int[] daysInTheMonth = { 31. 28, ...和String[] monthName = ",Jan,Feb,Mar".split(","); 即使您处理了所有月份，在代码中使用数据也将使其比现在短得多。

包含isLeapYear(year)方法将很有用。

为了进行测试，我建议您确保获得与JSR-310库相同的结果。 虽然我假设您不能直接使用此库，但可以使用它来测试代码是否正确。

Answer 4

我可能会创建方法getNextDay() ， getMonthForNextDay()和getYearForNextDay() 。 他们将获得用户输入作为参数，并分别简单地返回正确的日，月和年。 您也可以将其与Peter Lawreys的答案结合起来。