Perché il progettista WinForms genera un codice un po '"scomodo" nel suo metodo di smaltimento?

Question

Quando si crea una maschera o un controllo utente, il progettista WinForms genera un metodo dispose che assomiglia a questo:

protected override void Dispose(bool disposing) 
    { 
     if (disposing && (components != null)) 
     { 
      components.Dispose(); 
     } 
     base.Dispose(disposing); 
    } 

Il problema con questo codice è che può portare a un comportamento errato se è mai modificato per smaltire oggetti aggiuntivi. Ho visto i file .designer.cs con METODI DI SMALTIMENTO che assomigliano a questo:

protected override void Dispose(bool disposing) 
    { 
     if (disposing && (components != null)) 
     { 
      components.Dispose(); 
      if (_myDisposable != null) 
       _myDisposable.Dispose(); 
      if (_myOtherDisposable != null) 
       _myOtherDisposable.Dispose(); 
     } 
     base.Dispose(disposing); 
    } 

... che non è corretto, in quanto lo smaltimento dei _myDisposable e _myOtherDisposable non dovrebbe dipendere o meno di componenti è nullo.

Quindi, ignorando l'argomento sull'opportunità o meno di modificare questo codice generato dal designer e ignorando il fatto che è possibile modificarlo modificando i modelli, la mia domanda è: perché il progettista non genera codice sembra più questo?

protected override void Dispose(bool disposing) 
    { 
     if (disposing) 
     { 
      if(components != null) 
       components.Dispose(); 
     } 
     base.Dispose(disposing); 
    } 

Questo codice ha lo stesso risultato finale, ma è più sicuro e meno soggetto a errori durante la modifica.

Answer 1

La risposta è: perché la vostra convenienza non era la preoccupazione principale di chi ha scritto questa funzione in Microsoft. O forse pensavano che tu, essendo un dipendente non-Microsoft, non possa essere un buon programmatore, quindi, probabilmente dovresti stare lontano dalle attività rischiose come modificare il metodo Dispose() di un oggetto.

A proposito, il metodo Dispose() si trova al di fuori dell'area all'interno del file .Designer.cs che è designato come "non modificare questo codice generato dal designer", quindi suppongo che sia corretto modificarlo.

Answer 2

Hai ragione soprattutto, e non hai nemmeno menzionato il fatto che è inserito nel file Designer.cs.

Dovrai prima spostarlo (in MyForm.cs) e quindi modificarlo. Con un po 'di buon senso ...

Ma è in gran parte accademico, componenti == null sarà vero solo su un modulo completamente vuoto. Rilascia 1 pulsante o etichetta e il problema non si presenta.

Ho appena controllato, anche su un modulo vuoto non è null. (OK, solo per Fx 4+ a quanto pare)

Answer 3

Il modo consigliato di gestire lo smaltimento di risorse contenute in un modulo consiste nell'utilizzare gli eventi FormClosing o FormClosed. UserControl ha un evento Disposed per lo stesso scopo.

Answer 4

Direi che questo è dovuto al fatto che lo schema "ufficiale" di Microsoft IDisposable tenta di soddisfare troppe situazioni, inutilmente.

Per ulteriori dettagli, vedere questo eccellente articolo di Stephen Cleary: What your mother never told you about IDisposable. Ha alcune informazioni dettagliate sui problemi con IDisposable e su come risolverli.

Stephen offre una semplice linea guida: non combinare le risorse gestite e non gestite in un'unica classe; invece, avvolgere tutte le risorse non gestite in una classe IDisposable il cui unico scopo è quello di eliminare la risorsa non gestita.

Se si segue questa linea guida, tali metodi arcani Dispose cessano di essere necessari, allo stesso tempo risolvono un sacco di altri problemi più gravi con il modello consigliato da Microsoft.