声明变量并使用 TryParse 在同一行初始化它有任何问题吗？

Question

这个示例是用 C# 编写的，但我希望可以同样轻松地应用于其他示例。

我最近发现以下内容似乎工作得很好：

int i = Int32.TryParse(SomeString, out i) ? i : -1;

不知何故，变量i在技术上似乎在它出现在TryParse中时不应该被访问。或者我是否正确地假设 int i 有效地声明了变量，即使还没有语句结束？

Answer 1

int i 声明变量，并在 out 参数中使用它来初始化它。由于必须在结果之前评估谓词，因此 i 在使用之前声明并初始化。 （out 参数必须在返回之前分配，因此在任何情况下它都肯定会被初始化。）

也就是说，我的一些同事会因风格原因看到类似的东西而大发雷霆。 :-)

编辑： 在调查了这是如何发生的之后，我将提出一些可能的替代辅助方法。静态类的命名充当此处辅助方法的意图文档。

internal static class TryConvert
{
    /// <summary>
    /// Returns the integer result of parsing a string, or null.
    /// </summary>
    internal static int? ToNullableInt32(string toParse)
    {
        int result;
        if (Int32.TryParse(toParse, out result)) return result;
        return null;
    }

    /// <summary>
    /// Returns the integer result of parsing a string,
    /// or the supplied failure value if the parse fails.
    /// </summary>
    internal static int ToInt32(string toParse, int toReturnOnFailure)
    {
        // The nullable-result method sets up for a coalesce operator.
        return ToNullableInt32(toParse) ?? toReturnOnFailure;
    }
}

internal static class CallingCode
{
    internal static void Example(string someString)
    {
        // Name your poison. :-)
        int i = TryConvert.ToInt32(someString, -1);
        int j = TryConvert.ToNullableInt32(someString) ?? -1;

        // This avoids the issue of a sentinel value.
        int? k = TryConvert.ToNullableInt32(someString);
        if (k.HasValue)
        {
            // do something
        }
    }
}

Answer 2

我最近发现以下似乎工作得很好

 int i = Int32.TryParse(SomeString, out i) ? i : -1;

它工作，但它并不好。

声明变量并使用 TryParse 在同一行上初始化它有任何问题吗？

是的，可读性。我认为这看起来很糟糕，而且它正在做双重工作。

---

您的部分问题是您希望 -1 作为默认值。 Int32.TryParse 显式定义 0 作为转换失败时的输出值。

为了便于阅读，我仍然会将其分成两行。

int i;
if (! int.TryParse(SomeString, out i))  i = -1;

当您经常需要这个时，请编写一个（静态但不是扩展）辅助方法：

int i = Utils.ParseInt(SomeString, -1);

Answer 3

请记住，CIL 中没有三元运算符。

int i = Int32.TryParse(SomeString, out i) ? i : -1;

您的代码将转换为代表以下 C# 代码的 CIL：

int i;
if (Int32.TryParse(SomeString, out i))
  i = i;
else
  i = -1;

这完全没问题。