初見です。プログラミング歴が２か月の超初心者です。Windowsアプリのソースコードについて質問です。

Question

「あっち向いてホイ」ゲームのソースコードをサクラエディタで書いています。
完成後は、ビジュアルスタジオコード2022に貼り付けてソリューションのリビルドの後にデバック開始をする
予定です。
もちろん、必要な画像は既にファイル内に用意してあります。
途中まで完成しているのですが最後の部分が書けなくて悩んでいます。
このゲームの全体の構図としては、コンピューター側の指の画像１枚とプレイヤー側の指の画像４枚で
スタートボタンを押すとコンピューター側の指の画像が回転し始めプレイヤーが４枚の
上向きの指の画像、下向きの指の画像、左向きの指の画像、右向きの指の画像の中より１枚を選択すると
勝敗が確定して「あなたの勝ちです！」または「あなたの負けです！」が画面に表示されるという流れです。

実現したいこと

現時点では、スタートボタンを押してコンピューター側の指の画像が回転するまでは完成していて
残りは、コンピューター側の指の画像とプレイヤー側の画像が違うなら「あなたの勝ちです！」と表示され、
コンピューター側の指の画像とプレイヤー側の画像が同じなら「あなたのまけです！」と表示させたいのです。
ただ、それだけのゲームと呼べるものではありませんが最後まで完成させたいです。
どうか、ご教示ください。

該当のソースコード

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;

namespace RockPaperScissors
{
/// <summary>
/// あっち向いてほいゲーム
/// </summary>
public partial class Form1 : Form
{
int INTERVAL_TIME = 100;

    const string CpuHand0ImageFile = "Hand0.png";
    const string CpuHand1ImageFile = "Hand1.png";
    const string CpuHand2ImageFile = "Hand2.png";
    const string CpuHand3ImageFile = "Hand3.png";

    int counter = 0;

    /// <summary>
    /// コンストラクタ
    /// </summary>
    public Form1()
    {
        InitializeComponent();
    }

    /// <summary>
    /// ロード
    /// </summary>
    /// <param name="sender"></param>
    /// <param name="e"></param>
    private void Form1_Load(object sender, EventArgs e)
    {
        // タイマーインターバル(1000で1秒)
        Timer1.Interval = INTERVAL_TIME;

        PicCPU.Image = Image.FromFile(CpuHand0ImageFile);
    }

    /// <summary>
    /// タイマーを動かす
    /// </summary>
    private void GameStart()
    {
        Timer1.Enabled = true;
        //counter = 0;
    }

    /// <summary>
    /// コンピュータの絵をまわす
    /// </summary>
    /// <param name="sender"></param>
    /// <param name="e"></param>
    private void Timer1_Tick(object sender, EventArgs e)
    {
        //cpuResult = (JankenType)rdm.Next(0, 3);

        if (counter == 0)
        {
            PicCPU.Image = Image.FromFile(CpuHand0ImageFile);
        }
        if (counter == 1)
        {
            PicCPU.Image = Image.FromFile(CpuHand1ImageFile);
        }
        if (counter == 2)
        {
            PicCPU.Image = Image.FromFile(CpuHand2ImageFile);
        }
        if (counter == 3)
        {
            PicCPU.Image = Image.FromFile(CpuHand3ImageFile);
        }
        counter++;
        if (counter > 3) counter = 0;
    }

    /// <summary>
    /// スタートボタンクリック時の処理
    /// </summary>
    /// <param name="sender"></param>
    /// <param name="e"></param>
    private void StartButton_Click(object sender, EventArgs e)
    {
        GameStart();
    }
}

}

Comments

[TN8001]

> ゲームのソースコードをサクラエディタで書いています。
> 完成後は、ビジュアルスタジオコード2022に貼り付けて

なんでサクラエディタなんでしょうか？
提示コードにはコントロール類の配置がないですが、Designer.csも手書きということですか？？

> プレイヤーが４枚の上向きの指の画像、下向きの指の画像、左向きの指の画像、右向きの指の画像の中より１枚を選択する

PictureBoxかなんかに画像があって、それをユーザーがクリック（選択）するんですよね？
クリックイベントでタイマーを止めて判定すればよさそうですが、どこに詰まっていますか？

[m.ts10806]

始めからVisual Studio Codeで作らないのはどういう意図や経緯でしょうか。
GUIではなくCUIということ？と思ったら StartButton_Click　とちゃんとボタンのクリックイベント拾うように作られてるんですね。
タイトルも「何に詰まっているか」「何をしたいか」の要件を記載した方が良いと思います。質問タグだけで済む、内容がないタイトルになってるので。

[YAmaGNZ]

C#だったらVisualStudioCodeよりVisualStudioのほうがいいのでは？

[fana]

> 最後まで完成させたいです

途中なのであれば，必要な要素を追加していけばよいのではないでしょうか．

とりあえず「プレイヤーが４つのうちから１つを選択する」ための手段を用意することが絶対に必要だと思うので，それを追加してみてはどうですか？
で，選択操作が成された際のイベントで勝敗判定をすれば良い．

例えば，選択手段が　４つのボタン　とかで良いならば，「スタートボタン」と同様な実装作業を行うだけでしょうから簡単に済みそうですよね．
どのボタンが押されたのかと，現在の counter の値とを付き合わせて判定するだけの話なのではないかと想像します．

その他，

* 「回転」を止める必要がありそうなので，然るべきタイミングで止める処理を行う
* 勝敗結果の表示手段を追加し，前記勝敗判定の結果をそこに表示する
* 各ボタンがその時々で　押せる/押せない　みたいな話が必要なら追加する

等，とにかく不足している要素を１つずつ足していけばよいのではないかと．

[ikedas]

これ、たぶん、
「『あっち向いてホイ』ゲームを作ってください」
「ソースコードをサクラエディタで書きます」
「完成後は、ビジュアルスタジオコード2022に貼り付けてソリューションのリビルドの後にデバック開始をしましょう」
「もちろん、必要な画像はあらかじめファイル内に用意しておいてくださいね」
と指示した方がいらっしゃるということではないですか。

もしもそうなら、その方に質問してください。またその方が多忙などで捕まらないのなら、同じように指示を受けて作業している方は周りにいませんか。その方でもいいので相談してみてください。

率直に言って、質問者さんに必要なのは極めて初歩的な知識なので、対面で手取り足取り教えてもらうのが一番効率的です。
逆に、ネット越しに顔の見えない相手から文章だけを介して教えてもらうのは、効率が悪すぎてほとんど無理なレベルです。

[stdio]

上5件の意見と似ていますが、色々と足りない部分が多いように感じます。
C#ということで[緑のバイク](http://midorinobike.in.coocan.jp/CS/index.html)を参考に開発を進めてみてはいかがでしょうか？

[fana]

> namespace RockPaperScissors

という記述だけから勝手に想像するに，コレよりも前に　ほとんど似たような別の物　を作った経験があるのではないかな？　とか思う．
仮にそうであれば，その時と同様に作業を進めていけばいいのではなかろうか？　という気がする．

Answer 1

IDEのコード補完無しでC#のコードを書くのは割と面倒で、サクラエディタは非効率すぎるのでやめましょう。初心者なら尚更です。
VisualStudio2022があるなら、素直にそちらで書きましょう。

ただ、それだけのゲームと呼べるものではありませんが最後まで完成させたいです。
どうか、ご教示ください。

何が問題で完成できないのか、質問を具体的にしてください。
（〇〇というエラーが出ていてビルドが出来ない、実行すると〇〇というエラーが発生する等）
現状のままだと、質問ではなく只のコード作成依頼です。希望のコード作成して欲しいだけなら、お金を払ってプログラム作成を依頼するサイトがあるので、そういった所を利用するか、チャットAIにでも頼めばよいかと思います。

Answer 2

コメントに反応がなく詳細が不明なので、普通にVisual Studioを使用しました。

画像を用意するのが面倒なため文字で代用しましたが、ロジックは画像でも使えるはずです。

完成が目標のようなので、非常に冗長ですがわかりやすさを優先しました（個人的にはTimer1_Tick内のif羅列は配列を使用して、各Clickは共通のイベントハンドラで短縮したいところです）

using System;
using System.Windows.Forms;


namespace Qb7ezl17qryp65r
{
    public partial class Form1 : Form
    {
        private int counter = 0;

        public Form1()
        {
            InitializeComponent();
        }

        private void Timer1_Tick(object sender, EventArgs e)
        {
            if (counter == 0)
            {
                // 以下TextはすべてImageに置き換え
                // 都度Image.FromFileするのは大変無駄なのと、
                // ==判定できなくなるので同一のImageを使うようにする
                //cpu.Image = up.Image;

                cpu.Text = up.Text;
            }
            if (counter == 1)
            {
                cpu.Text = right.Text;
            }
            if (counter == 2)
            {
                cpu.Text = down.Text;
            }
            if (counter == 3)
            {
                cpu.Text = left.Text;
            }
            counter++;
            if (counter > 3) counter = 0;
        }

        private void Button1_Click(object sender, EventArgs e)
        {
            timer1.Enabled = true;
        }

        private void Up_Click(object sender, EventArgs e)
        {
            if (timer1.Enabled == false) return;

            timer1.Enabled = false;

            if (cpu.Text != up.Text) MessageBox.Show("あなたの勝ちです！");
            else MessageBox.Show("あなたのまけです！");
        }

        private void Right_Click(object sender, EventArgs e)
        {
            if (timer1.Enabled == false) return;

            timer1.Enabled = false;

            if (cpu.Text != right.Text) MessageBox.Show("あなたの勝ちです！");
            else MessageBox.Show("あなたのまけです！");
        }

        private void Down_Click(object sender, EventArgs e)
        {
            if (timer1.Enabled == false) return;

            timer1.Enabled = false;

            if (cpu.Text != down.Text) MessageBox.Show("あなたの勝ちです！");
            else MessageBox.Show("あなたのまけです！");
        }

        private void Left_Click(object sender, EventArgs e)
        {
            if (timer1.Enabled == false) return;

            timer1.Enabled = false;

            if (cpu.Text != left.Text) MessageBox.Show("あなたの勝ちです！");
            else MessageBox.Show("あなたのまけです！");
        }
    }
}

namespace Qb7ezl17qryp65r
{
    partial class Form1
    {
        /// <summary>
        /// 必要なデザイナー変数です。
        /// </summary>
        private System.ComponentModel.IContainer components = null;

        /// <summary>
        /// 使用中のリソースをすべてクリーンアップします。
        /// </summary>
        /// <param name="disposing">マネージド リソースを破棄する場合は true を指定し、その他の場合は false を指定します。</param>
        protected override void Dispose(bool disposing)
        {
            if (disposing && (components != null))
            {
                components.Dispose();
            }
            base.Dispose(disposing);
        }

        #region Windows フォーム デザイナーで生成されたコード

        /// <summary>
        /// デザイナー サポートに必要なメソッドです。このメソッドの内容を
        /// コード エディターで変更しないでください。
        /// </summary>
        private void InitializeComponent()
        {
            this.components = new System.ComponentModel.Container();
            this.up = new System.Windows.Forms.Label();
            this.right = new System.Windows.Forms.Label();
            this.down = new System.Windows.Forms.Label();
            this.left = new System.Windows.Forms.Label();
            this.cpu = new System.Windows.Forms.Label();
            this.button1 = new System.Windows.Forms.Button();
            this.timer1 = new System.Windows.Forms.Timer(this.components);
            this.SuspendLayout();
            // 
            // up
            // 
            this.up.AutoSize = true;
            this.up.Font = new System.Drawing.Font("MS UI Gothic", 60F, System.Drawing.FontStyle.Regular, System.Drawing.GraphicsUnit.Point, ((byte)(128)));
            this.up.Location = new System.Drawing.Point(256, 36);
            this.up.Name = "up";
            this.up.Size = new System.Drawing.Size(116, 80);
            this.up.TabIndex = 0;
            this.up.Text = "👆";
            this.up.Click += new System.EventHandler(this.Up_Click);
            // 
            // right
            // 
            this.right.AutoSize = true;
            this.right.Font = new System.Drawing.Font("MS UI Gothic", 60F, System.Drawing.FontStyle.Regular, System.Drawing.GraphicsUnit.Point, ((byte)(128)));
            this.right.Location = new System.Drawing.Point(314, 116);
            this.right.Name = "right";
            this.right.Size = new System.Drawing.Size(116, 80);
            this.right.TabIndex = 1;
            this.right.Text = "👉";
            this.right.Click += new System.EventHandler(this.Right_Click);
            // 
            // down
            // 
            this.down.AutoSize = true;
            this.down.Font = new System.Drawing.Font("MS UI Gothic", 60F, System.Drawing.FontStyle.Regular, System.Drawing.GraphicsUnit.Point, ((byte)(128)));
            this.down.Location = new System.Drawing.Point(256, 196);
            this.down.Name = "down";
            this.down.Size = new System.Drawing.Size(116, 80);
            this.down.TabIndex = 2;
            this.down.Text = "👇";
            this.down.Click += new System.EventHandler(this.Down_Click);
            // 
            // left
            // 
            this.left.AutoSize = true;
            this.left.Font = new System.Drawing.Font("MS UI Gothic", 60F, System.Drawing.FontStyle.Regular, System.Drawing.GraphicsUnit.Point, ((byte)(128)));
            this.left.Location = new System.Drawing.Point(192, 116);
            this.left.Name = "left";
            this.left.Size = new System.Drawing.Size(116, 80);
            this.left.TabIndex = 3;
            this.left.Text = "👈";
            this.left.Click += new System.EventHandler(this.Left_Click);
            // 
            // cpu
            // 
            this.cpu.AutoSize = true;
            this.cpu.Font = new System.Drawing.Font("MS UI Gothic", 60F, System.Drawing.FontStyle.Regular, System.Drawing.GraphicsUnit.Point, ((byte)(128)));
            this.cpu.Location = new System.Drawing.Point(38, 116);
            this.cpu.Name = "cpu";
            this.cpu.Size = new System.Drawing.Size(116, 80);
            this.cpu.TabIndex = 4;
            this.cpu.Text = "👆";
            // 
            // button1
            // 
            this.button1.Location = new System.Drawing.Point(167, 320);
            this.button1.Name = "button1";
            this.button1.Size = new System.Drawing.Size(75, 23);
            this.button1.TabIndex = 5;
            this.button1.Text = "スタート";
            this.button1.UseVisualStyleBackColor = true;
            this.button1.Click += new System.EventHandler(this.Button1_Click);
            // 
            // timer1
            // 
            this.timer1.Tick += new System.EventHandler(this.Timer1_Tick);
            // 
            // Form1
            // 
            this.AutoScaleDimensions = new System.Drawing.SizeF(6F, 12F);
            this.AutoScaleMode = System.Windows.Forms.AutoScaleMode.Font;
            this.ClientSize = new System.Drawing.Size(438, 376);
            this.Controls.Add(this.button1);
            this.Controls.Add(this.cpu);
            this.Controls.Add(this.left);
            this.Controls.Add(this.down);
            this.Controls.Add(this.right);
            this.Controls.Add(this.up);
            this.Name = "Form1";
            this.Text = "Form1";
            this.ResumeLayout(false);
            this.PerformLayout();

        }

        #endregion

        private System.Windows.Forms.Label up;
        private System.Windows.Forms.Label right;
        private System.Windows.Forms.Label down;
        private System.Windows.Forms.Label left;
        private System.Windows.Forms.Label cpu;
        private System.Windows.Forms.Button button1;
        private System.Windows.Forms.Timer timer1;
    }
}