Выбор случайных чисел в С#

Привет всем, у меня возникла проблема выбора случайных чисел для shuffke колоды карт. Чтобы случайным образом перетасовать, мы попытались выбрать случайную карту в каждом цикле, но кажется, что если у нас кончился один вид, мы не можем выбрать другой случай случайным образом. Он переходит в бесконечный цикл. Какие-либо предложения?

(Перевод: Мака = лопата, синек = клуб, купа = сердце, karo = бриллиант, vale = jack, kiz = queen, papaz = king)

    public class KartKarma : MonoBehaviour {
    public GameObject[] Deste;
    public static GameObject KupaAs, Kupa2, Kupa3, Kupa4, Kupa5, Kupa6, Kupa7, Kupa8, Kupa9, Kupa10, KupaVale, KupaKiz, KupaPapaz;
    public static GameObject KaroAs, Karo2, Karo3, Karo4, Karo5, Karo6, Karo7, Karo8, Karo9, Karo10, Karo11, Karo12, Karo13;
    public static GameObject SinekAs, Sinek2, Sinek3, Sinek4, Sinek5, Sinek6, Sinek7, Sinek8, Sinek9, Sinek10, Sinek11, Sinek12, Sinek13;
    public static GameObject MacaAs, Maca2, Maca3, Maca4, Maca5, Maca6, Maca7, Maca8, Maca9, Maca10, Maca11, Maca12, Maca13;
    public GameObject[] Kupa = {KupaAs, Kupa2, Kupa3, Kupa4, Kupa5, Kupa6, Kupa7, Kupa8, Kupa9, Kupa10, KupaVale, KupaKiz, KupaPapaz};
    public GameObject[] Karo = {KaroAs, Karo2, Karo3, Karo4, Karo5, Karo6, Karo7, Karo8, Karo9, Karo10, Karo11, Karo12, Karo13};
    public GameObject[] Sinek = {SinekAs, Sinek2, Sinek3, Sinek4, Sinek5, Sinek6, Sinek7, Sinek8, Sinek9, Sinek10, Sinek11, Sinek12, Sinek13};
    public GameObject[] Maca = {MacaAs, Maca2, Maca3, Maca4, Maca5, Maca6, Maca7, Maca8, Maca9, Maca10, Maca11, Maca12, Maca13};
    List<GameObject> deste = new List<GameObject>();

// Use this for initialization
void Start () {
    int i = 0;
    while(i < 26){
        int a = Random.Range(1,5);
        if (a == 1) {
            int b = Random.Range(0,13);
            if(Maca != null){
                while(Maca[b] == null){
                    b = Random.Range(0,13);
                }
                deste.Add(Maca[b]);
                Maca[b] = null;
            }
            else {
                while(a == 1)
                    a = Random.Range(2,5);
            }
        }

        if (a == 2) {
            int b = Random.Range(0,13);
            if(Sinek != null){
                while(Sinek[b] == null)
                    b = Random.Range(0,13);
                deste.Add (Sinek[b]);
                Sinek[b] = null;
            }
            else {
                while(a == 2)
                    a = Random.Range(1,5);
            }
        }

         if (a == 3) {
            int b = Random.Range(0,13);
            if(Karo != null){
                while(Karo[b] == null)
                    b = Random.Range(0,13);
                deste.Add (Karo[b]);
                Karo[b] = null;
            }
            else {
                while(a == 3)
                    a = Random.Range(1,5);
            }
        }

        if (a == 4) {
            int b = Random.Range(0,13);
            if(Kupa != null){
                while(Kupa[b] == null){
                    b = Random.Range(0,13);
                }
                deste.Add(Kupa[b]);
                Kupa[b] = null;
            }
            else {
                                    while(a == 4)
                    a = Random.Range(1,5);
            }
        }
        i++;
    }
}

void Update () {

}

}

2
06 февр. '14 в 15:25
источник поделиться
3 ответов

Используйте правильный алгоритм перетасовки, например, Knuth (aka Fisher-Yates):

/// <summary>Used to shuffle collections.</summary>

public class Shuffler
{
    /// <summary>Shuffles the specified array.</summary>
    /// <typeparam name="T">The type of the array elements.</typeparam>
    /// <param name="array">The array to shuffle.</param>

    public void Shuffle<T>(IList<T> array)
    {
        for (int n = array.Count; n > 1; )
        {
            int k = _rng.Next(n);
            --n;
            T temp = array[n];
            array[n] = array[k];
            array[k] = temp;
        }
    }

    private readonly Random _rng = new Random();
}

Затем:

var shuffler = new Shuffler();
...
shuffler.Shuffle(deste);

Поскольку ссылка, опубликованная @Paul, указывает - помните, что Random не является особенно отличным генератором случайных чисел. Вы можете подставить лучший вариант, если хотите, но лежащий в основе алгоритм перетасовки остается прежним.

3
06 февр. '14 в 15:27
источник
while(a == 1)
    a = Random.Range(2,5);

Вот ваш бесконечный цикл. Кроме того, эта часть:

while(Sinek[b] == null)
    b = Random.Range(0,13);

также неприятно, потому что это не гарантирует, что состояние будет когда-либо соблюдено.

2
06 февр. '14 в 15:32
источник

Я думаю, что линии, имеющие

while(a == 1)
while(a == 2)
while(a == 3)
while(a == 4)

вызвав эту проблему. Повторите проверку своей логики.

0
06 февр. '14 в 15:30
источник

Посмотрите другие вопросы по меткам или Задайте вопрос