Cài đặt Singleton này có đúng và chỉ an toàn không?

Question

Việc triển khai singleton này có chính xác và an toàn không?

class Class 
{ 
    public static readonly Class Instance; 

    static Class() 
    { 
     Instance = new Class(); 
    } 

    private Class() {} 
} 

Answer 1

Về mặt kỹ thuật, phiên bản của bạn nên làm việc. Tuy nhiên, tôi sẽ không khuyên bạn nên để lộ một trường công khai trong lớp Singleton của bạn, và thích sử dụng một thuộc tính (chỉ với một getter). Điều này sẽ giúp chứng minh API của bạn trong tương lai nếu bạn cần thực hiện các thay đổi sau này. Tôi cũng khuyên bạn nên niêm phong bất kỳ thực hiện singleton, như subclassing một lớp singleton là hầu như luôn luôn là một ý tưởng tồi và có vấn đề.

tôi sẽ, cá nhân, sử dụng như sau trong C#, nếu bạn đang nhắm mục tiêu NET 3.5 hoặc sớm hơn:

public sealed class Singleton 
{ 
    static readonly Singleton instance = new Singleton(); 

    public static Singleton Instance 
    { 
     get 
     { 
      return instance; 
     } 
    } 

    static Singleton() { } 
    private Singleton() { } 
} 

Nếu bạn đang sử dụng .NET 4, bạn có thể làm cho điều này dễ dàng hơn cho chính mình qua Lazy<T>:

public sealed class Singleton 
{ 
    private static readonly Lazy<Singleton> instance = new Lazy<Singleton>(() => new Singleton()); 
    private Singleton() {} 
    public static Singleton Instance { get { return instance.Value; } } 
} 

phiên bản .NET 4 cũng có lợi thế là hoàn toàn lười biếng - ngay cả khi lớp Singleton bạn có phương pháp tĩnh khác được sử dụng trước khi truy cập của "Instance" bất động sản. Bạn cũng có thể làm một phiên bản .NET 3.5- hoàn toàn lười biếng, bằng cách sử dụng một lớp lồng nhau riêng. Jon Skeet demonstrated this on his blog.

Answer 2

thảo luận tốt về làm thế nào để làm điều đó là ở đây:

http://www.yoda.arachsys.com/csharp/singleton.html

Answer 3

Yes. Tôi cũng sẽ làm cho lớp 'sealed' để tránh bất kỳ sự nhầm lẫn nào trong tương lai.

Answer 4

Bạn nên làm như khởi tạo trong khai báo biến:

public static readonly Class Instance = new Class();