使用隐式类型局部变量

Question

我刚刚安装了 ReSharper 的试用版，我注意到的第一件事就是它总是建议用隐式类型的局部变量替换显式类型的局部变量，例如：

public string SomeMethod(int aParam)
{
    int aNumber = SomeOtherMethod(aParam);
    // should be changed to:
    var aNumber = SomeOtherMethod(aParam);
}

我认为显式类型的变量更具可读性（更明确）。

您对 ReSharper 的建议有何看法？ 使用隐式类型变量有什么优点吗？ 什么时候使用隐式/显式变量？

Answer 1

我个人只在通过阅读声明就可以清楚地区分变量类型时才使用“var”，例如：

var someVariable = new List<int>();

在上面的示例中，很明显“var”指的是“List”。

当我必须去某些方法定义来找出“var”代表什么变量类型或者必须依赖于 Visual Studio Intelli-popup 或任何被调用的东西时，我不喜欢使用“var”，例如我觉得不太好：

var someVaraible = SomeMethod();

我的意思是，“SomeMethod”函数应该返回什么？ 只看这行代码你就能看出来吗？ 不，你不能，所以这就是为什么我在这些情况下避免使用“var”。

Answer 2

关于这一点有很多讨论，但我认为这一切都取决于个人品味，就像几乎在任何地方都使用“this”关键字一样。

我个人更喜欢显式类型变量，但是当使用嵌套泛型集合时，使用隐式类型变量可以变得更具可读性。 看看：

Dictionary<string, Dictionary<string, string>> myDictionary = new Dictionary<string, Dictionary<string, string>>();

vs：

var myDictionary = new Dictionary<string, Dictionary<string, string>>();

编辑：这个SO主题涵盖了相同的主题，并有一些不错的回复： 使用什么：var 或对象名称类型？

编辑2：现在大量使用异步工作，我发现使用显式类型化变量有时可以防止讨厌的错误。 考虑这个愚蠢的示例，您希望返回用户的 ID。 另请考虑 GetUserAsync 返回 Task。 如果您使用隐式类型变量，您最终会使用类似这样的内容：

public long GetUserId()
{
  var user = GetUserAsync();
  return user.Id;
}

这可以编译，但它是错误的。 “user”实际上是一个Task。 它编译为 Task 也有一个 Id 属性。 在这种情况下，人们可能会意外返回任务的 ID，而不是用户。

public long GetUserId()
{
  User user = GetUserAsync();
  return user.Id;
}

上面的代码无法编译，因为编译器会抱怨您无法将任务转换为用户。 添加 await 关键字当然可以解决这个问题。

我实际上曾经遇到过这种情况:-)

Answer 3

以防万一有些人还没有注意到，您可以轻松更改 Reshaper 中的“建议”（Reshaper -> 选项 -> 语言 -> 上下文操作 -> “用 'var' 替换显式类型规范”）。

我个人更喜欢到处都有明确的类型规范，但我对此并不太挑剔。

Answer 4

有时键入 var 伪关键字比键入巨大的类型名称更容易，尤其是在可能涉及泛型的情况下。 但是，您应该知道它们在功能上是相同的。 无论哪种方式，都没有性能差异或任何差异。 编译器派生赋值右侧的类型，并将 var 替换为该类型。 它不像 VB 变体那样在运行时发生。

Answer 5

FWIW，var 关键字在很多情况下都是清晰可读的。 特别是如果...

1. 赋值的右侧是构造函数表达式。

   var map = new Dictionary>();

2. 局部变量有好名字。

华泰