Question

我有一个方法（有时）以"dddd MMMM dd"格式（04年1月1日星期一）接收需要解析为DateTime的字符串。我有时会说，因为它也可以作为值传递到"Today"或"Tomorrow"。

处理这个的代码很简单：

if (string.Compare(date, "Today", true) == 0)
    _selectedDate = DateTime.Today;
else if (string.Compare(date, "Tomorrow", true) == 0)
    _selectedDate = DateTime.Today.AddDays(1);
else
    _selectedDate = DateTime.Parse(date);

直到12月中旬才开始工作。你们中的一些人可能已经发现了什么问题。

在新年的任何日期，这都会失败并出现错误：

“字符串未被识别为有效的DateTime，因为星期几不正确。”

它已经过了"Monday January 04"，这是2010年的有效日期，但不是在2009年。

所以我的问题是：有没有办法设定今年或明年的年份？现在，作为一个快速而肮脏的解决方案，我有这个：

if (!DateTime.TryParseExact(date, "dddd MMMM dd", CultureInfo.InvariantCulture, DateTimeStyles.None, out _selectedDate))
    if (!DateTime.TryParseExact(date + " " + (DateTime.Now.Year + 1), "dddd MMMM dd yyyy", CultureInfo.InvariantCulture, DateTimeStyles.None, out _selectedDate))
        throw new FormatException("That date is not valid.");

因此它将尝试使用当前年度解析它，如果它不成功，它将在下一年再次尝试。如果它在那之后失败，它只会假设它是一个无效的日期，因为我只需要提前1年担心，但如果有人有更灵活的解决方案，我会很感激。（注意，我不需要担心验证传入的日期，它对当前或下一年都有效。）

Answer 1

首先，您的单元测试应该已经抓住了这一点。您可能希望重新访问为此方法编写的测试，以便从这些经验中学习如何更全面地覆盖您的功能。

其次，您使用String.Compare代替String.Equals是否有任何特殊原因？我认为以下内容更具可读性：

date.Equals("Today", StringComparison.InvariantCultureIgnoreCase);

我认为更清楚地了解正在发生的事情（特别是因为我们不必记住最终bool参数在String.Compare中的含义）。

现在，了解问题的核心。你的方法非常好，非常清楚地表达了逻辑。不过我会做一个小的重构：

public DateTime ParseInThisYearOrNextYear(string s, out DateTime dt)
{
    if (!Parse(s, "dddd MM dd", out dt))
    {
        if (!Parse(s + " " + DateTime.Now.Year + 1, "dddd MM dd yyyy", out dt))
        {
            throw new FormatException();
        }
    }

    return dt;
}

bool Parse(string s, string format, out DateTime dt)
{
    return DateTime.TryParseExact(
        s,
        format,
        CultureInfo.InvariantCulture,
        DateTimeStyles.None,
        out dt
    );
}

这会将您的方法分为两个不同的功能部分，并防止重复自己（CultureInfo.InvariantCulture和DateTimeStyles.None）使测试和维护更容易一些。（你可能想要一个比Parse更好的方法名;我选择了一个短的方法来防止滚动条出现在这里的代码窗口中。）

作为最后一个警告（不知道您系统的详细信息），您可能还需要考虑检查前一年！想象一下以下情况：

输入为“12月31日星期四”（2009年有效）。
系统将1月1日边界滚动到2010年。
执行代码并检查2010年和2011年两者都失败。

根据系统的性质，需要考虑一些事项。

在不知道年份的情况下使用DateTime.TryParseExact

1 个答案: