C#コンソールアプリでタイピング練習アプリの改良

Question

前提・実現したいこと

C# コンソールアプリでタイピング練習アプリを作っています。

ほぼ完成していますが、改良のヒントを得たくて投稿しました。

ゲームの流れは下記の通り
・英単語が横位置(範囲80文字)ランダムで、一定時間毎に1行ずつ落ちていく。
・キー入力を始めると、正しい文字であれば、一文字ずつ'*'(アスタリスク)に変わる。
・全て正しく入力されれば、スコアカウントされ、次の英単語に移る。
・入力を間違えるか、再下段まで行くと、ミスカウントされ、次の英単語に移る。

キー入力始めの時に英単語が落ちなくなる(ReadKeyによるブロック)のを防ぎたいです。
(英単語が常に一定時間で落ちながら、正しい文字を入力したらそのタイミングで、'*'に変わっていくようにしたい)

english → 下の行に落ちるまでに、正しく3文字入力出来たら
***lish → となって落ちていく

該当のソースコード

using System;

namespace typ
{
	class Program
	{
		static int level = 0;
		static int score = 0;
		static int miss = 0;

		static int typing()
		{
			string[] word = new string[] { "english", "japanese", "french", "spanish", "hoge", "fuga" };

			Console.Clear();

			Console.ForegroundColor = ConsoleColor.Yellow;
			Console.Write("  Typing of Fall Down Text    ");
			Console.ForegroundColor = ConsoleColor.White;
			Console.WriteLine("Score {0}  Miss {1}", score.ToString("000"), miss.ToString("000"));
			Console.ForegroundColor = ConsoleColor.Gray;
			Console.WriteLine(new string('-', 80));
			Console.SetCursorPosition(0, 22);
			Console.WriteLine(new string('-', 80));

			int s = Environment.TickCount;

			Random rnd = new Random(s++);

			int r = rnd.Next(60);
			int c = rnd.Next(word.Length);

			for (int l = 2; l < 22; l++)
			{
				int wait = word[c].Length + (1050 - level * 180);

				if (Console.KeyAvailable)
				{
					for (int i = 0; i < word[c].Length; i++)
					{
						if (word[c].Substring(i, 1) != "*")
						{
							if (Console.ReadKey(true).Key.ToString() == word[c].Substring(i, 1).ToUpper())
							{
								word[c] = word[c].Substring(0, i) + "*" + word[c].Substring(i + 1, word[c].Length - (i + 1));

								if (l > 2)
								{
									Console.SetCursorPosition(0, l - 1);
									Console.Write(new string(' ', 80));
								}

								Console.SetCursorPosition(r, l);
								Console.ForegroundColor = ConsoleColor.Green;
								Console.Write(word[c]);
								Console.ForegroundColor = ConsoleColor.Gray;

								continue;
							}
							else if (Console.ReadKey(true).Key == ConsoleKey.Escape)
							{
								Console.Clear();

								return 1;
							}
							else
							{
								Console.SetCursorPosition(44, 0);
								Console.ForegroundColor = ConsoleColor.White;
								Console.WriteLine((++miss).ToString("000"));
								Console.ForegroundColor = ConsoleColor.Gray;

								return 0;
							}
						}
					}
				}
				else
				{
					if (l > 2)
					{
						Console.SetCursorPosition(0, l - 1);
						Console.Write(new string(' ', 80));
					}

					Console.SetCursorPosition(r, l);
					Console.ForegroundColor = ConsoleColor.Green;
					Console.Write(word[c]);
					Console.ForegroundColor = ConsoleColor.Gray;

					if (word[c] == new string('*', word[c].Length))
					{
						Console.SetCursorPosition(36, 0);
						Console.ForegroundColor = ConsoleColor.White;
						Console.WriteLine((++score).ToString("000"));
						Console.ForegroundColor = ConsoleColor.Gray;

						break;
					}

					System.Threading.Thread.Sleep(wait);
				}
			}

			Console.Clear();

			return 0;
		}

		static void Main(string[] args)
		{
			Console.Write("Level(1 - 5) = ");

			string lv = Console.ReadLine();

			level = int.Parse(lv);

			if (level < 1) level = 1;
			else if (level > 5) level = 5;

			while (true)
			{
				if (typing() == 1) break;
			}
		}
	}
}

補足情報

Microsoft Visual Studio Community 2019
Version 16.5.0
.NET Framework version 4.8.03761

Comments

[Zuishin]

Console.KeyAvailable が true の場合、キーバッファにキー入力がたまっているので、Console.ReadKey はすぐに処理を戻します。その結果、キーバッファが空になれば KeyAvailable は false を返します。これが true の時だけキーを読めばいいと思います。

[yulico]

ありがとうございます。
KeyAvailable が true の時だけキーを読むようにしてみます。
ありがとうございました。

Answer 1

Console.KeyAvailableプロパティを利用して、キー入力があった時のみReadKeyしてはどうでしょうか

よくよくソースを見たら、
正しいキーを押した場合はReadKeyを１回しか実行しないので問題はないのですが、間違ったキーを押した場合、elseifにてもう一回ReadKeyが呼ばれ、そこでブロックしているのではないですか？

KeyAvailableプロパティにてキー入力があったと判断した後、ReadKeyを一旦変数で受けて、その変数を利用して、その後の判断を行うようにすればよろしいかと思います。

Comments

[yulico]

早い回答、ありがとうございます。

>Console.KeyAvailableプロパティを利用して、キー入力があった時のみReadKeyしてはどうでしょうか
やっているつもりなのですが、場所が悪いのでしょうか・・・
見直してみます。

[YAmaGNZ]

KeyAvailableにて入力があったと判断した後にReadKeyを複数回呼ぶような形になっています。
ですので、キー入力があったと判断した後、ReadKeyの結果を一旦変数に受け取り、その変数の値で比較処理を行うべきです。

[yulico]

ああ、確かにループさせてしまってますね(^^;;
変数処理に直してみます。
ありがとうございました。