Question

public static bool CheckExpired()
{
    DateTime expiryDate, currentDate = DateTime.Today;
    DateTime.TryParse(date, out expiryDate);

    if (expiryDate > currentDate)
    {
        return true;
    }
    else { return false; }
}

这就是我现在所拥有的。但是，如果日期格式不正确，我希望有第三个选项。就像现在一样，它直接跳到else并返回false。

问题是我想要三个可能的结果：

true - ＆gt;尚未过期

false - ＆gt;已过期

第三 - ＆gt;输入的日期无效

我只是坚持如何到达那里。我知道我可以很容易地使用String返回类型，但是有什么方法吗？

Answer 1

一些选择;

使用bool?类型;那么你可以返回null作为结果。与此相关的是，“null”在您的上下文中并没有真正意义，因此使用不明确
抛出Exception。可能会使用法更清晰（无法通过格式错误的日期），但意味着您需要在所有来电者中使用try-catch
使用enum作为返回值，可以显式设置结果的名称。

我认为枚举可能在这里最有意义，并且对于该方法的消费者来说是最清楚的;

public enum ExpiredResult
{
    Expired,
    NotExpired,
    FormatError,
}

Answer 2

由于DateTime.TryParse()成功后会返回一个bool，你可以触发它。

if(DateTime.TryParse(date, out expiryDate))
{
    if (expiryDate > currentDate)
    {
        return true;
    }
    else { return false; }
}
else
{
    Console.Write("Invalid date.");
}

然后使用Nullable代替bool作为返回类型。

Answer 3

您可以使用NullableBool，

    public static bool? CheckExpired(string date)
    {
        DateTime expiryDate, currentDate = DateTime.Today;
        if (DateTime.TryParse(date, out expiryDate))
        {
            return expiryDate > currentDate;
        }
        return null;
    }

true - ＆gt;尚未过期

false - ＆gt;已过期

null - ＆gt;输入的日期无效

Answer 4

我稍微重构了一下代码。但这可能是一个解决方案：

public static bool CheckExpired()
{
    DateTime expiryDate, currentDate = DateTime.Today;

    if (!DateTime.TryParse(date, out expiryDate))
    {
         throw new Exception("Invalid date");
    }

    return (expiryDate > currentDate);
}

Answer 5

您可以使用可空的bool（bool？）或int

如果字符串无法转换为日期时间，它将返回null

public static bool? CheckExpired()
    {
        DateTime expiryDate, currentDate = DateTime.Today;
        DateTime.TryParse(date, out expiryDate);
        if (expiryDate == new DateTime())
        {
            return null;
        }
        if (expiryDate > currentDate)
        {
            return true;
        }
        else 
        { return false; }
    }

Answer 6

听起来像enum ...

的工作

Answer 7

我会选择枚举。有点像：

public static DateValidationResult CheckExpired(string date)
{
    DateTime expiryDate, currentDate = DateTime.Today;
    if (!DateTime.TryParse(date, out expiryDate))
        return DateValidationResult.InvalidDate;

    return expiryDate > currentDate ? DateValidationResult.Ok : DateValidationResult.Fail;
}

public enum DateValidationResult
{
    InvalidDate,
    Ok,
    Fail
}

Answer 8

试试这个：

public static bool? CheckExpired(string date)
{
    DateTime expiryDate;
    DateTime currentDate = DateTime.Today;

    if (!DateTime.TryParse(date, out expiryDate))
    {
        return null;
    }

    return (expiryDate > currentDate);
}

检查null返回值，或有效的true / false值，如下所示：

string date = "2/4/2013";
bool? isExpired = CheckExpired(date);

if (!isExpired.HasValue)
{
    Console.Write("Invalid date");
}
else if (isExpired.Value)
{
    Console.Write("Expired");
}
else // if (!isExpired.Value)
{
    Console.Write("Valid");
}

Answer 9

就我个人而言，我只是编写一个Enum来封装返回状态，如下所示：

public enum ExipiryStatus
{
    Expired,
    NotExpired,
    InvalidDate
}

public static ExipiryStatus CheckExpired(string date)
{
    DateTime expiryDate, currentDate = DateTime.Today;

    if (DateTime.TryParse(date, out expiryDate))
    {
        if (expiryDate > currentDate)
        {
            return ExipiryStatus.Expired;
        }
        else
        {
            return ExipiryStatus.NotExpired;
        }
    }
    else
    {
        return ExipiryStatus.InvalidDate;
    }
}

对于这种事情，我真的不喜欢可空的布尔。它在调用网站上从未非常清楚返回值应该是什么。有了枚举，它是明确的。

[编辑]被打败了。：）

使用非字符串返回类型进行日期检查 - 设计

9 个答案: