Inline Jeśli instrukcja - zwarcie

Question

Jak rozumiem i czytam, możesz użyć zwarcia w instrukcji if (& & lub ||), aby drugi warunek nie był uruchamiany. a jeśli chcesz, aby oba warunki zostały uruchomione, użyjesz pojedynczych operandów (& lub |).

więc powiedzieć, jeśli mam inline if jak poniżej:

var test = (MyObject != null || string.IsNullOrEmpty(MyObject.Property)) ? string.Empty : MyObject.Property; 

Ten rzuci object reference błąd, jeśli MyObject jest null, które moim zdaniem nie powinno tak używam zwarciem. Czy ktoś może to wyjaśnić.

Answer 1

Używasz niewłaściwego warunku. Ta część:

MyObject != null || string.IsNullOrEmpty(MyObject.Property) 

powinno być:

MyObject == null || string.IsNullOrEmpty(MyObject.Property) 

RHS o || wykonywana tylko wtedy, gdy lewa ręka jest fałszywe. Chcesz go wykonać tylko wtedy, gdy MyObject jest nie null.

EDIT: Jeśli naprawdę chcesz MyObject != null część, można zmienić całą rzecz do:

var test = MyObject != null && !string.IsNullOrEmpty(MyObject.Property) 
     ? MyObject.Property : ""; 

Uwaga Odwrócenie 2. i 3. argumentów operatora warunkowego zbyt chociaż.

Answer 2

Należy mieć == nie an =

var test = (MyObject == null || string.IsNullOrEmpty(MyObject.Property) ? string.Empty : MyObject.Property 

Answer 3

Spróbuj tego!

var test = (MyObject == null || string.IsNullOrEmpty(MyObject.Property) 
      ? string.Empty : MyObject.Property 

Answer 4

MyObject != null || string.IsNullOrEmpty(MyObject.Property) 

Tutaj mówisz.

Jeśli mój obiekt nie jest pusty. lub string.IsNullOrEmpty (MyObject.Property)

Co oznacza, że ​​jeśli MyObject ma wartość null, spróbuje wykonać drugą część.

MyObject == null || string.IsNullOrEmpty(MyObject.Property) 

To nie rzuci wyjątku null

Answer 5

tak się dzieje, ponieważ MyObject jest null a więc pierwszy warunek jest false więc druga część musi być oceniana poznać całość kondycji. Zmienić linię do tego:

MyObject != null && string.IsNullOrEmpty(MyObject.Property) 

Answer 6

powinny Wolisz czytelność zamiast line-count, np

string prop = string.Empty; 
if(MyObject != null && MyObject.Property != null) 
    prop = MyObject.Property; 

(powód dla wyjątku został już wyjaśniony w innych odpowiedzi)