web-dev-qa-db-ja.com

C#でのカードシャッフル

カードのデッキの内容を一覧表示し、その人がデッキをシャッフルしたい回数を尋ねてから、それらをシャッフルするプロジェクトのコードを作成しようとしています。 System.Randomクラスを使用して2つのランダムな整数を作成するには、メソッドを使用する必要があります。

これらは私のクラスです:

Program.cs:

namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            Deck mydeck = new Deck();
            foreach (Card c in mydeck.Cards)
            {
                Console.WriteLine(c);
            }
            Console.WriteLine("How Many Times Do You Want To Shuffle?");

        }
    }
}

Deck.cs:

namespace ConsoleApplication1
{
    class Deck
    {    
        Card[] cards = new Card[52];
        string[] numbers = new string[] { "2", "3", "4", "5", "6", "7", "8", "9", "J", "Q", "K" };
        public Deck()
        {
            int i = 0;
            foreach(string s in numbers)
            {
                cards[i] = new Card(Suits.Clubs, s);
                i++;

            }
            foreach (string s in numbers)
            {
                cards[i] = new Card(Suits.Spades, s);
                i++;

            }
            foreach (string s in numbers)
            {
                cards[i] = new Card(Suits.Hearts, s);
                i++;

            }
            foreach (string s in numbers)
            {
                cards[i] = new Card(Suits.Diamonds, s);
                i++;

            }
        }

        public Card[] Cards
        {
            get
            {
                return cards;


            }
        }
    }  
}

Enums.cs:

namespace ConsoleApplication1
{        
    enum Suits 
    {
        Hearts,
        Diamonds,
        Spades,
        Clubs
    }
}

Card.cs:

namespace ConsoleApplication1
{
    class Card
    {
        protected Suits suit;
        protected string cardvalue;
        public Card()
        {
        }
        public Card(Suits suit2, string cardvalue2)
        {
            suit = suit2;
            cardvalue = cardvalue2;
        }
        public override string ToString()
        {
            return string.Format("{0} of {1}", cardvalue, suit);
        }
    }
 }

カードを好きなだけシャッフルする方法を教えてから、シャッフルされたカードをリストしてください。

13
Jeff

Fisher-Yatesシャッフル を使用します。

C#コードは次のようになります。

static public class FisherYates
{
    static Random r = new Random();
    //  Based on Java code from wikipedia:
    //  http://en.wikipedia.org/wiki/Fisher-Yates_shuffle
    static public void Shuffle(int[] deck)
    {
        for (int n = deck.Length - 1; n > 0; --n)
        {
            int k = r.Next(n+1);
            int temp = deck[n];
            deck[n] = deck[k];
            deck[k] = temp;
        }
    }
}
50
hughdbrown

カードのデッキをシャッフルすることは、最初は些細なことのように思えますが、通常、ほとんどの人が思い付くアルゴリズムは正しくありません。

Jeff Atwood( Coding Horror )は、このテーマに関するいくつかの非常に優れた記事を書きました。

http://www.codinghorror.com/blog/archives/001008.html

http://www.codinghorror.com/blog/archives/001015.html

(特に2番目は必読です)

17

これは、抽象化に巻き込まれすぎている可能性がある場合だと思います。

ソフトウェアでカードのデッキをシャッフルすることは、ランダムな順序でデッキをユーザーに提供することです。これは、実際には事前にそれらをシャッフルする必要はありません。

デッキを初期化します。 (私は通常、カードを表すために1から52までの数字を使用し、どのカードであるかを数学的に計算します。)

  1. 乱数ジェネレーターを使用して、利用可能なカードのデッキからカードを選ぶことにより、カードを配ります。
  2. そのカードをデッキの最後にあるカードと交換します。
  3. そのカードを山札から取り除くために、山札の端を指すカウンターをデクリメントする。
  4. カードの描画が完了するまで、手順1に進みます。

編集:そして一般的に言って、良い乱数ジェネレーターがあれば、それを複数回「シャッフル」しても何も得られません。

これは、示したデータ構造を使用して可能になるはずです。デッキの終わりを追跡するために、「Draw」メソッドとメンバー変数を追加する必要があります。事前に実際に「シャッフル」を実行することに夢中になっている場合は、Aは教授のジャークであり、Bは52枚のカードを引くたびにデッキがシャッフルされます。すべてのカードを引いたら、「DeckEmpty」メソッドと、すべてのカードを再び含めるようにデッキの終わりをリセットするメソッドを提供する必要があります。

5
NoMoreZealots

デッキを正しくシャッフルするには、Randomクラスだけを使用するのではなく、シードは2 ^ 32しかないため、Randomオブジェクトは52の場合に2 ^ 32(想定)の異なる順序しか与えられません。 (階乗52)実際のデッキを作成する方法。

私は2つのGUIDを使用して32バイトのランダムデータを作成しています-> 4バイトの8つのシードとそれらの8つの異なるシードでカードをシャッフルします

それから種によって私は一定数のカードを手に入れます[5,5,6,6,6,7,8,9]

これが私が使用するコードです

    public void Shuffle(Guid guid1, Guid guid2)
    {
        int[] cardsToGet = new int[] { 5, 5, 6, 6, 6, 7, 8, 9 };
        byte[] b1 = guid1.ToByteArray();
        byte[] b2 = guid2.ToByteArray();

        byte[] all = new byte[b1.Length + b2.Length];
        Array.Copy(b1, all, b1.Length);
        Array.Copy(b2, 0, all, b1.Length, b2.Length);

        List<Card> cards = new List<Card>(this);
        Clear();

        for (int c = 0; c < cardsToGet.Length; c++)
        {
            int seed = BitConverter.ToInt32(all, c * 4);
            Random random = new Random(seed);
            for (int d = 0; d < cardsToGet[c]; d++)
            {
                int index = random.Next(cards.Count);
                Add(cards[index]);
                cards.RemoveAt(index);
            }
        }
    }

あなたのシャッフルはうまくいくかもしれませんが、それは本当に効率的ではなく、本物そっくりではありません。この方法を試す必要があります:

//The shuffle goes like this: you take a portion of the deck, then put them in random places
private void Shuffle()
{
 int length = DeckofCards.Count;
 int level = 20; //number of shuffle iterations

 List<Card> Shuffleing; //the part of the deck were putting back
 Random rnd = new Random();
 int PickedCount, BackPortion; //the last used random number

 for (int _i = 0; _i < level; _i++)
 {
  PickedCount = rnd.Next(10, 30); //number of cards we pick out
  Shuffleing = DeckofCards.GetRange(0, PickedCount);
  DeckofCards.RemoveRange(0, PickedCount);

  while (Shuffleing.Count != 0)
  {
   PickedCount = rnd.Next(10, DeckofCards.Count - 1); //where we place a range of cards
   BackPortion = rnd.Next(1, Shuffleing.Count / 3 + 1); //the number of cards we but back in one step
   DeckofCards.InsertRange(PickedCount, Shuffleing.GetRange(0, BackPortion)); //instering a range of cards
   Shuffleing.RemoveRange(0, BackPortion); //we remove what we just placed back
  }
 }
}

このようにして、より少ない反復でよりリアルなシャッフルを得ることができます

2
Lyozsi

全体として、各デッキをCardオブジェクトの配列を含むオブジェクトと見なします。各Cardオブジェクトには、値とスイートのintプロパティが含まれています。これらのプロパティは、値とスイートの列挙型に適用して、名前付きバージョンを次のように収集できます。使用しているデッキの種類ごとに。 (これにより、このコードのビットがより用途が広くなり、値の比較が容易になります3 <11(ジャック)!〜)あなたのスタイルは学校のプロジェクトで機能します。私はそれでOCDを取得しています!

class Card
{
    public int value
    { get; set; }

    public int suite
    { get; set; }
}


abstract class Deck
{
    public Card[] cards
    { get; set; }

    public void ShuffleCards(int timesToShuffle)
    {
        Card temp;
        Random random = new Random();
         // int timesToShuffle = random.Next(300, 600); #Had it setup for random shuffle
        int cardToShuffle1, cardToShuffle2; 

        for (int x = 0; x < timesToShuffle; x++)
        {
            cardToShuffle1 = random.Next(this.cards.Length);
            cardToShuffle2 = random.Next(this.cards.Length);
            temp = this.cards[cardToShuffle1];

            this.cards[cardToShuffle1] = this.cards[cardToShuffle2];
            this.cards[cardToShuffle2] = temp;
        }
    }
}

これは、基本デッキクラスを使用し、それを必要なデッキのタイプに継承することを前提としています(これにより、同じコードをUnoデッキなどに適用できます)。通常のタイプのデッキクラスのコード。

class NormalDeck : Deck
{
    // This would go in the NormalGame class to apply the enumerators to the values as a cipher.
    // Need int values for logic reasons (easier to work with numbers than J or K !!!
    // Also allows for most other methods to work with other deck<Type> (ex: Uno, Go Fish, Normal cards)
    public enum Suites
    {
        Hearts,
        Diamonds,
        Spades,
        Clover
    };

    // Same comment as above. 
    public enum Values
    { Ace, Two, Three, Four, Five, Six, Seven, Eight, Nine, Ten, Jack, Queen, King };

    public void NewNormalDeck()
    {
        // Clear the deck of cards
        if (this.cards != null)
        {
            Array.Clear(this.cards, 0, this.cards.Length);
        }

        //Set Value to length of Normal deck of Cards without Jokers 
        cards = new Card[52];

        // to keep count of which card we are.  
        int curNumofCards = 0;

        // Cycle through all of the suites listed in "suites" then all the values of     that suite
        for (int x = 0; x < Enum.GetValues(typeof(Suites)).GetLength(0); x++)
        {
            for (int y = 0; y < Enum.GetValues(typeof(Values)).GetLength(0); y++)
            {
                Card newCard = new Card();
                newCard.suite = x;
                newCard.value = y;
                this.cards[curNumofCards] = newCard;
                curNumofCards++;
            }
        }
    }
}
0
user3342574