La mise en œuvre de IDisposable correctement

Ce serait la mise en œuvre correcte, bien que je ne vois pas de quoi vous jetez dans le code que vous avez posté. Vous avez seulement besoin de mettre en œuvre IDisposable lorsque:

1. Vous avez des ressources non managées
2. Vous êtes en attente pour les références de choses qui sont eux-mêmes jetables.

Rien dans le code que vous avez posté et doit être éliminée.

public class User : IDisposable
{
    public int id { get; protected set; }
    public string name { get; protected set; }
    public string pass { get; protected set; }

    public User(int userID)
    {
        id = userID;
    }
    public User(string Username, string Password)
    {
        name = Username;
        pass = Password;
    }

    //Other functions go here...

    public void Dispose()
    {
        Dispose(true);
        GC.SuppressFinalize(this);
    }

    protected virtual void Dispose(bool disposing)
    {
        if (disposing) 
        {
            //free managed resources
        }
        //free native resources if there are any.
    }

}

L'exemple suivant montre les meilleures pratiques à mettre en œuvre IDisposable interface. Référence

Gardez à l'esprit que vous avez besoin d'un destructeur(finaliseur) uniquement si vous avez les ressources non managées dans votre classe. Et si vous ajoutez un destructeur, vous devez supprimer la Finalisation dans les Disposer, sinon il va provoquer vos objets réside dans la mémoire pour les deux les cycles (Note: Lire comment la Finalisation de travaux). Ci-dessous un exemple d'élaborer tous les ci-dessus.

public class DisposeExample
{
//A base class that implements IDisposable. 
//By implementing IDisposable, you are announcing that 
//instances of this type allocate scarce resources. 
public class MyResource: IDisposable
{
//Pointer to an external unmanaged resource. 
private IntPtr handle;
//Other managed resource this class uses. 
private Component component = new Component();
//Track whether Dispose has been called. 
private bool disposed = false;
//The class constructor. 
public MyResource(IntPtr handle)
{
this.handle = handle;
}
//Implement IDisposable. 
//Do not make this method virtual. 
//A derived class should not be able to override this method. 
public void Dispose()
{
Dispose(true);
//This object will be cleaned up by the Dispose method. 
//Therefore, you should call GC.SupressFinalize to 
//take this object off the finalization queue 
//and prevent finalization code for this object 
//from executing a second time.
GC.SuppressFinalize(this);
}
//Dispose(bool disposing) executes in two distinct scenarios. 
//If disposing equals true, the method has been called directly 
//or indirectly by a user's code. Managed and unmanaged resources 
//can be disposed. 
//If disposing equals false, the method has been called by the 
//runtime from inside the finalizer and you should not reference 
//other objects. Only unmanaged resources can be disposed. 
protected virtual void Dispose(bool disposing)
{
//Check to see if Dispose has already been called. 
if(!this.disposed)
{
//If disposing equals true, dispose all managed 
//and unmanaged resources. 
if(disposing)
{
//Dispose managed resources.
component.Dispose();
}
//Call the appropriate methods to clean up 
//unmanaged resources here. 
//If disposing is false, 
//only the following code is executed.
CloseHandle(handle);
handle = IntPtr.Zero;
//Note disposing has been done.
disposed = true;
}
}
//Use interop to call the method necessary 
//to clean up the unmanaged resource.
[System.Runtime.InteropServices.DllImport("Kernel32")]
private extern static Boolean CloseHandle(IntPtr handle);
//Use C# destructor syntax for finalization code. 
//This destructor will run only if the Dispose method 
//does not get called. 
//It gives your base class the opportunity to finalize. 
//Do not provide destructors in types derived from this class.
~MyResource()
{
//Do not re-create Dispose clean-up code here. 
//Calling Dispose(false) is optimal in terms of 
//readability and maintainability.
Dispose(false);
}
}
public static void Main()
{
//Insert code here to create 
//and use the MyResource object.
}
}

Vous devez utiliser le Jetables Modèle comme ceci:

private bool _disposed = false;
protected virtual void Dispose(bool disposing)
{
if (!_disposed)
{
if (disposing)
{
//Dispose any managed objects
//...
}
//Now disposed of any unmanaged objects
//...
_disposed = true;
}
}
public void Dispose()
{
Dispose(true);
GC.SuppressFinalize(this);  
}
//Destructor
~YourClassName()
{
Dispose(false);
}

Idisposable est de mettre en œuvre chaque fois que vous voulez un déterministe (confirmé) collecte des ordures.

class Users : IDisposable
{
~Users()
{
Dispose(false);
}
public void Dispose()
{
Dispose(true);
GC.SuppressFinalize(this);
//This method will remove current object from garbage collector's queue 
//and stop calling finilize method twice 
}    
public void Dispose(bool disposer)
{
if (disposer)
{
//dispose the managed objects
}
//dispose the unmanaged objects
}
}

Lors de la création et de l'aide aux Utilisateurs de la classe d'usage "à l'aide de" bloquer pour éviter d'appeler explicitement la méthode dispose:

using (Users _user = new Users())
{
//do user related work
}

fin de l'aide du bloc de créer des Utilisateurs de l'objet seront éliminés par implicite invoquer de la méthode dispose.