我真的想返回最短日期吗？

Question

一位老同事曾经引用他父亲关于工具的话，“你必须比它更聪明。”

在下面的代码中，Resharper 告诉我，“分配的值不用于任何执行路径”（指向第一行）。如果我接受它的帮助，则不会为 dt 分配值（“今天”）。

这是“我必须比它更聪明”并忽略他们的警告的情况，还是这是工具比我更聪明的情况，我只是不理解它？

我对这种情况的看法是，如果 if 语句失败，则返回当前日期（我想要的默认值），但如果我默认 Resharper 的“要求”，它将返回 Datetime 的默认值，即最短日期，我假设是 7/4/1776 或 1/1/0000 左右。

DateTime dt = DateTime.Now;
if (!(DateTime.TryParse(substr, out dt))) {
    using (var dtpDlgForm = new ReturnDate("Please select the Date that the file was created:")) {
        if (dtpDlgForm.ShowDialog() == DialogResult.OK) {
            dt = dtpDlgForm.ReturnVal;
        } 
    }
}
return dt;

Answer 1

您接受的答案显示了您应该做什么，但没有解释为什么 Resharper 首先抱怨。由于此解释可能对发现您的问题的其他人有用，因此这里是：

您应该遵循 Resharper 的建议并将第一行更改为：

DateTime dt;

这声明了变量dt，但没有给它赋值。这里不需要赋值，因为out keyword肯定会在下一行赋值。来自文档：

虽然作为out参数传递的变量在被传递之前不必初始化，被调用的方法需要在方法返回之前赋值。

强调我的。分配值 DateTime.Now 是不必要的，而且会产生误导，因为这个值永远不会被使用。

我对这种情况的看法是，如果 if 语句失败，则返回当前日期

这不是您的代码所做的。来自documentation：

result：当该方法返回时，如果转换成功，则包含与s中包含的日期和时间等效的DateTime值，如果转换失败，则或MinValue。

如果解析失败，则使用您发布的代码，dt 将包含值 DateTime.MinValue，而不是您分配的值 DateTime.Now。

Answer 2

您的预期逻辑允许DateTime 三个可能的返回值（按优先顺序）：

1. substr的解析值。
2. 从对话框中选择的值。
3. 当前日期和时间。

您可以通过在条件成功时执行单独的return 语句来实现此逻辑：

DateTime dt;
if (DateTime.TryParse(substr, out dt))
    return dt;

using (var dtpDlgForm = new ReturnDate(
    "Please select the Date that the file was created:")) 
{
    if (dtpDlgForm.ShowDialog() == DialogResult.OK)
        return dtpDlgForm.ReturnVal;
}

return DateTime.Now;

编辑：有关为什么不应为将用作out 参数的变量赋值的解释，请参阅Mark Byers’s answer。

Answer 3

这是因为您要为 dt 分配一个值，然后将其作为 out param 传递。如果变量作为输出参数传入：

• 必须先在该函数中分配一个值，然后才能在其中使用
• 必须在该函数返回之前为其赋值

在您的情况下，您可以通过将第一行更改为来修复 ReSharper 消息中的消息：

DateTime dt;

我对这种情况的看法是，如果 if 语句失败，则 返回当前日期

这不是真的。无论您传入的 DateTime 对象的值如何，它都会始终返回 DateTime.MinValue。

来自MSDN - DateTime.TryParse(string, out DateTime)：

结果

类型：System.DateTime%

此方法返回时，[result] 包含 DateTime 值 相当于 s 中包含的日期和时间，如果转换 成功，或 MinValue 如果转换失败。转换 如果 s 参数为 null、为空字符串 ("") 或不为空，则失败 包含日期和时间的有效字符串表示。这 参数未初始化传递。

（已添加重点）

Answer 4

out 参数始终具有分配给它的值。始终保证被调用函数在返回之前为其分配一个值。因此，无论如何它都会覆盖最初分配的值。

Answer 5

这里的重点是使用out parameter modifier：

虽然作为输出参数传递的变量在传递之前不必初始化，但被调用的方法需要在方法返回之前赋值。

DateTime.TryParse 无论如何都会为dt 分配一个默认值，如果它无法解析substr，则生成的dt 将具有最小日期时间的值。

Answer 6

此代码将使警告消失。但我认为道格拉斯的答案更容易阅读。

DateTime dt;
if (!(DateTime.TryParse(substr, out dt))) {
    dt = DateTime.Now;
    using (var dtpDlgForm = new ReturnDate("Please select the Date that the file was created:")) {
        if (dtpDlgForm.ShowDialog() == DialogResult.OK) {
            dt = dtpDlgForm.ReturnVal;
        } 
    }
}
return dt;

Answer 7

DateTime.TryParse(substr, out dt);

可以返回 false，但仍会修改 dt。它将尝试尽其所能完成dt，并尽可能地初始化一些值。在 c# 中使用 out 修饰符时，您是在告诉程序对其进行初始化，并且您不应期望保留传入的值。

你能做的是

DateTime dt;
if (!(DateTime.TryParse(substr, out dt))) {
    using (var dtpDlgForm = new ReturnDate("Please select the Date that the file was created:")) {
        if (dtpDlgForm.ShowDialog() == DialogResult.OK) {
            dt = dtpDlgForm.ReturnVal;
        } 
        else {
            dt = DateTime.Now;
        }
    }
}

return dt;