我想使用 c# 随机洗牌包含 10 张牌的列表（稍后将增加到 52 张）。但我有问题。只有一半被洗牌

Question

static List<List<int>> CardDistribution(List<int> Deck,List<int> PlayerCard, List<int> OpponentCard)
{
    int TotsalCards = Deck.Count;
    int CardTaken;
    int IndexofCardTaken;
    int value;

    Random CardTaker = new Random();

    for (int PlC = 0; PlC < TotsalCards; PlC++)
    {
        TotsalCards--;
        if (TotsalCards > 1)
        {
            if (PlC % 2 == 0)
            {
            Redo:
                IndexofCardTaken = CardTaker.Next(0, TotsalCards + 1);
                CardTaken = Deck[IndexofCardTaken];
                if (PlayerCard.Contains(CardTaken))
                {
                    goto Redo;
                }
                else
                {
                    PlayerCard.Add(CardTaken);
                    value = Deck[IndexofCardTaken];
                    Deck[IndexofCardTaken] = Deck[TotsalCards];
                    Deck[TotsalCards] = value;
                }
            }
            else if (PlC % 2 != 0)
            {
            Redo:
                IndexofCardTaken = CardTaker.Next(0, TotsalCards + 1);
                CardTaken = Deck[IndexofCardTaken];
                if (OpponentCard.Contains(CardTaken))
                {
                    goto Redo;
                }
                else
                {
                    OpponentCard.Add(CardTaken);
                    value = Deck[IndexofCardTaken];
                    Deck[IndexofCardTaken] = Deck[TotsalCards];
                    Deck[TotsalCards] = value;
                }
            }
        }
        else if(TotsalCards == 1)
        {
            OpponentCard.Add(Deck[0]);
        }
    }
    List<List<int>> ReturnDecks = new List<List<int>>();
    ReturnDecks.Add(PlayerCard);
    ReturnDecks.Add(OpponentCard);
    return ReturnDecks;
}

我正在使用另外两个函数来填充原始 Deck 列表并检查所有列表都收到了哪些卡片。

我看到只有一半的值被洗牌。
请帮忙，因为我在这里看到了所有其他类似的问题，但它们在这种情况下没有帮助。

Answer 1

洗牌的通用方法称为Fisher Yates Shuffle，很容易在List<T>上实现

static Random rnd = new Random();

public static void Shuffle<T>(List<T> list)
{
    for(var i=0;i<list.Count-2;i++) 
    {
        var j = rnd.Next(i, list.Count);
        var temp = list[i];
        list[i] = list[j];
        list[j] = temp;
    }
}

只需将您的列表传递给该方法，它就会对列表进行洗牌。它适用于 List<int> 或 List<Card> 或您能想到的任何其他列表。

实时示例（使用 10 个数字的列表）：https://dotnetfiddle.net/UaS3wO

所以我认为你的逻辑（如果我理解正确的话）是

1. 洗牌
2. 轮流给玩家和对手和玩家一张牌
3. 归还玩家和对手套牌

--

static List<List<int>> CardDistribution(List<int> Deck,List<int> PlayerCard, List<int> OpponentCard)
{
    Shuffle(Deck);
    for(var i=0;i<Deck.Count;i++)
    { 
        if((i % 2) == 0) PlayerCard.Add(Deck[i])
        else OpponentCard.Add(Deck[i])
    }
    return new List<List<int>>
    {
         PlayerCard,
         OpponentCard
    }
}

注意：实际上没有必要返回玩家和对手的牌组 - 如果您将空的List<int> 传递给上述方法，它将被填满。所以这也可以：

static void CardDistribution(List<int> Deck,List<int> PlayerCard, List<int> OpponentCard)
{
    Shuffle(Deck);
    for(var i=0;i<Deck.Count;i++)
    { 
        if((i % 2) == 0) PlayerCard.Add(Deck[i])
        else OpponentCard.Add(Deck[i])
    }
}