博客园  :: 首页  :: 联系 :: 订阅 订阅  :: 管理

关于隐式类型var使用最佳实践的讨论

Posted on 2009-11-05 11:15  生鱼片  阅读(2101)  评论(12编辑  收藏  举报

在这里http://www.codeproject.com/KB/cs/Best_practise_using_var.aspx看到一篇文章,作者阐述了自己对隐式类型var的想法,简单整理下大家也讨论下:

 

C#3.0带来了一个新的特性隐式类型,使用var关键字定义,如:

Int i=2 就可以写成var i=2

编译器会根据表达式来推断出显示的类型。但是并不是所有的情况都适合使用,比如方法重载的时候就会使我们的代码很难理解,如下面的例子:

private static int GetUserInput()

        {

            Random r = new Random(1);

            return r.Next();

        }

 

        private static int Add(int numberOne, int numberTwo)

        {

            return numberOne + numberTwo;

        }

        private static float Add(float numberOne, float numberTwo)

        {

            return Convert.ToInt32(numberOne + numberTwo);

        }

 

        private static void InvokeAdd()

        {

            var numberOne = GetUserInput();

            var numberTwo = GetUserInput();

            //...

            var addedNumber = Add(numberOne, numberTwo); //version of ‘?Add’¡¥ method is not evident

        }

上面的例子中哪个版本的Add被调用我们就很难一眼看出

 

下面是一些最佳实践

  1. 应该使用var:

  a.匿名类型

var anonymousType = new { Name = "Dilbert" }; 

  b.查询表达式

var queryExpression = from c in customers where c.Name == "Dilbert" select c; 

  c.复杂泛型类型

var searchList = new Dictionary<string>();


  1. 不应该使用var:

a.    已知类型

b.  var customer = new Customer(); //Do not use


var numberList = new int[] { 1, 2, 3, 4, 5 }; //Do not use.


c.    常量

var i = 5; //Do not use


d.    简单表达式赋值

e.  var count = customers.Count();//Do not use


var customerName = customer.Name;//Do not use


f.     类型无法推断

IList<customer> customers = new List(); //Do not use

 

文章的评论中有人提到:var i = 2;int i = 2;只是编译器的一个trick而已,比如下面代码:

 private static void CreateImplicitInteger()
        {
           
var i = 2;
        }

       
private static void CreateExplicitInteger()
        {
           
int i = 2;
        }


 

他们对应的IL是一样的,如下:

.method private hidebysig static void  CreateImplicitInteger() cil managed
{
 
// Code size       4 (0x4)
  .maxstack  1
  .locals init ([
0] int32 i)
  IL_0000:  nop
  IL_0001:  ldc.i4.
2
  IL_0002:  stloc.
0
  IL_0003:  ret
}
// end of method Program::CreateImplicitInteger




and

.method private hidebysig static void  CreateExplicitInteger() cil managed
{
 
// Code size       4 (0x4)
  .maxstack  1
  .locals init ([
0] int32 i)
  IL_0000:  nop
  IL_0001:  ldc.i4.
2
  IL_0002:  stloc.
0
  IL_0003:  ret
}
// end of method Program::CreateExplicitInteger


 

而且该评论者不同意作者推荐对下面代码使用显示类型,而是鼓励使用var隐式类型
var customer = new Customer();

因为这里使用var并不会带来代码的可读性变差,如果我要改变Customer为一个CustomerBase的基类
下面改一个地方代码.
var customer = new Customer();
下面改两个地方代码

Customer customer = new Customer(); 

还有人指出:

var numberList = new int[] { 1, 2, 3, 4, 5 }; //Do not use

名字应该为numberArray,应为var不会改变任何东西。