从网页解析日期

Question

我的前端有两个输入(TextBox)。

下面的代码将它们从字符串表示形式解析为DateTime对象。如果失败，则设置默认值。

DateTime dateFrom;
DateTime dateTo;
if (!DateTime.TryParse(txtDateFrom.Text, out dateFrom)) { dateFrom = DateTime.Today.AddDays(-7); }
if (!DateTime.TryParse(txtDateTo.Text, out dateTo)) { dateTo = DateTime.Now; }

然而，代码看起来很笨重。有更好的方法来写吗？

Answer 1

你将无法改变笨重，但有几件事你可以做：

• 缩进你的代码。它将占用更多的空间，但也将更具可读性：

DateTime dateFrom;
DateTime dateTo;

if (!DateTime.TryParse(txtDateFrom.Text, out dateFrom)) 
{ 
    dateFrom = DateTime.Today.AddDays(-7); 
}
if (!DateTime.TryParse(txtDateTo.Text, out dateTo)) 
{ 
    dateTo = DateTime.Now; 
}

• 等待包含内联变量的C# 6.这样就可以这样编写TryParse：

if (!DateTime.TryParse(txtDateFrom.Text, out DateTime dateFrom))

甚至是

if (!DateTime.TryParse(txtDateFrom.Text, out var dateFrom))

• 使用DateTime.UtcNow。通过使用DateTime.Now，您将依赖于服务器的位置，这可能是将来某个时候在其他地方托管服务器时的问题。
• 考虑在前端而不是文本框中使用日期选择器。这是一个更友好的用户，它将避免无效的日期。

Answer 2

我喜欢Nick Udell的概念，但我更喜欢在这种情况下使用扩展方法，并且避免使用否定的情况--我发现在凌晨2点，当某些东西被破坏时，这些方法比使用正案例更令人困惑。

public static DateTime TryParse(this DateTime defaultValue, string TextToParse) 
{
    DateTime.TryParse(TextToParse, out defaultValue);
    return defaultValue;
}

用法：

DateTime dateFrom = DateTime.Today.AddDays(-7).TryParse(txtDateFrom.Text); 
DateTime dateTo = DateTime.Now.TryParse(txtDateTo.Text);