Singleton với tham số

Question

Tôi cần một lớp đơn để được khởi tạo với một số đối số. Con đường tôi đang làm nó bây giờ là:

class SingletonExample 
{ 
    private SingletonExample mInstance; 
    //other members... 
    private SingletonExample() 
    { 

    } 
    public SingletonExample Instance 
    { 
     get 
     { 
       if (mInstance == null) 
       { 
        throw new Exception("Object not created"); 
       } 
       return mInstance; 
     } 
    } 

    public void Create(string arg1, string arg2) 
    { 
     mInstance = new SingletonExample(); 
     mInstance.Arg1 = arg1; 
     mInstance.ObjectCaller = new ObjectCaller(arg2); 
     //etc... basically, create object... 
    } 
} 

Các ví dụ được tạo ra 'muộn', có nghĩa là tôi không có tất cả các đối số cần thiết khi khởi động ứng dụng.

Nói chung tôi không thích bắt buộc đặt hàng các cuộc gọi phương thức, nhưng tôi không thấy cách khác ở đây. IoC cũng sẽ không giải quyết nó, vì tôi có thể đăng ký nó trong thùng chứa, tôi cũng có thể gọi Create() ...

Bạn có coi đây là kịch bản OK không? Bạn có ý tưởng nào khác không?

chỉnh sửa: Tôi biết rằng những gì tôi viết là một ví dụ nó không đề an toàn, thread-safe không phải là một phần của câu hỏi

Answer 1

Một Singleton có tham số có mùi cá.

Hãy xem xét câu trả lời whateva và đoạn mã sau:

Singleton x = Singleton.getInstance("hello", "world"); 
Singleton y = Singleton.getInstance("foo", "bar"); 

Rõ ràng, x == công trình y và y với các thông số tạo ra x, trong khi các thông số tạo y của chỉ đơn giản là bỏ qua. Kết quả có lẽ ... ít gây nhầm lẫn.

Nếu bạn thực sự, thực sự giảm như bạn phải làm điều đó, làm điều đó như thế này:

class SingletonExample 
{ 
    private static SingletonExample mInstance; 
    //other members... 
    private SingletonExample() 
    { // never used 
     throw new Exception("WTF, who called this constructor?!?"); 
    } 
    private SingletonExample(string arg1, string arg2) 
    { 
     mInstance.Arg1 = arg1; 
     mInstance.ObjectCaller = new ObjectCaller(arg2); 
     //etc... basically, create object...  
    } 
    public static SingletonExample Instance 
    { 
     get 
     { 
       if (mInstance == null) 
       { 
        throw new Exception("Object not created"); 
       } 
       return mInstance; 
     } 
    } 

    public static void Create(string arg1, string arg2) 
    { 
     if (mInstance != null) 
     { 
      throw new Exception("Object already created"); 
     } 
     mInstance = new SingletonExample(arg1, arg2);    
    } 
} 

Trong một môi trường đa luồng, thêm đồng bộ để tránh điều kiện chủng tộc.

Answer 2

Tôi thực sự không thể nhìn thấy một singleton trong mã của bạn. Sử dụng phương thức getInstance tĩnh, được tham số hóa trả về singleton và tạo nó nếu nó chưa được sử dụng trước đó. câu trả lời

Answer 3

tốt hơn:

1. Tạo một giao diện: ISingleton (có chứa bất cứ hành động mà bạn muốn nó làm gì để)

2. Và kiểu của bạn: Singleton : ISingleton

3. Giả sử bạn có thể vào UnityContainer:

IUnityContainer _singletonContainer = new UnityContainer(); // or whatever code to initialize the container

1. Khi bạn đã sẵn sàng để tạo ra sử dụng loại bạn (giả sử bạn đang sử dụng Unity cho DI):

_singletonContainer.RegisterType(typeof(ISingleton), new Singleton(params));

1. Nếu bạn muốn lấy singleton chỉ cần sử dụng:

var localSingletonVar = _singletonContainer.Resolve<ISingleton>();

Lưu ý: Nếu vùng chứa không có loại được đăng ký cho giao diện ISingleton, thì nó phải ném ngoại lệ, hoặc trả về null.

Cũ Trả lời:

public class Singleton 
{ 

    private static Singleton instance = null; 

    private Singleton(String arg1, String arg2) 
    { 
    } 

    public static Singleton getInstance(String arg1, String arg2) 
    { 
     if (instance != null) 
     { 
      throw new InvalidOperationException("Singleton already created - use getinstance()"); 
     } 
     instance = new Singleton(arg1, arg2); 
     return instance; 
    } 

    public static Singleton getInstance() 
    { 
     if (instance == null) 
      throw new InvalidOperationException("Singleton not created - use GetInstance(arg1, arg2)"); 
     return instance; 
    } 
} 

Tôi muốn đi với một cái gì đó tương tự (bạn có thể cần phải kiểm tra nếu dụ được tạo ra quá), hoặc, nếu container DI của bạn hỗ trợ ném ngoại lệ đối với các loại phi đăng ký, tôi sẽ đi với điều đó.

ATTN: Mã an toàn không an toàn :)

Answer 4

Singleton là xấu xí nhưng do một người dùng whateva không thể bị làm phiền để sửa mã riêng của mình ...

public class Singleton 
{ 
    private static Singleton _instance = null; 

    private static Object _mutex = new Object(); 

    private Singleton(object arg1, object arg2) 
    { 
     // whatever 
    } 

    public static Singleton GetInstance(object arg1, object arg2) 
    { 
     if (_instance == null) 
     { 
      lock (_mutex) // now I can claim some form of thread safety... 
      { 
       if (_instance == null) 
       { 
        _instance = new Singleton(arg1, arg2); 
       } 
      } 
     } 

     return _instance; 
    } 
} 

Skeet viết blog về năm này trước, tôi nghĩ, đó là khá đáng tin cậy. Không có ngoại lệ cần thiết, bạn không phải trong kinh doanh của việc nhớ những gì các đối tượng được coi là singletons và xử lý bụi phóng xạ khi bạn nhận được nó sai.

Chỉnh sửa: loại không liên quan sử dụng những gì bạn muốn, object chỉ được sử dụng tại đây để thuận tiện.

Answer 5

Giải pháp khóa đơn đôi được cung cấp bởi annakata sẽ không hoạt động mọi lúc trên mọi nền tảng. có một lỗ hổng trong cách tiếp cận này cũng được ghi chép lại. Không sử dụng cách tiếp cận này hoặc bạn sẽ kết thúc với các vấn đề.

Cách duy nhất để giải quyết vấn đề này là sử dụng từ khóa dễ bay hơi, ví dụ:

private static volatile Singleton m_instance = null; 

Đây là phương pháp tiếp cận an toàn chủ đề duy nhất.

Answer 6

Nếu bạn đang sử dụng .NET 4 (hoặc cao hơn), bạn có thể sử dụng loại System.Lazy. Nó sẽ chăm sóc cho bạn các vấn đề an toàn chủ đề và làm điều đó lười biếng, do đó bạn sẽ không tạo ra dụ không cần thiết. Cách này mã ngắn gọn và sạch sẽ.

public sealed class Singleton 
{ 
    private static readonly Lazy<Singleton> lazy = 
     new Lazy<Singleton>(() => new Singleton(),LazyThreadSafetyMode.ExecutionAndPublication); 

    private Singleton() { } 

    public static Singleton Instance { get { return lazy.Value; } } 
}