if-else 检查所有字符串值（长度，是数字）

Question

我使用 C# 编写了一个小型控制台应用程序以开始使用该语言。我的目标是向用户询问以下内容：

• 名字
• 姓氏
• 出生年份
• 出生月份
• 出生日期

我已将所有输入字段设置如下：

System.Console.Write("Name: ");
String name = System.Console.ReadLine();

最后，应用程序将数据保存到 .txt 文件中，如果给定的数据有效。我需要检查名称字段的长度是否在 1-30 之间，并且日期输入仅接受其相应限制内的数字答案（例如：您只能为 'month' 提供 1-12 之间的值..）

我尝试搜索不同的验证方法，但我不知道如何将它们组合在一起并为此应用程序创建一个干净的“检查器”部分。

这是验证我的名字和姓氏字段的原因，但我认为您不能将日期字段放在同一个检查中？

public static Boolean Checker(String check)
{
   if (check.Length <= 30 && check.Length > 0)
   {
      return true;
   }
   return false;
}

有什么建议吗？

Answer 1

在不知道字符串代表什么的情况下，您无法在单个方法中合理地验证这些输入。

首先，我建议您只要求输入日期而不是三个单独的值。将日期输入验证为单个值而不是三个单独的值要容易得多。 NET 库提供了许多方法来通过单个调用解析日期（DateTime.TryParse、DateTime.TryParseExact）。相反，拥有三个单独的输入需要您复制逻辑来检查闰年、检查月份的最后一天以及由本地化问题引起的日期的许多其他细微方面。

所以，我想您只要求输入名字、姓氏和出生日期，然后您将验证更改为

public static Boolean Checker(String check, bool isDate)
{
   if(isDate)
   {
       DateTime dt;
       // Here you could add your formatting locale as you find appropriate
       return DateTime.TryParse(check, out dt); 
   }
   else
       return check.Length > 0 && check.Length <= 30;
}

这样你的输入应该是这样的

// Infinite loop until you get valid inputs or quit
for(;;)
{
    System.Console.Write("Name: ('quit' to stop)");
    String name = System.Console.ReadLine();
    if(name == "quit") return;

    if(!Checker(name, false))
    {
         // Input not valid, message and repeat
         Console.WriteLine("Invalid name length");
         continue;
    }


    System.Console.Write("Date of Birth: (quit to stop)");
    String dob = System.Console.ReadLine();
    if(dob == "quit") return;

    if(!Checker(dob, true))
    {
         // Input not valid, message and repeat
         Console.WriteLine("Invalid name length");
         continue;
    }
    // if you reach this point the input is valid and you exit the loop
    break;
}

Answer 2

为了检查输入字符串是否为日期，您可以尝试将其解析为 DateTime 对象：

            DateTime date;
            if (!DateTime.TryParseExact(inputString, "MM/dd/yyyy", System.Globalization.CultureInfo.InvariantCulture,System.Globalization.DateTimeStyles.None, out date))
            { 

            }

Answer 3

你应该有一个中心方法，如果它是有效的，则从输入创建一个User-object，并且有一个方法来检查每个输入类型，如FirstName 或DayOfBirth。例如：

public class User
{
    public static User CreateUserFromText(string firstName,string surName,string yearBirth,string monthBirth,string dayBirth)
    {
        if (firstName == null || surName == null || yearBirth == null || monthBirth == null || dayBirth == null)
            throw new ArgumentNullException(); // better tell what argument was null

        User user = new User
        {
            FirstName = firstName.Trim(),
            SurName = surName.Trim()
        };
        bool validInput = IsFirstNameValid(user.FirstName) && IsSurNameValid(user.SurName);
        DateTime dob;
        if (!validInput || !IsValidDayOfBirth(yearBirth, monthBirth, dayBirth, out dob))
            return null;
        user.DayOfBirth = dob;
        return user;
    }

    public DateTime DayOfBirth { get; set; }

    public string SurName { get; set; }

    public string FirstName { get; set; }

    private static bool IsFirstNameValid(string firstName)
    {
        return firstName?.Length >= 1 && firstName?.Length <= 30;
    }

    private static bool IsSurNameValid(string surName)
    {
        return surName?.Length >= 1 && surName?.Length <= 30;
    }

    private static bool IsValidDayOfBirth(string year, string month, string day, out DateTime dayOfBirth)
    {
        DateTime dob;
        string dateStr = $"{year}-{month}-{day}";
        bool validDayOfBirth = DateTime.TryParse(dateStr, out dob);
        dayOfBirth = validDayOfBirth ? dob : DateTime.MinValue;
        return validDayOfBirth;
    }
}