2011-07-29 160 views
6

我有一个枚举,例如enum Color { Red, Brown }。我也有这种类型的一些变量:比较枚举的最佳方法

Color c1 = Brown, c2 = Red 

什么是比较恒定的值,最好的方法:

if (c1 == Color.Brown) { 
    //is brown 
} 

if (c1.equals(Color.Brown)) { 
    //is brown 
} 
+2

两者都很好,但我认为为了简单起见,我用==去。 –

+0

我会用'.equals(...)'去,因为它让我更开心。 :D – Moonbeam

+0

我宁愿把常数放在左边。这样,如果你不小心把=你得到一个编译器错误。 Color.Brown == c1 – QuentinUK

回答

13

使用==。不能有多个相同枚举的实例(在类加载器的上下文中,但让我们忽略该点),所以它总是安全的。

也就是说,使用equals()也是安全的,并且也会执行参考平等。这几乎是一种风格选择。

就我个人而言,我很少发现自己使用if语句来枚举。我赞成switch块。

switch (c1) { 
    case Brown: 
     //is brown 
     break; 
    case Red: 
     //... 
} 
+0

该代码片段使我成为一只伤心的熊猫。 D: – Moonbeam

+1

@Moonbeam:那是为什么?显然这是一个人为的例子,但它是OP的例子,而不是我的例子。 –

+3

使用==,因为枚举变量允许为空。 IMO没有理由使用equals() –