C# と VB.NET の質問掲示板

学校のクラブでカードゲームを作っているのですがなぜか配列の順番にカードが表示されません、どなたか助けてください・・・・
ちなみにゲームは次のカードが今のカードより低いか高いかを当てるやつです。

開発環境はC#Express
使用言語はC#です。


コードは以下

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
using System.IO;
using System.Runtime.InteropServices;

namespace TestCards
{
    public partial class Form1 : Form
    {
        Cards myCards = new Cards();
        Boolean myDraw = true;
        //Int32 cardCount = 0;
        
        public Form1()
        {
            InitializeComponent();        
        }

        private void Form1_Paint(object sender, PaintEventArgs e)
        {
            if (myDraw)
            {
                Point myPoint = new Point(75, 50);
                //if (cardCount >= 0)
                //{
                    myCards.DrawCard(e, myCards.GetNewCard(), myPoint);
                    //myPoint.X += 105;
                //}
            }
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            
        }

        private void hButton_Click(object sender, EventArgs e)
        {
            myDraw = true;
            this.Refresh();
            int prevNum = myCards.previousNum();
            int newNum = myCards.GetNewCard();

            if (newNum > prevNum)
            {
                MessageBox.Show(this, "You Win!!");
            }
            else
            {
                MessageBox.Show(this,"You Lose!!");
            }
        }

        private void lButton_Click(object sender, EventArgs e)
        {
            this.Refresh();
            int prevNum = myCards.previousNum() % 12;
            int newNum = myCards.GetNewCard() % 12;
            //int v = x % 12;
            //int z = y % 12;
            if (newNum < prevNum)
            {
                MessageBox.Show(this, "You Win!!");
            }
            else
            {
                MessageBox.Show(this,"You Lose!!");
            }
        }

        public class Cards
        {
            const int width = 79;
            const int height = 123;
            static Random _random = new Random();
            int[] cardArray = new int [52];
            int deckCardNumber;
            int previousCardNumber;

            public Cards()
            {
                deckCardNumber = 0;
                for (int i = 0; i < 52; i++)
                    cardArray[i] = i;
                Shuffle(cardArray);
            }

            public int GetNewCard()
            {
                deckCardNumber++;
                deckCardNumber %= 52;
                return cardArray.ElementAt(deckCardNumber);
            }
            public int previousNum()
            {
                deckCardNumber++;
                deckCardNumber %= 52;
                return cardArray.ElementAt(deckCardNumber-1);
            }
                

            
            //public int nextNum()
            //{
            //    object n = cardArray[1];
            //    return cardArray[1];
            //}

            public void DrawCard(PaintEventArgs e, int cardNumber, Point p)
            {
                // Create a Bitmap object from a file.  
                Bitmap myBitmap = new Bitmap("..//..//cards.png");
                int xOffset = 0; // TBD - Need to handle all cards
                int yOffset = 0;    // TBD - Need to handle all cards

                if ((cardNumber <= 12) && (cardNumber >= 0))
                {
                    xOffset = +cardNumber * width; // TBD - Need to handle all cards
                }
                else if ((cardNumber <= 25) && (cardNumber >= 13))
                {
                    xOffset = +(cardNumber - 13) * width;
                    yOffset = +123;
                }
                else if ((cardNumber <= 38) && (cardNumber >= 26))
                {
                    xOffset = +(cardNumber - 26) * width;
                    yOffset = +246;
                }
                else if ((cardNumber <= 51) && (cardNumber >= 39))
                {
                    xOffset = +(cardNumber - 39) * width;
                    yOffset = +369;
                }
                else if ((cardNumber <= 53) && (cardNumber >= 52))
                {
                    xOffset = +(cardNumber - 52) * width;
                    yOffset = +492;
                }
                else if (cardNumber >= 54)
                {
                    xOffset = +158;
                    yOffset = +492;
                }







                // Clone a portion of the Bitmap object.
                Rectangle cloneRect = new Rectangle(xOffset, yOffset, width, height);
                System.Drawing.Imaging.PixelFormat format = myBitmap.PixelFormat;
                Bitmap cloneBitmap = myBitmap.Clone(cloneRect, format);

                // Draw the cloned portion of the Bitmap object.
                e.Graphics.DrawImage(cloneBitmap, p);
            }

            public static void Shuffle<j>(j[]array)
            {
                var random =  _random;
               
                for (int i = array.Length; i > 1; i--)
                {
                    // Pick random element to swap.
                    int k = random.Next(i);
                    // 0 <= j <= i-1
                    // Swap.
                    j tmp = array[k];
                    array[k] = array[i - 1];
                    array[i - 1] = tmp;
                }
            }
        }
    }
}

■No66485 (Cubers さん) に返信
　コードの各所で、コンピュータに何をさせようとしているか、わかっていますか？
プログラムは、人間が思ったとおりに動くわけではありません。
書いてあるとおりに動きます。
「思ったこと」と「書いたこと」が一致しないと、期待通りに動いてはくれません。
よって、期待通りに動かないときには、
「思ったこと」と「書いたこと」が一致しているか、
確認するところからはじめます。

> private void Form1_Paint(object sender, PaintEventArgs e)
> {
>     if (myDraw)
>     {
>         Point myPoint = new Point(75, 50);
>         myCards.DrawCard(e, myCards.GetNewCard(), myPoint);
>     }
> }
これは、Paint イベントが発生すると通ります。
myCards.DrawCard メソッドの呼出に GetNewCard メソッドの戻り値を使用しています。
すると、Paint イベントが発生するとカードが進んでいきますが、問題ないですか？
Paint イベントがどの様な時に発生するか、押さえていますか？
「Refresh メソッドを呼び出したとき」だけではないですよ。


> public int GetNewCard()
> {
>     deckCardNumber++;
>     deckCardNumber %= 52;
>     return cardArray.ElementAt(deckCardNumber);
> }
> public int previousNum()
> {
>     deckCardNumber++;
>     deckCardNumber %= 52;
>     return cardArray.ElementAt(deckCardNumber-1);
> }
表示したい番号は、myCards.GetNewCard() で取得するのですよね？
では、最初に GetNewCard メソッドを実行すると、取得されるのは、何番目ですか？
また、取得したいのは、何番目ですか？
その違いは、どうして発生すると思いますか？

previous がおかしくないですか？
cardArray.ElementAt(deckCardNumber - 1) ですが、その前にインクリメントしています。
すると、今と同じものになりますよ？
また、メソッド呼出前に deckCardNumber が 51 だったら、
インクリメントして52、
剰余をとって 0、
配列の-1番目の要素にアクセスしようとしてエラー、ですよ。


> public void DrawCard(PaintEventArgs e, int cardNumber, Point p)
> {
>     // Create a Bitmap object from a file.
>     Bitmap myBitmap = new Bitmap("..//..//cards.png");
>     int xOffset = 0; // TBD - Need to handle all cards
>     int yOffset = 0;    // TBD - Need to handle all cards
>
>     if ((cardNumber <= 12) && (cardNumber >= 0))
>     {
>         xOffset = +cardNumber * width; // TBD - Need to handle all cards
>     }
>     else if ((cardNumber <= 25) && (cardNumber >= 13))

yOffset = (cardNumber / 13) * 123;

　cardNumber が int なら、int / int は int、除算の商のみになります。
cardNumber が 0〜12 の間は (cardNumber / 13) * 123 = 0 * 123 = 0 です。
cardNumber が 13〜25 の間は (cardNumber / 13) * 123 = 1 * 123 = 123 です。
cardNumber が 26〜38 の間は (cardNumber / 13) * 123 = 2 * 123 = 246 です。