ID'yi doğru bir şekilde uygulamak

Sınıflarımda IDisposable'ı aşağıdaki gibi uyguluyorum:

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()
    {
        // Clear all property values that maybe have been set
        // when the class was instantiated
        id = 0;
        name = String.Empty;
        pass = String.Empty;
    }
}

VS2012'de, Kod Analizim ID'yi tek kullanımlık olarak doğru bir şekilde uyguladığını söylüyor, ancak burada yanlış yaptığımdan emin değilim.
Tam metin aşağıdaki gibidir:

CA1063 ID'yi doğru bir şekilde uygulayın 'Kullanıcı'da geçersiz kılınabilen bir Dispose (bool) uygulaması sağlayın veya türü mühürlü olarak işaretleyin. Dispose (yanlış) çağrısı (false) yalnızca yerel kaynakları temizlemelidir. Dispose çağrısı (true) hem yönetilen hem de yerel kaynakları temizlemelidir. stman Kullanıcı.cs 10

Referans için: CA1063: Uygulama kimliği doğru bir şekilde atılabilir

Bu sayfayı okudum, ama korkarım burada ne yapılması gerektiğini gerçekten anlamıyorum.

Herhangi bir kimse sorunun ne olduğunu ve / veya IDisposable nasıl uygulanması gerektiğini daha lamens terimleri ile açıklayabilir, bu gerçekten yardımcı olacaktır!

Gönderdiğiniz kodda atmanız gereken hiçbir şey görmeme rağmen bu doğru uygulama olurdu. Sadece şu durumlarda uygulamanız gerekir IDisposable:

1. Yönetilmeyen kaynaklarınız var
2. Tek kullanımlık olan şeylerin referanslarına tutunuyorsunuz.

Gönderdiğiniz koddaki hiçbir şeyin atılması gerekmez.

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.
    }
}

Her şeyden önce, strings ve ints "temizlemeniz" gerekmez - çöp toplayıcı tarafından otomatik olarak halledilecektir. Temizlenmesi gereken tek şey Disposeyönetilmeyen kaynaklar veya uygulanan yönetilen kaynaklar IDisposable.

Bununla birlikte, bunun sadece bir öğrenme uygulaması olduğunu varsayarsak, uygulamanın önerilen yolu, IDisposablekaynakların iki kez atılmamasını sağlamak için bir "güvenlik mandalı" eklemektir:

public void Dispose()
{
    Dispose(true);

    // Use SupressFinalize in case a subclass 
    // of this type implements a finalizer.
    GC.SuppressFinalize(this);   
}
protected virtual void Dispose(bool disposing)
{
    if (!_disposed)
    {
        if (disposing) 
        {
            // Clear all property values that maybe have been set
            // when the class was instantiated
            id = 0;
            name = String.Empty;
            pass = String.Empty;
        }

        // Indicate that the instance has been disposed.
        _disposed = true;   
    }
}

Aşağıdaki örnek, IDisposablearabirimi uygulamak için genel en iyi uygulamayı gösterir . Referans

Yalnızca sınıfınızda yönetilmeyen kaynaklarınız varsa bir yıkıcıya (sonlandırıcı) ihtiyacınız olduğunu unutmayın. Bir yıkıcı eklerseniz, Atma işleminde Sonlandırmayı bastırmanız gerekir , aksi takdirde nesnelerinizin iki çöp döngüsü için bellekte kalmasına neden olur (Not: Sonlandırmanın nasıl çalıştığını okuyun ). Aşağıdaki örnek yukarıda açıklanmıştır.

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.
    }
}

IDisposableÇöp Toplayıcı tarafından otomatik olarak temizlenmeyecek yönetilmeyen kaynakları temizlemeniz için bir araç sağlamak için mevcuttur .

"Temizlediğiniz" tüm kaynaklar yönetilen kaynaklardır ve bu nedenle Disposeyönteminiz hiçbir şey başaramaz. Sınıfınız hiç uygulamamalı IDisposable. Çöp Toplayıcı tüm bu alanlarla tek başına ilgilenecektir.

Tek Kullanımlık Kalıbı şu şekilde kullanmanız gerekir :

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);
}

Sınıf yönetilmeyen herhangi bir kaynak (dosya, veritabanı bağlantısı vb.) Edinmediği için Usersınıfınızı yapmanıza gerek yoktur . Genellikle, sınıfları en az bir alanı veya / ve özelliği varmış gibi işaretleriz . Uygularken , Microsoft tipik şemasına göre daha iyi koyun:IDisposableIDisposableIDisposableIDisposable

public class User: IDisposable {
  ...
  protected virtual void Dispose(Boolean disposing) {
    if (disposing) {
      // There's no need to set zero empty values to fields 
      // id = 0;
      // name = String.Empty;
      // pass = String.Empty;

      //TODO: free your true resources here (usually IDisposable fields)
    }
  }

  public void Dispose() {
    Dispose(true);

    GC.SuppressFinalize(this);
  } 
}

Idisposable, deterministik (onaylanmış) bir çöp toplama istediğinizde uygulanır.

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
        }
    }

Users sınıfını oluştururken ve kullanırken açıkça dispose yöntemini çağırmaktan kaçınmak için "using" bloğunu kullanın:

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

blok kullanılarak oluşturulan Users nesnesinin sonu, dispose yönteminin örtülü olarak çağrılmasıyla bertaraf edilecektir.

Gerçekten bir anti-desen olan Microsoft Dispose deseninin birçok örneğini görüyorum. Birçoğunun belirttiği gibi, sorudaki kod hiç IDisposable gerektirmez. Ama nereye uygulayacaksanız lütfen Microsoft modelini kullanmayın. Daha iyi cevap bu makaledeki önerileri takip etmek olacaktır:

https://www.codeproject.com/Articles/29534/IDisposable-What-Your-Mother-Never-Told-You-About

Yararlı olabilecek tek şey kod analizi uyarısını bastırmaktır ... https://docs.microsoft.com/en-us/visualstudio/code-quality/in-source-suppression-overview?view=vs- 2017