1

これが取引です。ローカルリソース内の画像ととの画像を比較し、配列内で同じである場合は、勝ったユーザーにメッセージを送信しようとしXOいますwinCondition

私が見つけた問題はb1.Image、私に適切な比較を与えていないということです。これを比較しています:

+  b1.Image  {System.Drawing.Bitmap}  System.Drawing.Image  {System.Drawing.Bitmap}

代わりに画像名を比較したい場合。

これらの問題の1つは、turnNumber = 5勝ったときは勝った、勝てなかったときは勝ったということです。これはによるものだと思いますb1.Image problem。別の四角をクリックすると、もう一度勝ちをチェックします。

ゲームが終了したらボタンを無効にしたいのですが、どうすればいいのかわかりません。

それは次のように簡単でしょうか:

   foreach (Button btnEnabled in buttonArray)
   {
       btnEnabled.Enabled = false;
   }

ここにコードがあります、そしてまた、助けてくれてありがとう。私はこれに数日間苦労しています。

namespace BGreinAssignment2
{
    public partial class frmTicTacToe : Form
    {
        //Global variables
        private bool player1Turn = false;
        private bool player2Turn = true;

        private int[,] winCondition = 
        {
            {0,1,2}, //Horizontal top
            {3,4,5}, //Horizontal Middle
            {6,7,8}, //Horizontal Bottom
            {0,3,6}, //Vertical Left
            {1,4,7}, //Vertical Middle
            {2,5,8}, //Vertical Right
            {0,4,8}, //Diagonal Top Left to Bottom Right or Vice-Versa
            {2,4,6}  //Diagonal Top Right to Bottom Left or Vice-Versa
        };
        private Button[] buttonArray;
        private int turnNumber = 0;

        public frmTicTacToe()
        {
            InitializeComponent();
        }

        //Creates the button array for checks and sets X to go first.
        private void frmTicTacToe_Load(object sender, EventArgs e)
        {
            //Creates button array for checking if image is there/check for beginning of game
            buttonArray = new Button[9] {btnTopLeft, btnTopMid, btnTopRight, btnMidLeft, btnMid, btnMidRight, btnBotLeft, btnBotMid, btnBotRight};

            //Sets player 1 to go first to satisfy the "X always goes first"
            player1Turn = true;
            player2Turn = false;
        }

        /// <summary>
        /// Checks the buttons if the images don't create a win condition through the winCheck method,
        /// displays message box to user with "Draw!" break is included so it doesn't say "Draw!" for each button it checks.
        /// </summary>
        private void drawCheck()
        {
            foreach (Button checkDraw in buttonArray)
            {
                if (checkDraw.Image != null)
                {
                    MessageBox.Show("Draw!");
                    break;
                }
            }
        }

        /// <summary>
        /// Checks the win condition to see if the images are the same. If they are, it will show a message box with the winner.
        /// </summary>
        /// <param name="btnChecks">Creates an array to check the button images</param>
        /// <returns>If there is a winner, returns true and shows message box</returns>
        private bool winCheck(Button[] btnChecks)
        {
            bool win = false;
            for (int i = 0; i < 8; i++)
            {
                int a = winCondition[i, 0], b = winCondition[i, 1], c = winCondition[i, 2];

                Button b1 = btnChecks[a], b2 = btnChecks[b], b3 = btnChecks[c];

                if (b1.Image == null || b2.Image == null || b3.Image == null)
                {
                    continue;
                }

                if (b1.Image == b2.Image && b2.Image == b3.Image)
                {
                    win = true;
                    MessageBox.Show("Game over. " + b1.Image + " Wins!");
                }
            }
            return win;
        }

        //If player chooses top left square
        private void btnTopLeft_Click(object sender, EventArgs e)
        {
            if (btnTopLeft.Image == null)
            {
                if (player1Turn == true)
                {
                    if (turnNumber == 0)
                    {
                        btnTopLeft.Image = BGreinAssignment2.Properties.Resources.tic_tac_toe_X;
                        player1Turn = false;
                        player2Turn = true;
                        turnNumber++;
                    }
                    else
                    {
                        btnTopLeft.Image = BGreinAssignment2.Properties.Resources.tic_tac_toe_X;
                        player1Turn = false;
                        player2Turn = true;
                        turnNumber++;
                        if (turnNumber >= 5)
                        {
                            winCheck(buttonArray);
                        }
                        if (turnNumber == 9)
                        {
                            drawCheck();
                        }
                    }
                }
                else
                {
                    btnTopLeft.Image = BGreinAssignment2.Properties.Resources.tic_tac_toe_O;
                    player1Turn = true;
                    player2Turn = false;
                    turnNumber++;
                    if (turnNumber <= 5)
                    {
                        winCheck(buttonArray);
                    }
                    if (turnNumber == 9)
                    {
                        drawCheck();
                    }
                }
            }
            else
            {
                MessageBox.Show("This space has already been selected");
            }
        }

        //Excluded rest of code (just button clicks, repeats of same for 9 squares)

        /// <summary>
        /// Resets the game for the player.
        /// </summary>
        /// <param name="sender"></param>
        /// <param name="e"></param>
        private void btnPlayAgain_Click(object sender, EventArgs e)
        {
            //For each button, set image to null then reset turn counter and set turn to player 1.
            foreach (Button btnSpaces in buttonArray)
            {
                btnSpaces.Image = null;
            }
            turnNumber = 0;
            player1Turn = true;
            player2Turn = false;
        }
    }
}
4

1 に答える 1

1

Imageあなたが発見したように、 sを直接比較することは答えではありません。これはテストしていませんが、同じリソースから読み込まれているにもかかわらず、各画像のプロパティが大きく異なる可能性があるためだと思います。たとえば、1つのボタンがわずか20ピクセルの正方形で、別のボタンが50の場合、HeightとのWidthプロパティは異なります。その他のプロパティについては、ドキュメントを参照してください。

@ John3136が言ったように、これを処理するためのより良い方法は、モデルを図面から分割することです。Tagこれは、各ボタンのプロパティを使用して「O」または「X」のいずれかを格納するのと同じくらい簡単です。比較することを除いて、今と同じようにすべてをチェックしますb1.Tag == b2.Tag && b2.Tag == b3.Tag。より良い、しかしより複雑な方法は、ゲームボードを表す何かの配列を持つことです。他のプログラミングの問題と同様に、それを行うための多くの可能な方法があります-列挙型を作成してその配列、または上記のいずれかの配列の配列を格納する、char[9]最も簡単ですが、機能する可能性があります。(またstring[]bool?[]List<>配列の代わりに、最終的に)。いずれの場合も、画像を更新すると同時にモデルを更新する必要がありますが、勝利をテストするためにはるかに簡単な比較を行います。


ボタンを無効にすることについては完全に正しいですが、ここに質問があります。他のプレイヤーがすでにクリックしたボタンを誰かがクリックするとどうなりますか?もちろん、ユーザーに警告することはできますが、それでも有効な選択のように見えます。クリックしたらすぐに無効にしてみませんか?


また、もう少し遠くに行って、「//Excluded rest of code (just button clicks, repeats of same for 9 squares)」行のヒントを示します。同じコードを2回以上繰り返す場合(および一部の人は2回以上言う場合)、引数として区別するものをすべて受け取る関数を作成することをお勧めします。たとえば、

    //If player chooses top left square
    private void btnTopLeft_Click(object sender, EventArgs e)
    {
        DoClick(btnTopLeft);
    }

    //If player chooses top center square
    private void ptnTopCenter_CLick(object sender, EventArgs e)
    {
        DoClick(btnTopCenter);
    }

    private void DoClick(Button button)
    {
      // Manipulate button appropriately
    }

これは、あなたがまだ行う方法を知らないことを何もしていません-あなたはすでに他の関数を呼び出しており、引数でボタンを渡します(を参照winCheck())。これは、作成する必要のあるコードの量を減らし、特定のバグを修正するために必要な場所を減らすための単なる考え方です。

于 2012-10-04T02:23:46.693 に答える