Operatore condizionale C# non una dichiarazione?

Question

Ho un semplice frammento di codice poco che mi è frustrante:

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

In fase di compilazione, si genera l'errore:

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

HashSet.Add è documentato per restituire un bool, in modo che il ternario (?) l'operatore dovrebbe funzionare, e questo sembra un modo completamente legittimo per tracciare il numero di elementi unici e duplicati che aggiungo a un hash-set.

Quando lo riformatto come if-then-else, funziona correttamente.

Qualcuno può spiegare l'errore e se c'è un modo per farlo come un semplice operatore ternario?

Answer 1

In base al messaggio di errore, l'operatore ternario non può essere utilizzato come istruzione. Si avrebbe bisogno di fare qualcosa di simile per trasformarlo in un incarico:

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

Detto questo, mi consiglia di utilizzare solo if-then-else. È meno confuso perché non comporta la creazione di variabili fittizie "magiche" ...

Answer 2

Il compilatore non si lamenta di Add, si lamenta del fatto che l'espressione condizionale non è un'istruzione completa.

Alcuni linguaggi (come JavaScript) consentono di utilizzare un'espressione condizionale alla logica di diramazione come si è fatto qui ma C# richiede di assegnare il risultato dell'espressione condizionale a una variabile. Una volta assegnato il risultato dell'espressione, hai fatto una dichiarazione completa e il compilatore è felice.

Answer 3

Non stai impostando il valore del risultato del ternario su qualsiasi cosa sia il motivo.

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

Answer 4

L'operatore ternario non è una dichiarazione. Quindi, non può essere usato da solo in un'istruzione - è l'equivalente di scrittura

"something that is not a statement"; 

Per chiarire, si dovrebbe prendere l'operatore ternario e utilizzare un caso.

Answer 5

Se questo non è accettabile, perché dovrebbe essere la tua linea? Basta usare una dichiarazione if :-)

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

Answer 6

gmcalab e sr pt hanno ragione; l'operatore ternario ha lo scopo di darti un risultato, proprio come 1 + 1 ti dà 2. Non si poteva semplicemente scrivere:

1 + 1;

La confusione qui (penso) è che stai pensando all'operatore ternario come se fosse una funzione.

Answer 7

È necessario utilizzare il valore dall'operatore ternario per qualcosa ...

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

o - l'utilizzo di un se

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

Answer 8

La description del operatore ternario in riferimento al linguaggio dice che

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

Sembra che il ternario può essere utilizzato solo nel contesto di incarico, sebbene il riferimento al linguaggio non lo dichiari esplicitamente. Non stai facendo un incarico sul risultato.

A mio parere, la riscrittura come se fosse altrimenti sarebbe più chiara.

Answer 9

Come altri hanno sottolineato, l'operatore condizionale non è un'espressione di dichiarazione legale. (Le espressioni della dichiarazione legale sono assegnazioni, chiamate, incrementi, decrementi e costruzioni.)

Tuttavia, anche qui c'è un problema stilistico. A mio parere, le espressioni dovrebbero essere utili per i loro valori e le dichiarazioni dovrebbero essere utili per i loro effetti collaterali. Quello che stai incontrando è che hai un'espressione che è utile solo per il suo effetto collaterale, e questo è un cattivo odore di codice.

Si ha un effetto collaterale, quindi utilizzare un'istruzione condizionale anziché un'espressione condizionale.