Comment puis-je prolonger un WinForm la méthode dispose?

J'obtiens ce message d'avertissement de FxCop:

"'RestartForm' contient le champ " RestartForm.fait "qui est de IDisposable type: "ManualResetEvent'. Changement de la méthode dispose sur "RestartForm' à l'appel de la Jeter ou de la Fermer sur ce champ."

Ok, je comprends ce que cela signifie et pourquoi c'est ce qui doit être fait... Sauf System.Windows.Forms.Form ne vous autorise pas à remplacer soit .Close() ou .Dispose(), alors que faire? Actuellement, je suis en cours d'exécution avec cette solution:

    private void RestartForm_FormClosing(object sender, FormClosingEventArgs e)
    {
        done.Set();
        done.Close();
    }

Qui fonctionne comme prévu pour ma demande... Mais FxCop affiche toujours ce message. Suis-je couvert et que je peux l'ignorer, ou est-il une autre façon pour moi de faire cela?

  • Vous pouvez également vous abonner aux Éliminés de l'événement dans votre constructeur, si vous ne voulez pas toucher code généré par le concepteur.
InformationsquelleAutor Matthew Scharley | 2009-06-27
  1. 106

    Vous devez remplacer la Dispose méthode de Form

    Généralement, il est automatiquement remplacé dans les RestartForm.Le concepteur.cs de fichier, de sorte que vous aurez besoin de déplacer le disposer dans votre fichier de code, de sorte que vous pouvez ajouter le code que vous devez ajouter, sans qu'il soit réécrit par le concepteur.

    Dans le RestartForm.cs

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

    //Dispose stuff here
  }

  base.Dispose(disposing);
}
    • C'est exactement ce que j'étais en manque... je n'ai même pas pensé à vérifier si le concepteur a été de le mettre dans son code généré automatiquement... Explique pourquoi il n'est pas sur la liste des dérogations, il a déjà été mis en œuvre, mais pas par moi!
    • Où dois-je placer mon jeter des trucs dans le cas disposing et components != null sont évalués à la même ligne? Par exemple. if (disposing && (components != null)) (par défaut pour vs2013)
    • la logique est la même. Vous avez besoin de déplacer la méthode de votre concepteur de fichier, puis casser les conditions. votre code devrait ressembler à mon exemple ci-dessus lorsque vous avez terminé.
    • Ok, merci de prendre soin de répondre.
    InformationsquelleAutor heavyd
  2. 8

    J'utilise cette méthode 🙂

                Image bgImage = Image.FromFile(workingDir + "\\" + button.BackgroundImage);
            currentButton.Image = bgImage;
            currentButton.Disposed += (Object sndr, EventArgs evnt) => bgImage.Dispose();
    • N'est-ce pas juste vous laisser avec un récepteur d'événements pour vous désinscrire, vous laissant ouvert à une fuite de ressources, puisque vous avez déjà éliminés de la forme?
    InformationsquelleAutor AlexTheo
  3. 2

    Si RestartForm s'étend Système.De Windows.Les formulaires.Forme, vous devriez être en mesure de remplacer dispose(bool disposing). Vous devez implémenter correctement ce pour votre "RestartForm" classe pour vous débarrasser de vos IDisposables.

    Il devrait ressembler à:

    public override Dispose(bool disposing)
{
   if (disposing)
   {
       //Dispose was called from user code. Dispose of managed resources here.
       done.Dispose();
   }

   //Dispose of unmanaged resources here, and invoke base dispose.
   base.Dispose(disposing);
}
    InformationsquelleAutor womp
  4. 0

    Vous avez besoin pour remplacer la méthode dispose, cette méthode provient du de la classe de base

    protected override void Dispose(bool disposing)
{
  if (disposing)
  {
    event.Dispose();
  }
  base.Dispose(disposing);
}
    InformationsquelleAutor Shay Erlichmen