L'analisi del codice non mostra l'avviso CA1804 nonostante la variabile di stringa locale non utilizzata in C# (VS2010 Premium)

Question

Ho il seguente codice che fa non produce un avviso CA1804 (la variabile dichiarata non viene mai utilizzata) dall'analisi del codice (VS2010 Premium):

... 
if(boolVariable) 
{ 
    string errorText = "Bla Bla Bla"; // Never used 
    ErrorProvider.SetError(SomeControl, "Some Warning"); 
} 
else 
{ 
    string errorText = "Acme Acme Acme"; // Used below 
    ErrorProvider.SetError(SomeControl, errorText); 
} 
... 

Quando rimuovo il ErrorProvider.SetError (...) linee, viene visualizzato l'avviso CA1804, ma perché non è questo il caso nel codice di esempio di cui sopra?

(Btw: il codice in sé non è troppo grande e solo scopo di illustrare la mia domanda.)

Tutte le idee che potrebbero causare questo comportamento? Immagino che questo potrebbe essere dovuto al fatto che il codice IL è ottimizzato in modo tale da mettere la dichiarazione al di fuori dello se, che a sua volta significherebbe che l'avviso non dovrebbe apparire in un esempio come quello sopra, ma Non sono sicuro se sia vero.

Grazie in anticipo

G.

Answer 1

Questo sarebbe a causa di un'ottimizzazione eseguita dal compilatore C#. Nella IL generata, la dichiarazione variabile viene issata dal blocco if:

string errorText; 
if (boolVariable) 
{ 
    errorText = "Bla Bla Bla"; 
    this.ErrorProvider.SetError(this.SomeControl, "Some Warning"); 
} 
else 
{ 
    errorText = "Acme Acme Acme"; 
    this.ErrorProvider.SetError(this.SomeControl, errorText); 
} 

Rimozione solo la seconda chiamata SetError è effettivamente sufficiente per innescare CA1804.

BTW, il compilatore C# dovrebbe aver emesso un avviso CS0219 per questo, apparentemente ignorato. Raccomanderei caldamente di considerare gli avvertimenti del compilatore come errori se si è interessati alla qualità del codice da remoto, cosa che si presume se si sta eseguendo l'analisi del codice. Perché aggiungere uno strumento di screening aggiuntivo prima di trarre il massimo vantaggio da quello che stai già utilizzando?