无法获取 shuffle 方法来收集列表

Question

我遇到了这个愚蠢的问题，我不知道如何解决，我正在创建一个二十一点游戏，我有一个带有 Deck() 和 Shuffle() 方法的卡片类，以及一个分发卡片的经销商类.

shuffle 方法是我在这个网站上获得的扩展方法，很有趣，但我无法让它从 Deck() 方法接收卡片列表...

我最初使用字典，但在改组字典时遇到了麻烦，并在此站点上寻求帮助 Here 他们建议使用列表，现在我在这里。

这里是 Card 和 Dealer 类

卡片.cs

public static class Card
{
    private static List<string> deckOfCards = new List<string>();


    private static string[] Suite = new string[4] {"Clubs", "Hearts", "Spades", "Diamonds" };
    private static string[] FaceValue = new string[13] {"Ace", "2", "3", "4", "5", "6", "7", "8", "9", "10", "Jack", "Queen", "King" };


    public static void Deck()
    {
        for (int s = 0; s < 4; s++ )
        {
            string sut = Suite[s];

            for (int fV = 0; fV < 13; fV++)
            {
                string value = FaceValue[fV];

                deckOfCards.Add(sut + value);
            }
        }
        // End of For loop.
        Shuffle(deckOfCards);
    }

    public static void Shuffle<T>(this IList<T> list)
    {
        Random rng = new Random();
        int n = list.Count;
        while (n > 1)
        {
            n--;
            int k = rng.Next(n + 1);
            T value = list[k];
            list[k] = list[n];
            list[n] = value;
        }
    }
}

Dealer.cs

class Dealer
{
    private List<string> randomisedCards = new List<string>();

    public Dealer()
    {
        randomisedCards.Shuffle();
    }


    public string dealCard()
    {
        string randCard = randomisedCards[0];
        randomisedCards.RemoveAt(0);

        return randCard;
    }
}

强烈建议批评，因为这是您学习的方式，但请记住，我仍然是初学者，完全没有经验。

谢谢

Answer 1

我认为将 Deck 保存为类中的静态值是一个可怕的想法，我建议这样做：

public static class Cards
{
    private static string[] Suite = new string[4] {"Clubs", "Hearts", "Spades", "Diamonds" };
    private static string[] FaceValue = new string[13] {"Ace", "2", "3", "4", "5", "6", "7", "8", "9", "10", "Jack", "Queen", "King" };


    public static List<string> CreateDeck()
    {
        var deck = new List<string>();
        for (int s = 0; s < 4; s++ )
        {
            string sut = Suite[s];

            for (int fV = 0; fV < 13; fV++)
            {
                string value = FaceValue[fV];

                deck.Add(sut + value);
            }
        }
        // End of For loop.
        Shuffle(deck);
        return deck;
    }

    private static void Shuffle<T>(this IList<T> list)
    {
        Random rng = new Random();
        int n = list.Count;
        while (n > 1)
        {
            n--;
            int k = rng.Next(n + 1);
            T value = list[k];
            list[k] = list[n];
            list[n] = value;
        }
    }
}

像这样使用它：

class Dealer
{
    private List<string> randomisedCards;

    public Dealer()
    {
        randomisedCards = Cards.CreateDeck();
    }


    public string dealCard()
    {
        string randCard = randomisedCards[0];
        randomisedCards.RemoveAt(0);

        return randCard;
    }
}

请注意，我没有检查您的代码（CreateDeck、Shuffle、...）

我还建议通过以下方式使这个更清洁：

• 为您的Card（Suite、Face）实现一个类
• 为您的Deck（卡片集合）实现一个类
• 使一切都不可改变（在外面 - 意思是：你可以按原样洗牌，但不要让别人改变你的牌组 - 为界面创建新牌 - 例如，庄家抽一张牌应该给你一张牌和另一副牌（缺少这张牌）——这提高了你的代码的可测试性