编写高质量代码改善C#程序的157个建议——建议8: 避免给枚举类型的元素提供显式的值
建议8: 避免给枚举类型的元素提供显式的值
一般情况下,没有必要给枚举类型的元素提供显式的值。创建枚举的理由之一,就是为了代替使用实际的数值。不正确地为枚举类型的元素设定显式的值,会带来意想不到的错误。
如果为建议7中的枚举类型Week增加一个元素,代码如下所示:
enum Week { Monday = 1, Tuesday = 2, ValueTemp, Wednesday = 3, Thursday = 4, Friday = 5, Saturday = 6, Sunday = 7 }
设想一下ValueTemp的值会是什么?为了验证结果,我们来看这段代码的输出:
Week week = Week.ValueTemp;
Console.WriteLine(week);
Console.WriteLine(week == Week.Wednesday);
输出为:
Wednesday
True
很遗憾,我们明明为Week赋值为ValueTemp,可是得到的结果却是Wednesday。
事实上,如果为枚举类型显式地赋过值,那么很有可能在下个版本中,你为了某些增加的需要,会为枚举添加元素,在这个时候,就像我们为Week增加元素ValueTemp一样,极有可能会一不小心增加一个无效值。
上一个建议中已经讲到如果没有为元素显式赋值,编译器会逐个为元素的值+1。当编译器发现元素ValueTemp的时候,它会自动在Tuesday = 2的基础上+1,所以,实际ValueTemp的值和Wednesday的值都是3。而枚举本身所包括的枚举元素都是值类型,所以产生了上面的输出。
从上面的例子我们应该已经注意到,枚举元素允许设定重复的值。所以,当我们看到下面这段代码的输出时,应该不会感到吃惊:
enum Temp { Value1 = 1, Value2 = 1 }
private static void NewMethod2() { Temp temp1 = Temp.Value1; Temp temp2 = Temp.Value2; Console.WriteLine(temp1 == temp2); Console.WriteLine(temp1.Equals(temp2)); Console.WriteLine(temp1.CompareTo(temp2)); Console.WriteLine(temp1 == Temp.Value1); Console.WriteLine(temp1 == Temp.Value2); }
输出为:
True True 0 True True
注意 本建议也有例外。例如,当为一个枚举类型指定System.FlagsAttribute 属性时,就意味着可以对这些值执行 AND、OR、NOT 和 XOR 按位运算,这样一来,就要求枚举的每个元素的值都是 2 的若干次幂,指数依次递增。如Week的版本就应该为:
[Flags] enum Week { None = 0x0, Monday = 0x1, Tuesday = 0x2, Wednesday = 0x4, Thursday = 0x8, Friday = 0x10, Saturday = 0x20, Sunday = 0x40 } class MyClass { Week week = Week.Thursday | Week.Sunday; }
输出为:
Thursday, Sunday
转自:《编写高质量代码改善C#程序的157个建议》陆敏技