Soppressione degli avvisi "non è mai usato" e "non è mai assegnato a" in C #

Ho un file HTTPSystemDefinitions.cs nel progetto C # che sostanzialmente descrive l'ISAPI di Windows precedente per l'utilizzo da parte del codice gestito.

Ciò include il set completo di strutture rilevanti per ISAPI non tutte o che vengono utilizzate dal codice. Durante la compilazione tutti i membri del campo di queste strutture causano un avviso come il seguente: -

Il campo di avviso "UnionSquare.ISAPI.HTTP_FILTER_PREPROC_HEADERS.SetHeader" non è mai assegnato e avrà sempre il valore predefinito null

o

Attenzione Il campo 'UnionSquare.ISAPI.HTTP_FILTER_PREPROC_HEADERS.HttpStatus' non viene mai utilizzato

Possono essere disabilitati con #pragma warning disable? In caso affermativo, quali sarebbero i numeri di errore corrispondenti? In caso contrario, c'è qualcos'altro che posso fare? Tieni presente che ho solo cosa fare per questo file, è importante che riceva avvisi come questi provenienti da altri file.

modificare

Struttura di esempio: -

struct HTTP_FILTER_PREPROC_HEADERS
{
    //
    //  For SF_NOTIFY_PREPROC_HEADERS, retrieves the specified header value.
    //  Header names should include the trailing ':'.  The special values
    //  'method', 'url' and 'version' can be used to retrieve the individual
    //  portions of the request line
    //

    internal GetHeaderDelegate GetHeader;
    internal SetHeaderDelegate SetHeader;
    internal AddHeaderDelegate AddHeader;

    UInt32  HttpStatus;               // New in 4.0, status for SEND_RESPONSE
    UInt32  dwReserved;               // New in 4.0
}

Sì, questi possono essere soppressi.

Normalmente sono contrario a sopprimere gli avvisi, ma in questo caso le strutture usate per l'interoperabilità richiedono assolutamente che alcuni campi siano presenti, anche se non si intende mai (o si può) usarli, quindi in questo caso penso che dovrebbe essere giustificato .

Normalmente, per sopprimere questi due avvisi, dovresti correggere il codice offensivo. Il primo ("... non viene mai usato") di solito è un odore di codice di avanzi delle versioni precedenti del codice. Forse il codice è stato eliminato, ma i campi sono rimasti indietro.

Il secondo è solitamente un odore di codice per i campi utilizzati in modo errato. Ad esempio, potresti riscrivere in modo errato il nuovo valore di una proprietà nella proprietà stessa, senza mai scrivere nel campo sottostante.

Per eliminare gli avvisi per "Il campo XYZ non viene mai utilizzato ", procedi come segue:

#pragma warning disable 0169
... field declaration
#pragma warning restore 0169

Per sopprimere gli avvisi per "Il campo XYZ non è mai assegnato e avrà sempre il suo valore predefinito XX ", procedi come segue:

#pragma warning disable 0649
... field declaration
#pragma warning restore 0649

Per trovare da soli tali numeri di avviso (ad es. Come facevo a sapere di usare 0169 e 0649), fai questo:

• Compila il codice normalmente, questo aggiungerà alcuni avvisi all'elenco degli errori in Visual Studio
• Passa alla finestra Output e all'output Build e cerca gli stessi avvisi

• Copia il codice di avviso a 4 cifre dal messaggio pertinente, che dovrebbe assomigliare a questo:

  C: \ Dev \ VS.NET \ ConsoleApplication19 \ ConsoleApplication19 \ Program.cs (10,28): avviso CS 0649 : il campo 'ConsoleApplication19.Program.dwReserved' non è mai assegnato e avrà sempre il suo valore predefinito 0

Avvertenza : come da commento di @Jon Hanna , forse alcuni avvertimenti sono necessari per questo, per i futuri cercatori di questa domanda e risposta.

• Innanzitutto, l'atto di sopprimere un avvertimento è simile a deglutire pillole per il mal di testa. Certo, a volte potrebbe essere la cosa giusta da fare, ma non è una soluzione universale. A volte, un mal di testa è un vero sintomo che non dovresti mascherare, lo stesso con gli avvertimenti. È sempre meglio cercare di trattare gli avvisi risolvendone la causa, invece di rimuoverli ciecamente dall'output di compilazione.
• Detto questo, se hai bisogno di sopprimere un avvertimento, segui lo schema che ho esposto sopra. La prima riga di codice #pragma warning disable XYZK,, disabilita l'avviso per il resto di quel file , o almeno finché non #pragma warning restore XYZKviene trovato un corrispondente . Riduci al minimo il numero di righe su cui disabiliti questi avvisi. Il modello sopra disabilita l'avviso per una sola riga.
• Inoltre, come dice Jon, un commento sul perché lo stai facendo è una buona idea. Disabilitare un avviso è sicuramente un odore di codice se fatto senza motivo, e un commento impedirà ai futuri manutentori di passare il tempo a chiedersi perché lo hai fatto, o anche rimuoverlo e provare a correggere gli avvisi.

Un'altra "soluzione" per correggere questi avvisi è creare la struttura public. Gli avvisi non vengono quindi emessi perché il compilatore non può sapere se i campi vengono utilizzati (assegnati) all'esterno dell'assembly.

Detto questo, i componenti di "interoperabilità" di solito non dovrebbero essere pubblici, ma piuttosto internalo private.

Ho ottenuto VS per generare lo scheletro di implementazione System.ComponentModel.INotifyPropertyChangede gli eventi sono stati implementati come campi che hanno attivato gli avvisi CS0067.

In alternativa alla soluzione data nella risposta accettata ho convertito i campi in proprietà e l'avviso è scomparso .

Ciò ha senso poiché la sintassi delle dichiarazioni di proprietà sugar viene compilata in un campo più metodi getter e / o setter (aggiungi / rimuovi nel mio caso) che fanno riferimento al campo. Questo soddisfa il compilatore e gli avvisi non vengono generati:

struct HTTP_FILTER_PREPROC_HEADERS
{
    //
    //  For SF_NOTIFY_PREPROC_HEADERS, retrieves the specified header value.
    //  Header names should include the trailing ':'.  The special values
    //  'method', 'url' and 'version' can be used to retrieve the individual
    //  portions of the request line
    //

    internal GetHeaderDelegate GetHeader {get;set;}
    internal SetHeaderDelegate SetHeader { get; set; }
    internal AddHeaderDelegate AddHeader { get; set; }

    UInt32 HttpStatus { get; set; }               // New in 4.0, status for SEND_RESPONSE
    UInt32 dwReserved { get; set; }               // New in 4.0
}

Gli utenti C / C ++ devono (void)var;sopprimere gli avvisi di variabili inutilizzate. Ho appena scoperto che puoi anche sopprimere gli avvisi di variabili inutilizzate in C # con operatori bit per bit:

        uint test1 = 12345;
        test1 |= 0; // test1 is still 12345

        bool test2 = true;
        test2 &= false; // test2 is now false

Entrambe le espressioni non producono avvisi di variabili inutilizzate nei compilatori VS2010 C # 4.0 e Mono 2.10.