Co jest nie tak z ?? operator używany tak:

Question

Więc insteed pisania:

if (obj.Collection == null) 
    obj.Collection = new Collection(); 
obj.Collection.Add(something); 

myślałem o piśmie:

obj.Collection = obj.Collection ?? new Collection; 
obj.Collection.Add(something); 

To niby czuje się źle, zwłaszcza tę część „obj.Collection = obj.Collection .. . "

Co wy myślicie?

Chodzi,

Answer 1

Gdybym miał wybrać między tymi dwoma blokami kodu, użyłbym tego pierwszego. Jest bardziej czytelny i ma wspólny wzorzec. ?? jest przydatny w scenariuszach, w których należy ustawić wartość domyślną (np. DateTime date = nullableDateTimeInstance ?? defaultDate;).

Ale, szczerze mówiąc, starałbym się nie skończyć w sytuacji, w której chcę dodać do kolekcji i możliwe, że kolekcja jest zerowa. Zamiast tego upewniam się, że kolekcja jest inicjowana w konstruktorze lub w jakimkolwiek innym przypadku.

Answer 2

kodu emitują niepotrzebne przypisania (na obj.Collection do siebie), jeżeli obj.Collection nie jest pusty, ale inne od tego, które jest równoważne z oryginałem.

Wygląda dla mnie dobrze, pod warunkiem, że nie użyjesz go w krytycznych sekcjach. W każdym razie jest możliwe, że kompilator może zoptymalizować zadanie, ale nie wiem, czy to możliwe, czy nie.

Może to po prostu źle, ponieważ oryginalny kod jest tak powszechny i ​​prosty, że źle się go zmienia.

Answer 3

Czy chodziło Ci o:

if (obj.Collection == null) 
    { 
     obj.Collection = new Collection(); 
    } 
obj.Collection.Add(something); 

Jeśli tak, można przerobić go jako

(obj.Collection = (obj.Collection ?? new Collection())).Add(something); 

Answer 4

powiedziałbym, że zależy również od tego, co seter nieruchomości Collection robi. Rozważ to:

public Collection Collection 
{ 
    get { return _collection; } 
    set 
    { 
     _collection = value; 
     Thread.Sleep(1000); // or some expensive real work 
    } 
} 

w tym przypadku cesji obj.Collection = obj.Collection ?? new Collection() byłoby bardzo kosztowne.

Jednakże, jeśli trzeba utworzyć kolekcję „na żądanie”, to powszechnie używa podobny wzór w getter własności, podobnie jak to:

public Collection Collection 
{ 
    get { return _collection ?? (_collection = new Collection()); } 
}