Necesita una mejor manera de comparar valores múltiples contra una propiedad y devolver verdadero;

Estoy buscando una práctica estándar/mejor para escenarios donde necesito verificar la misma propiedad de un objeto contra una lista de valores que devuelven verdadero si alguno de los valores coincide con la propiedad.Necesita una mejor manera de comparar valores múltiples contra una propiedad y devolver verdadero;

Actualmente el código se parece a esto (que no escribo, estoy buscando a refactorizar) ...

if (object.property == "string1" 
        || object.property == "string2" 
        || object.property == "string3" 
         || object.property == "string4" 
         || object.property == "string5" 
           || object.property == "string6" 
           || object.property == "string7" 
            || object.property == "string8" 
            || object.property == "string9" 
             || object.property == "string10" 
             || object.property == "string11" 
              || object.property == "string12" 
              || object.property == "string13" 
               || object.property == "string14" 
               || object.property == "string15")

Fuente

2012-01-27 TXRAckAck

si has cadenas const - que usar la construcción 'switch-case'. –

Todas las respuestas válidas y mucho más bonitas que las que proporcioné. Buen trabajo muchachos. Voy a darle mi voto a Andrew porque parece ser el más legible y debería asignar la menor cantidad de memoria posible, ya que usa una matriz de cadenas. De nuevo Muchas gracias por las excelentes respuestas. – TXRAckAck

IEnumerable<string> items = new List<string>{ "string1", "string2" }; 

bool match = items.Contains(object.property);

Fuente

2012-01-27 23:33:57

impresionante y rápido – TXRAckAck

@TXRAckAck - gracias ;-) –

HashSet sería una mejor solución. O (1) en lugar de O (n). ... suponiendo que reutilice el HashSet , y no lo compile todo el tiempo. – MarkPflug

Usted puede poner los valores en una List<string> y luego hacer esto:

List<string> values = new List<string>() {"string1", "string2"}; 

if(values.Contains(object.Property)) return true;

Fuente

2012-01-27 23:34:19 TheBoyan

Otras respuestas sugieren el uso de un List<string>, pero HashSet<string> es más adecuado para esta tarea:

HashSet<string> set = new HashSet<string>() { "string1", "string2", ..., "string15" }; 

if (set.Contains(object.Property)) 
    //... do something ...

O, como sugiere anatoliiG, dejar que el compilador de manejarlo:

switch (object.property) 
{ 
    case "string1": 
    case "string2": 
    //... 
    case "string15": 
     //... do something ... 
     break; 
}

Fuente

2012-01-27 23:37:07 phoog

Solo asegúrese de reutilizar el HashSet, o no habrá ningún beneficio de rendimiento sobre List . – MarkPflug

Si el hashset ya está calculado, podría ser más rápido que una declaración de cambio gigante (especialmente cierto para conjuntos de datos más grandes). – Matthew

@Mark absolutamente correcto, y gracias por señalarlo. La 'nueva construcción de Foo()' en el código de ejemplo a menudo conduce a un código ineficiente o incorrecto; 'new Random()' es un ejemplo frecuente de eso. – phoog

Usted puede tratar de LINQ para un código más concisa, tales como:

bool match = new string[] { "string1", "string2" }.Any(p => p == object.property);

Fuente

2012-01-27 23:43:24

Votación por muchos motivos. No menos importante es el uso de "Count()> 0" en lugar de Any(); – MarkPflug

Tiene citas en object.property. Eso claramente no es lo que querías decir. Además, para más conciso, ¿qué hay de nuevo [] {"string1", "string2"}. No necesita la "Cadena", se puede inferir. Hecho. – MarkPflug

Muy bueno sobre el hecho de que la cadena se puede inferir. Nunca pensé en eso. Sí, no quise decir citas alrededor de obj.prop. –

Necesita una mejor manera de comparar valores múltiples contra una propiedad y devolver verdadero;

Respuesta

Cuestiones relacionadas