範囲内の非繰り返し整数（除外0）繰り返し整数シーケンスを生成する

Question

generating random numbers without consecutive repetitionを参照して、非繰り返し乱数ジェネレータを2から9までの範囲で作成したい。

私は、上記のURLの回答からラッピングコードをよく使います。あなたがパターンのように、5と9を繰り返しを見ることができるように、別の順序であなたがすべて見ながら

2 
7 
4 
9 

2 
3 
4 
5 

2 
7 
4 
9 

5 
9 
5 
9 

8 
7 
6 
5 

3 
9 
7 
5 

2 
7 
4 
9 

5 
9 
5 
9 

5 
9 
5 
9 

9 
9 
9 
9 

5 
9 
5 
9 

7 
5 
3 
9 

8 
7 
6 
5 

3 
9 
7 
5 

9 
5 
9 
5 

4 
3 
2 
9 

6 
3 
8 
5 

2 
7 
4 
9 

6 
3 
8 
5 

9 
5 
9 
5 

：

int oldrand = <prior random number>; 
int adder = randomNumberGenerator() % 4; 
int newrand = (oldrand + adder + 1) % 5; 

私のコードは、しかし、

using System; 
using System.Collections.Generic; 
using System.Linq; 
using System.Text.RegularExpressions; 
using System.Threading; 

namespace Rextester 
{ 
    public class Program 
    { 
     public static void Main(string[] args) 
     { 
      Random Rand = new Random(); 
      int oldrand = Rand.Next(0,7); 
      //Console.WriteLine(oldrand); 
      for (int i=0; i<50;i++) { 

      int adder = Rand.Next(0,7) % 7; 
      int newrand = (oldrand + adder + 1) % 8+2; 
      int newrand2 = (newrand + adder + 1) % 8+2; 
      int newrand3 = (newrand2 + adder + 1) % 8+2; 
      int newrand4 = (newrand3 + adder + 1) % 8+2; 

      Console.WriteLine(newrand); 
      Console.WriteLine(newrand2); 
      Console.WriteLine(newrand3); 
      Console.WriteLine(newrand4); 
      oldrand = newrand4; 
      Console.WriteLine(); 
      } 
     } 
    } 

    public class Rand { 
     private static readonly Random globalRandom = new Random(); 
     private static readonly object globalLock = new object(); 

     private static readonly ThreadLocal<Random> threadRandom = new ThreadLocal<Random>(NewRandom); 

     public static Random NewRandom() 
     { 
      lock (globalLock) 
      { 
       return new Random(globalRandom.Next()); 
      } 
     } 

     public static Random Instance { get { return threadRandom.Value; } } 

     public static int Next(int minValue, int maxValue) 
     { 
      return Instance.Next(minValue, maxValue); 
     } 

    } 
} 

は奇妙な結果を生成し、 4つの9。

Answer 1

コードの問題は、新しい乱数ごとに新しい乱数を選択する必要があることです。あなたのコードは単一の乱数を何度も何度も再利用するので、数学的に予測可能なシーケンスを得るだけで、どこから始めるのかによって重複が発生します。

ここで動作するコードのバージョンだ（と結果＆hellipをテストし、それはまた私が考える方法で書かれてはるかに読みやすい、再利用可能な、と理解している）：

static void Main(string[] args) 
    { 
     Random random = new Random(); 
     const int max = 8; 

     for (int i = 0; i < 50; i++) 
     { 
      int[] sequence = 
       GetRandomNonConsecutiveSequence(random, max, 4) 
       .Select(j => j + 2) 
       .ToArray(); 

      if (!CheckSequence(sequence)) 
      { 
       Console.WriteLine("Sequence failed: " + string.Join(", ", sequence)); 
      } 
     } 
    } 

    private static bool CheckSequence(int[] sequence) 
    { 
     int previous = sequence[0]; 

     for (int i = 1; i < sequence.Length; i++) 
     { 
      if (previous == sequence[i]) 
      { 
       return false; 
      } 

      previous = sequence[i]; 
     } 

     return true; 
    } 

    private static IEnumerable<int> GetRandomNonConsecutiveSequence(
     Random random, int max, int count) 
    { 
     int previous = random.Next(max); 

     yield return previous; 

     while (--count > 0) 
     { 
      yield return previous = Next(previous, max, random); 
     } 
    } 

    static int Next(int previous, int max, Random random) 
    { 
     return (previous + random.Next(max - 1) + 1) % max; 
    }