ゲームの停止ができない

Question

前提・実現したいこと

ゲームをクリアした際にゲームを停止させたいです。

発生している問題・エラーメッセージ

エラーメッセージはありませんでした。

該当のソースコード

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using System.IO;

namespace blockbreking_v2
{
    public partial class Form1 : Form
    {
        Racket racket;
        Ball ball;
        static int numOfBlocks = 20;
        Block[] blocks = new Block[numOfBlocks];
        public static int score;
        
        Random random = new Random();
        bool rightKey = false, leftKey = false;

        private void Initialize()
        {
            racket = new Racket(new Rectangle(pictureBox1.Width / 2 - 25, pictureBox1.Height - 10, 50, 10), 0, 0, 0, 10);
            ball = new Ball(new Rectangle(0, 0, 10, 10), 0, 255, 0, new Point(7, 8));

            int drawX = 10, drawY = 40;
            for (int i = 0; i < blocks.Length; i++)
            {
                blocks[i] = new Block(new Rectangle(drawX, drawY, 25, 10), random.Next(0, 255), random.Next(0, 255), random.Next(0, 255));
                drawX += 100;
                if (drawX >= pictureBox1.Width - 25)
                {
                    drawX = 10;
                    drawY += 30;
                }
            }
            score = 0;

        }
        public Form1()
        {
            InitializeComponent();
            Initialize();
        }

        private void timer1_Tick(object sender, EventArgs e)
        {
            pictureBox1.Invalidate();
        }

        private void Form1_KeyDown(object sender, KeyEventArgs e)
        {
            if (e.KeyCode == Keys.Right)
                rightKey = true;
            if (e.KeyCode == Keys.Left)
                leftKey = true;
        }

        private void Form1_KeyUp(object sender, KeyEventArgs e)
        {
            if (e.KeyCode == Keys.Right)
                rightKey = false;
            if (e.KeyCode == Keys.Left)
                leftKey = false;
        }

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

        private void saveSToolStripMenuItem_Click(object sender, EventArgs e)
        {
            if (saveFileDialog1.ShowDialog() == DialogResult.OK)
            {
                Encoding enc = Encoding.GetEncoding("Shift_JIS");
                using (StreamWriter sw = new StreamWriter(saveFileDialog1.FileName, false, enc))
                {
                    sw.Write(score);
                }
            }
        }

        private void button1_Click(object sender, EventArgs e)
        {
            button1.Visible = false;
            this.ActiveControl = null;
            Initialize();
            timer1.Enabled = true;
        }

        private void pictureBox1_Paint(object sender, PaintEventArgs e)
        {
            if (rightKey)
                racket.MoveRight(pictureBox1.Width);
            if (leftKey)
                racket.MoveLeft();
            racket.DrawImage(e.Graphics);
            ball.CalcSpeed(new Point(pictureBox1.Width, pictureBox1.Height), racket.GetPosition());
            ball.Move();
            ball.DrawImage(e.Graphics);
            foreach (Block b in blocks)
            {
                if (!b.Hit(ball.GetPosition()))
                    b.DrawImage(e.Graphics);
                else
                {
                    score += 10;
                    toolStripStatusLabel1.Text = score.ToString();
                    if (score == numOfBlocks*10)
                    {
                        timer1.Enabled = false;
                        float x = 140.0F;
                        float y = 70.0F;
                        float width = 200.0F;
                        float height = 50.0F;
                        RectangleF drawRect = new RectangleF(x, y, width, height);
                        Font drawFont = new Font("Arial", 30);
                        e.Graphics.FillRectangle(Brushes.Gold, x, y, width, height);   
                        e.Graphics.DrawString("    Clear", drawFont, Brushes.DeepSkyBlue, drawRect);

                    }
                }
            }

            
            if (!ball.JudgeFail(new Point(pictureBox1.Width, pictureBox1.Height)))
            {
                timer1.Enabled = false;
                float x = 140.0F;
                float y = 70.0F;
                float width = 200.0F;
                float height = 50.0F;
                RectangleF drawRect = new RectangleF(x, y, width, height);
                Font drawFont = new Font("Arial", 30);
                e.Graphics.FillRectangle(Brushes.Black, x, y, width, height);
                e.Graphics.DrawString("     Fail!", drawFont, Brushes.Red, drawRect);
            }
            
        }
    
    }
    class ParentClass
    {
        protected Rectangle position;
        protected SolidBrush brush;
        protected bool visible = true;

        public ParentClass(Rectangle position, int r, int g, int b)
        {
            this.position = position;
            brush = new SolidBrush(Color.FromArgb(r, g, b));
        }
        public virtual bool Hit(Rectangle obstaclePosition)
        {
            if (position.IntersectsWith(obstaclePosition) && visible)
            {
                return true;
            }
            else

                return false;

        }
        public Rectangle GetPosition()
        {
            return position;
        }
        public void SetPosition(Rectangle position)
        {
            this.position = position;
        }
        public bool GetVisible()
        {
            return visible;
        }
        public virtual void DrawImage(Graphics g)
        {
            if (visible)
                g.FillRectangle(brush, position);
        }
    }
    class Block : ParentClass
    {
        public Block(Rectangle position, int r, int b, int g) : base(position, r, g, b)
        {
        }
        public override bool Hit(Rectangle obstaclePosition)
        {
            if (position.IntersectsWith(obstaclePosition) && visible)
            {
                visible = false;//ブロックを見えなくする
                return true;
            }
            else
            {
                return false;
            }
        }
    }

    class Racket : ParentClass
    {
        int speed;//ラケットのスピード
        public Racket(Rectangle position, int r, int g, int b, int speed) : base(position, r, g, b)
        {
            this.speed = speed*3;
        }
        public void MoveRight(int width)
        {
            if (position.X + speed <= width - position.Width)
                position.X += speed;
            else
                position.X = width - position.Width;
        }
        public void MoveLeft()
        {
            if (position.X - speed >= 0)
                position.X -= speed;
            else
                position.X = 0;
        }
    }

    class Ball : ParentClass
    {
        bool preHit;
        Point speed;
        
        public Ball(Rectangle position, int r, int g, int b, Point speed) : base(position, r, g, b)
        {
            this.position = position;
            this.speed = speed;
            preHit = false;
        }
        public void CalcSpeed(Point pictureBocSize, Rectangle obstaclePosition)
        {
            if (position.IntersectsWith(obstaclePosition) && !preHit)
            {
                double degree = Math.Atan2(position.Y + position.Height / 2 - speed.Y - (obstaclePosition.Y + obstaclePosition.Height / 2), position.X + position.Width / 2 - speed.X - (obstaclePosition.X + obstaclePosition.Width / 2));
                double racketDegree = Math.Atan2(obstaclePosition.Height, obstaclePosition.Width);
                if (degree >= racketDegree && degree <= racketDegree)
                    speed.X = -speed.X;
                else
                    speed.Y = -speed.Y;
                preHit = true;
            }
            else
                preHit = false;
            if (position.Y + speed.Y < 0)
                speed.Y = -speed.Y;
            if (position.X + speed.X > pictureBocSize.X - position.Width)
                speed.X = -speed.X;
            if (position.X + speed.X < 0)
                speed.X = -speed.X;
        }
        public void Move()
        {
            int score = Form1.score;
            if (score >= 0&& score <= 60)
            {
                position.X += speed.X ;
                position.Y += speed.Y ;
            }
            else if(score >= 70 && score <= 130)
            {
                position.X += speed.X * 1;
                position.Y += speed.Y * 1;
            }
            else
            {
                position.X += speed.X * 1;
                position.Y += speed.Y * 1;
            }
            
        }
        public bool JudgeFail(Point pictureBoxSize)
        {
            if (position.Y > pictureBoxSize.Y)
                return false;
            else
                return true;
        }
        public override void DrawImage(Graphics g)
        {
            g.FillEllipse(brush, position);
        }
    }

}

試したこと

timer.Enable=falseで停止させているつもりです。Failの場合もtimer.Enable=falseで停止させて、Failの場合はきちんとゲームが停止します。
Clearの場合は一瞬Clearと描画され、すぐにボールが動き出してしまいます。
ブロックは描画されません。

補足情報（FW/ツールのバージョンなど）

visualstudio 2019 consoleapplication

Answer 1

• timer1_Tick()の役目　＝　ゲームの進行（状態の更新）
• pictureBox1_Paint()の役目　＝　ゲームの現状態の描画

とする．
すなわち，現コードでpictureBox1_Paint()内で行っていることを，

1. ゲームの処理ロジック
2. ゲームの現状態の描画

の２つにわけ，前者をtimer1_Tick()内に移動させる．

Answer 2

タイマー以外でもpictureBox1の描画が発生すると動くコードになっていますよ。
あまり手を入れたくなければ、pictureBox1_Paint()の先頭で

if (!timer1.Enabled)
{
    return;
}

とでもしておけばよいかと。
DWM無効の環境でゲーム停止時に他ウインドウで隠れた部分は再描画されなくなるでしょうけれど…

Comments

[likuson]

ありがとうございます。再描画はされなくなったのですが、クリア及びフェル(fail)の画面が一瞬で消えてしまいます。クリア及びフェルの画面の表示を永久に続行、又は、一定時間クリア及びフェルの画面の表示をする方法はあるのでしょうか。

[SHOMI]

再描画がかかって、その際にはなにも描画していないために消えているのでしょう。
returnにブレークポイントを貼るとfailやclearの後に止まり、ステップ実行してpictureBox1_Paint()を抜けるとpictureBox1の内容が消えていませんか？
pictureBox1_Paint()の先頭に入れたコードでreturnの前にfailやclearならその描画を行うようコードを入れてください。

[SHOMI]

「あまり手を入れたくなければ」と前置きしたとおり、簡易対策です。
きちんと対応するなら、fanaさんの回答の通り描画とゲーム進行に分離してください。

[fana]

せっかくの(?) "簡易" なのに，
「止めている時でも，これだけはやりたい」とか言い始めると，結局，現存する処理を if を境にして上下に２つに選り分ける：

timer1の状況によらずやる処理; //(1)上側
if (!timer1.Enabled)return; //境界
timer1がEnabledなときだけやる処理; //(2)下側

という形になるので，別の場所に処理を分けるのと比較して，行う作業の規模は変わらなくなってしまう気がしますね．

[SHOMI]

あまり手を入れずにメッセージを出し続けたいなら、PictureBoxに直接書かずにラベルをPictureBoxの上に表示するとかですかね。

[SHOMI]

後々手を入れることを考えると、きちんと分離しておいたほうがよいですね。
それに今のままだと他アプリなどの影響で再描画が掛かってしまうと進行スピードが不安定になりますし。
そういえば昔ゲーム画面上でマウスポインタを動かしまくると進行速度が上がるゲームが有ったな…

[fana]

The word "DWM" is helpful for me, so I press the "+" button.