[英]Using DateTime.TryParseExact without knowing the year
我有一个方法(有时)采用格式为"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年"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年担心,但如果有人有更灵活的解决方案,我会很感激。 (注意,我不需要担心验证传入的日期,它对当前或下一年有效)。
首先,你的单元测试应该抓住这个。 您可能希望重新访问为此方法编写的测试,以便从这种经验中学习如何更全面地覆盖您的功能。
第二,你是否有任何特殊原因使用String.Compare
而不是String.Equals
? 我认为以下更具可读性:
date.Equals("Today", StringComparison.InvariantCultureIgnoreCase);
我认为它更清楚地读取正在发生的事情(特别是因为我们不必记住最终的bool
参数在String.Compare
中的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
更好的方法名;我选择了一个短的方法来防止滚动条出现在这里的代码窗口中。)
作为最后一个警告(不知道您的系统的详细信息),您可能还要考虑检查前一年! 想象一下以下情况:
根据系统的性质,可以考虑一些事项。
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.