Singletonパターン - プロパティ

Question

の異なる値Iは、シングルトンパターンを使用して単純なクラスがあります。

public class MyClass 
{ 
    private MyClass() 
    { } 

    private static MyClass instance; 
    private static readonly object locked = new object(); 

    //Singleton 
    public static MyClass Instance 
    { 
     get 
     { 
      lock (locked) 
      { 
       if (instance == null) 
       { 
        instance = new MyClass(); 
       } 
      } 

      return instance; 
     } 
    } 

    public string Rand 
    { 
     get 
     { 
      return new Random().Next(2500).ToString(); 
     } 
    } 
} 

をしかし、私がしなければ：

var instance1 = MyClass.Instance; 
var instance2 = MyClass.Instance; 

instance1.Randとinstance2.Randは異なる値を持っています。

私のシングルトンには2つの異なるインスタンスがあり、これを解決する方法は何ですか？

Answer 1

あなたはランドにアクセスしようとするたびに、それは代わりに、この試すことが新しいランダムインスタンス

を作成させ、getメソッドを呼び出します：あなたは、ランダムな値毎回Randを計算している

private string _rand = return new Random().Next(2500).ToString(); 

public string Rand { 
get { 
    return _rand; 
} 
} 

Answer 2

ですから読んでください。

Answer 3

プライベートフィールドを作成するか、これを作成するたびに同じ値を作成する場合は、毎回新しいランダム値を作成するRandプロパティにアクセスするシングルインスタンスとの違いがあります

private string _rand; 

    public static MyClass Instance 
    { 
     get 
     { 
      lock (locked) 
      { 
       if (instance == null) 
       { 
        instance = new MyClass(); 
        _rand = return new Random().Next(2500).ToString(); 

       } 
      } 

      return instance; 
     } 
    } 

public string Rand { 
get { 
    return _rand; 
} 
} 

Answer 4

ただ、ミックスにいくつかのC＃6を追加し、 の違いのために、.NET 4.

public class MyClass 
     { 
      private static Lazy<MyClass> instance = new Lazy<MyClass>(() => new MyClass()); 

      public static MyClass Instance => instance.Value; 

      public string Rand { get; } = (new Random()).Next(2500).ToString(); 

      private MyClass() { } 
     } 

編集 のLazyを使用しますとpublic string Rand { get; } = (new Random()).Next(2500).ToString(); 最初のコードは、フィールドの初期化中（コンストラクタの前に実行される特別なステップ）に2番目のコードが実行されている間、プロパティ本体（get {すべてが聞こえる}）のコードを実行します。これは、自動プロパティ初期化子と呼ばれます。そして、あなたはあなたがhttps://blogs.msdn.microsoft.com/csharpfaq/2014/11/20/new-features-in-c-6/

NOTEを見ることができます詳細情報については、この

public readonly string rand = (new Random()).Next(2500).ToString(); 

      public string Rand 
      { 
       get 
       { 
        return rand; 
       } 
      } 

を身もだえに同じである：シングルトンは、あなたがそれを使用していないことができればそうAntipatternであると考えられています。