Где ошибка в коде C#?

Question

[Wirsak]

Пишу Крестики-Нолики. При вводе координат появляется ошибка "Индекс находился вне границ массива", которая указывает на строку(покажу в коде).

using System;

namespace MyGame
{
	class Field
	{
		//Двумерный массив для поля.
		int[,] array = new int[3, 3];

		//Заполнение массива единицами, единица обозначает, что клетка пуста.
		private void fill_array()
		{
			for (int i = 0; i < 3; i++)
				for (int j = 0; j < 3; j++)
					array[i, j] = 1;
		}

		//Вывод поля на экран
		private void draw_array()
		{
			for (int i = 0; i < 3; i++)
			{
				for (int j = 0; j < 3; j++)
					Console.Write(array[i,j] + " ");
				Console.WriteLine();
			}
		}

		public Field()
		{
			fill_array();
			draw_array();
		}

		//Ввод нуля и последующая проверка на выигрыш.
		public void Enter0()
		{
			int x, y;

			x = Console.Read();
			y = Console.Read();
			array[x, y] = 0;
			draw_array(); <== Указывает на этот метод
			if (Check(x, y))
				Player.Win();
		}

		//Проверка на выигрыш. Проверяет содедние клетки по горизонтали и вертикали. 
		//Если есть три нуля - возвращает истину.
		private bool Check(int x, int y)
		{
			int count = 0;

			for (int i = 0; i < 3; i++)
			{
				if (array[x, i] == 0)
					count++;
				else
					break;
			}

			if (count == 3)
				return true;
			else
			{
				count = 0;
				for (int i = 0; i < 3; i++)
				{
					if (array[i, y] == 0)
						count++;
					else
						break;
				}

				if (count == 3)
					return true;
				else
					return false;
			}
		}
	}
}

using System;

namespace MyGame
{
	static class Player
	{
		public static void Win()
		{
			Console.Clear();
			Console.WriteLine("You win!");
		}
	}
}


using MyGame;
using System;

class Game
{
	static void Main()
	{
		Field field = new Field();

		while (true)
		{
			field.Enter0();
		}
	}
}

Answer 1

[Роман]

Зачастую дебаггер показывает на следующую строку (инструкцию) после проблемной, как в этом случае.
У вас, чтобы всё работало, x и y должны быть равны числу 0, 1 или 2. А Console.Read читает символ, а не число. Т.е. введя 0, вы получите символ '0', что соответствует числу 48.
Вам нужно обработать ввод, отсечь не правильные символы и преобразовать символы в числа.

Comments

[Wirsak]

https://docs.microsoft.com/ru-ru/dotnet/api/system...
Тут написано, что Read возвращает int.

[Роман]

Wirsak, int используется вместо char потому что Read вернёт -1, когда не осталось данных для чтения.

[freeExec]

Wirsak, А проверять, что на самом деле возвращается вы не стали :)

[Wirsak]

freeExec, ну на самом деле там возвращается код символа, а не сам символ. Этого я не знал.

Answer 2

[Юрий Есин]

const int FieldWidth = 3;
    const int FieldHeight = 3;

    int[,] array = new int[FieldWidth, FieldHeight];

    //Ввод нуля и последующая проверка на выигрыш.
    public void Enter0()
    {
      int x, y;

      while (!int.TryParse(Console.Read(), out x) || x < 0 || x >= FieldWidth) continue; // Здесь можно обработать неверный ввод.
      while (!int.TryParse(Console.Read(), out y) || y < 0 || y >= FieldHeight) continue; // Здесь можно обработать неверный ввод.

      array[x, y] = 0;
      draw_array();
      if (Check(x, y))
        Player.Win();
    }

И везде в коде тройки замените на константы.

Comments

[Wirsak]

Спасибо, о константах не подумал.