La mejor forma de comparar enumeraciones

Question

Tengo una enumeración, por ejemplo enum Color { Red, Brown }. También tengo algunas variables de ese tipo:

Color c1 = Brown, c2 = Red 

lo que es mejor manera de comparar a un valor constante:

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

o

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

Answer 1

Use ==. No puede haber varias instancias de la misma constante enum (dentro del contexto de un cargador de clases, pero ignoremos ese punto) para que siempre sea seguro.

Dicho esto, usar equals() también es seguro y también realizará igualdad de referencia. Es más o menos una elección de estilo.

Personalmente, muy rara vez me encuentro usando if declaraciones para enumeraciones en absoluto. Estoy a favor de los bloques switch.

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