C# Operator warunkowy Nie oświadczenie?

Question

Mam proste kodu mały fragment, który jest frustrujące mnie:

HashSet<long> groupUIDs = new HashSet<long>(); 
groupUIDs.Add(uid)? unique++ : dupes++; 

w czasie kompilacji, to generuje błąd:

Only assignment, call, increment, decrement, and new object expressions can be used as a statement

HashSet.Add udokumentowaną powrócić bool, więc trójskładnikowej (?) Operator powinien zadziałać, i wygląda to na całkowicie zgodny z prawem sposób śledzenia liczby unikatowych i zduplikowanych elementów dodawanych do zestawu mieszającego.

Po sformatowaniu go jako "if-then-else" działa poprawnie.

Czy ktoś może wyjaśnić błąd i czy istnieje sposób, aby to zrobić jako prosty operator trójskładnikowy?

Answer 1

Zgodnie z komunikatem o błędzie operator trójargumentowy nie może być używany jako instrukcja. trzeba by zrobić coś takiego, aby włączyć go do przypisania:

int dummy = groupUIDs.Add(uid)? unique++ : dupes++; 

Mając na uwadze powyższe, polecam po prostu użyć if-then-else. Jest mniej kłopotliwy, ponieważ nie wiąże się z tworzeniem "magicznych" zmiennych manekinowych ...

Answer 2

Kompilator nie narzeka na Add narzeka na fakt, że wyrażenie warunkowe nie jest kompletnym stwierdzeniem.

Niektóre języki (np. JavaScript) pozwalają na użycie wyrażenia warunkowego w logice gałęzi, tak jak tutaj, ale C# wymaga przypisania wyniku wyrażenia warunkowego do zmiennej. Po przypisaniu wyniku wyrażenia wykonano pełną instrukcję, a kompilator jest zadowolony.

Answer 3

Nie ustawiasz wartości wyniku trójskładnikowego na nic, dlatego.

HashSet<long> groupUIDs = new HashSet<long>(); 
int count = groupUIDs.Add(uid)? unique++ : dupes++; 

Answer 4

Operator trójskładnikowy nie jest instrukcją. Tak, to nie może być stosowany samodzielnie w instrukcji - to odpowiednik pisania

"something that is not a statement"; 

Aby wyjaśnić, należy wyjąć i użyć operatora potrójny if.

Answer 5

Jeśli to nie jest akceptowalne, dlaczego linia ma być? Po prostu użyj instrukcji if :-)

 bool b = false; 
     b?callB():callA(); 

Answer 6

gmcalab i sr pt mają rację; operator trójskładnikowy ma dać ci wynik, tak jak 1 + 1 daje . Nie można po prostu napisać:

1 + 1;

Zamieszanie w tym miejscu (myślę) polega na tym, że myślisz o operatorze trójskładnikowym, jak to jest funkcją.

Answer 7

Musisz użyć wartości z operatora potrójnego dla czegoś ...

HashSet<long> groupUIDs = new HashSet<long>(); 
int newCount = groupUIDs.Add(uid)? unique++ : dupes++; 

lub - wykorzystanie if

HashSet<long> groupUIDs = new HashSet<long>(); 
if (groupUIDs.Add(uid)) 
    unique++; 
else 
    dupes++; 

Answer 8

description z potrójnego operatora w odniesieniu językowej mówi, że

If condition is true, first expression is evaluated and becomes the result; if false, the second expression is evaluated and becomes the result.

Wygląda trójoperandowy mogą być używane tylko w kontekście przydziału, chociaż odniesienie do języka nie określa go explicity. Nie wykonujesz zadania związanego z wynikiem.

Moim zdaniem, ponowne napisanie jako if/else byłoby bardziej przejrzyste.

Answer 9

Jak zauważyli inni, operator warunkowy nie jest wyrażeniem oświadczenia prawnego. (Wyrażenia wyrażenia prawnego to przypisania, wywołania, inkrementy, dekrementy i konstrukcje.)

Istnieje jednak również problem stylistyczny. Moim zdaniem wyrażeń powinny być przydatne dla ich wartości, a instrukcje powinny być przydatne dla ich skutków ubocznych. To, na co natrafiłeś, to to, że masz wyrażenie, które jest użyteczne tylko ze względu na efekt uboczny, a to jest nieprzyjemny zapach kodu.

Masz efekt uboczny, więc użyj instrukcji warunkowej zamiast wyrażenia warunkowego.