c# – 在不知道年份的情况下使用DateTime.TryParseExact
我有一个方法(有时)采用格式为“dddd MMMM dd”(04年1月1日星期一)的字符串,需要将其解析为DateTime.我有时说,因为它也可以在“今天”或“明天”作为价值传递.
处理这个的代码很简单: 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月中旬.你们中的一些人可能已经发现了什么问题. 这将在新年的任何日期失败并出现错误:
它正在通过“2010年1月4日星期一”,这是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",out _selectedDate)) throw new FormatException("That date is not valid."); 所以它将尝试使用当前年份解析它,如果它不成功,它将在下一年再次尝试.如果它在那之后失败,它只会假设它是一个无效的日期,因为我只需要提前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,out dt ); } 这将您的方法分成两个不同的功能部分,并防止重复自己(CultureInfo.InvariantCulture和DateTimeStyles.None)使测试和维护更容易一些. (你可能想要一个比Parse更好的方法名;我选择了一个短的方法来防止滚动条出现在这里的代码窗口中.) 作为最后一个警告(不知道您的系统的详细信息),您可能还要考虑检查前一年!想象一下以下情况: >输入为“12月31日星期四”(2009年有效). 根据系统的性质,可以考虑一些事项. (编辑:李大同) 【声明】本站内容均来自网络,其相关言论仅代表作者个人观点,不代表本站立场。若无意侵犯到您的权利,请及时与联系站长删除相关内容! |