Czy ktoś może pomyśleć o eleganckim sposobie zmniejszenia tego zagnieżdżonego wyrażenia if-else?

Question

if (Request.QueryString["UseGroups"] != null) 
{ 
    if (Request.QueryString["UseGroups"] == "True") 
    { 
    report.IncludeGroupFiltering = true; 
    } 
    else 
    { 
    report.IncludeGroupFiltering = false; 
    } 
} 
else 
{ 
    report.IncludeGroupFiltering = false; 
} 

Answer 1

Wystarczy jeden czek:

report.IncludeGroupFiltering = Request.QueryString["UseGroups"] == "True"; 

Nie ma potrzeby, aby ocenić Request.QueryString["UseGroups"] dwa razy - to może być tylko równe "True" jeśli jest niezerowe, a porównanie będzie działało idealnie (i zwraca fałsz), jeśli jest to jest równe null.

Wszelkie rozwiązania nadal robi dwie operacje są nadmiernie komplikuje sprawy :)

Answer 2

report.IncludeGroupFiltering = (Request.QueryString["UseGroups"] != null) 
           && (Request.QueryString["UseGroups"] == "True"); 

Answer 3

 
report.IncludeGroupFiltering = Request.QueryString["UseGroups"] == "True"; 

Answer 4

robisz to w zasadzie tak, jak będzie. Wystarczy usunąć zbędny wewnętrzna innego:

if(Request.QueryString["USeGroups"] != null) 
{ 
    if(Request.QueryString["UseGroups"] == "True") 
    report.IncludeGroupFiltering = true; 
} 
else report.IncludeGroupFiltering = false; 

Answer 5

report.IncludeGroupFiltering = Request.QueryString["UseGroups"] == "True" 

Answer 6

Może to:

report.IncludeGroupFiltering = false; 
if (Request.QueryString["UseGroups"] == "True") 
    report.IncludeGroupFiltering = true; 

Answer 7

report.IncludeGroupFiltering = "True" == Request.QueryString["UseGroups"];

Answer 8

Co na temat korzystania TryParse:

bool includeGroupFiltering; 
bool throwaway = Boolean.TryParse(Request.QueryString["UseGroups"], out includeGroupFiltering); 
report.IncludeGroupFiltering = includeGroupFiltering; 

Answer 9

Factor listę Request.QueryString ["UseGroups"] part to mak e jasno chcesz odnosić się do tego samego, a następnie staje się:

string useGroups = Request.QueryString["UseGroups"]; 
report.IncludeGroupFiltering = (useGroups != null) && (useGroups == "True"); 

Answer 10

myślę Request.QueryString["UseGroups"] == "True" i „Prawda” jest ciągiem tylko, że nie zachowuje się jak bool. Więc można napisać w linii

report.IncludeGroupFiltering = string.IsNullOrEmpty(Request.QueryString["UseGroups"])? 
           false : (Request.QueryString["UseGroups"] == "True"); 

Answer 11

ten sposób zrobić tego rodzaju kodu:

report.IncludeGroupFiltering = false; 

if (Request.QueryString["UseGroups"] != null && 
    Request.QueryString["UseGroups"] == "True"  //note that I am not a C# expert - this line /may/ throw an exception if it is indeed null. 
    { 
    report.IncludeGroupFiltering = true; 
    } 

Answer 12

Stałe łańcuchowe są niezmienne i rozpylane, są one również odwoływać obiektów. Jednak wynik Request.QueryString ["UserGroups"] jest referencją do łańcucha (lub odniesieniem do obiektu, który może być niejawnie przekonwertowany na ciąg ...), który może nie być atomizowany, więc nie możesz po prostu porównać referencji, które mogą być odrębne, nawet jeśli struny porównywane są jednakowo.

Operator == na parach ciągów znaków nie porównuje odniesień, ale zawartość ciągów znaków. Oznacza to, że Request.QueryString ["UserGroups"] zostanie dereferencjonowany, a to może spowodować wyjątki dereferencji wskaźnika zerowego. Dlatego istnieje test wstępny dla wartości null (ponieważ testy z "reference == null" NIE WYWOŁUJĄ odwołania, ale w rzeczywistości sprawdzają, czy jest zerowy czy nie).

Istnieje jednak możliwość uniknięcia sprawdzania zerowego: możesz użyć operatora === do porównania referencji, jeśli wynik zapytania .QueryString ["Grupy użytkowników"] został zatomizowany (ale może wymagać przydziału i hashowanie na statycznej liście atomów, nie jest dobrym pomysłem, jeśli QueryString jest bardzo duży.

Więc tak, najlepiej zrobić to pierwszy cache ciągu kwerendy w zmiennej łańcuchowej lokalnym oraz wykonać dwa testy:

final string queryString; // cache the string value 
if ((queryString = Request.QueryString["UserGroups"]) != null && 
    queryString == "True") { 
    ... 
} else { 
    ... 
} 

Ale biorąc pod uwagę, że ciała swoje if/else jest po prostu zapisać wynik if() 's stan, wystarczy napisać tak:

final string queryString; // temporary register caching the non-atomized string reference 
report.IncludeGroupFiltering = 
    (queryString = Request.QueryString["UserGroups"]) != null && 
    queryString == "True"; // compares the two strings contents 

ale tylko jeśli [] zawartość Request.QueryString już rozpyla struny, lub gdy ich niejawna konwersja do powrotu ciąg rozpylonych strun, zapisz ciąg porównuje i używa === zamiast:

final string queryString; // temporary register caching the atomized string reference 
report.IncludeGroupFiltering = 
    (queryString = Request.QueryString["UserGroups"]) != null && 
    queryString === "True"; // compares the atomized references 

nie będę sugerują, że to niebezpieczne założenie tutaj (to jest bardziej prawdopodobne, że wyniki kwerendy z odległego źródła nie zostaną rozpylone, ze względów bezpieczeństwa/pamięci, chyba że zostało już sprawdzone zwrócona wartość. Biorąc pod uwagę twój kod, podejrzewam, że jest to sprawdzanie poprawności zwróconych wartości z twojego zapytania, więc wynik najprawdopodobniej nie jest atomizowany: głównym powodem twojego kodu jest atomizacja zawartości Ciągu Zapytania do wspólnej wartości boolowskiej, która będzie później porównać znacznie łatwiej.

---

Uwaga: Ja absolutnie nie wiem, jaki jest typ wartości lub odniesienie zwrócony przez Request.QueryString [ „Grupy”]. Może się zdarzyć, że jest to obiekt, który implementuje metodę "bool operator == (ciąg)" lub która zwraca inny typ niż bool. Przechowywanie zwróconego obiektu w zmiennej łańcuchowej spowoduje jednak konwersję na ciąg znaków, jeśli typ obiektu nie jest pusty (i jeśli obiekt jest zgodny, w przeciwnym razie wystąpi wyjątek).

Możesz uniknąć tej konwersji nieznanego obiektu, jeśli sam obiekt można porównać do łańcucha jak „True”, z kodu:

report.IncludeGroupFiltering = 
    Request.QueryString["UserGroups"] != null && 
    // uses object's operator==(string) to compare its contents OR reference. 
    Request.QueryString["UserGroups"] == "True"; 

Wszystko to zależy od sposobu ogłoszony QueryString [] właściwość tablicy obiektu Request i jeśli zawartość tablicy może być polimorficzna (zmienny typ). Jeśli wiesz, jak to jest zadeklarowane, użyj dokładnie tego samego typu do zadeklarowania tymczasowego rejestru końcowego powyżej, aby uniknąć podwójnego dostępu do QueryString z żądania i podwójnego indeksowania tablicy QueryString.

Tutaj nie można stwierdzić, który kod będzie najlepszy dla ciebie, ponieważ nie mamy wszystkich deklaracji (C# dziedziczy złożoność/niejasność tego samego typu co C++, ze zbyt wieloma niejawnymi konwersjami i złożonymi schematami dziedziczenia).

Answer 13

report.IncludeGroupFiltering = ShouldIncludeGroupFiltering(Request.QueryString["UseGroups"]) 

private boolean ShouldIncludeGroupFiltering(String queryString) { 
    return ("True" == queryString) 
}