Operatore ternario difficile da leggere

Question

Qualche suggerimento su come rendere la query sottostante più "leggibile"?

var result = result 
       .OrderBy(a => 
        (conditionA) ? 
        valueA : 
        (conditionB ? valueB : 
        (conditionC ? 
        (conditionD ? 
        valueC : valueD) : 
        valueE))); 

È difficile da leggere con il lungo codice di condizione e valore.

Answer 1

Esistono diversi modi per migliorare la leggibilità del codice.

indentazione

Un modo è quello di rientrare il codice in un modo leggermente diverso, ma questo aiuta solo la leggibilità un po ':

var result = result.OrderBy(a => 
    conditionA ? valueA : 
    conditionB ? valueB : 
    conditionC ? conditionD ? valueC : 
           valueD : 
       valueE); 

se, altrimenti

Si potrebbe trasforma anche questi operatori ternari in una catena più leggibile di if, else.

var result = Result.OrderBy(a => { 
    if (conditionA) 
    { 
     return valueA; 
    } 
    else if (conditionB) 
    { 
     return valueB; 
    } 
    else if (conditionC) 
    { 
     if (conditionD) 
     { 
      return valueC; 
     } 
     else 
     { 
      return valueD; 
     } 
    } 
    else 
    { 
     return valueE; 
    } 
}); 

IComparer <>

Una possibilità sarebbe quella di scrivere una propria implementazione di IComparer<> e passarlo al metodo OrderBy. Non so quale sia il tipo di oggetto o il tipo di chiave nel codice, quindi assumerò le chiavi string.

public class MyClassComparer : IComparer<MyClass> 
{ 
    public int Compare(MyClass x, MyClass y) 
    { 
     string xKey = getKey(x); 
     string yKey = getKey(y); 
     return string.Compare(xKey, yKey); 
    } 

    private string getKey(MyClass item) 
    { 
     if (item.conditionA) 
     { 
      return item.valueA; 
     } 
     else if (item.conditionB) 
     { 
      return item.valueB; 
     } 
     else if (item.conditionC) 
     { 
      if (item.conditionD) 
      { 
       return item.valueC; 
      } 
      else 
      { 
       return item.valueD; 
      } 
     } 
     else 
     { 
      return item.valueE; 
     } 
    } 
} 

metodo di estensione

Un ultima opzione sarebbe quella di spostare il codice per un metodo di estensione:

public static class MyClassExtensions 
{ 
    public static string GetSortingKey(this MyClass item) 
    { 
     if (item.conditionA) 
     { 
      return item.valueA; 
     } 
     else if (item.conditionB) 
     { 
      return item.valueB; 
     } 
     else if (item.conditionC) 
     { 
      if (item.conditionD) 
      { 
       return item.valueC; 
      } 
      else 
      { 
       return item.valueD; 
      } 
     } 
     else 
     { 
      return item.valueE; 
     } 
    } 
} 

Utilizzando l'ultima opzione, la chiamata a OrderBy è semplicemente:

result.OrderBy(a => a.GetSortingKey())