Шахматная доска. Как убрать ошибки в коде?

@Lidasag · Регистрация: 25.12.2017

Добрый день! Задача: Дана шахматная доска размером nn клеток. Верхняя левая клетка доски черная и имеет номер (1, 1). 1.для заданного значения n определить количество черных ячеек шахматной доски; 2.по номеру ячейки (k, m) определить ее цвет; 3.определить, являются ли ячейки с номерами (k1, m1) и (k2, m2) одного цвета; 4.определить, находится ли фигура, стоящая в ячейке с номером (k1, m1), под ударом второй фигуры, стоящей в ячейке с номером (k2, m2), при условии, что ход второй фигуры и ей является: a) пешка; b) слон; c) ладья; d) ферзь; e) конь.

C#

Console.Write("Размерность доски: ");
            int n = int.Parse(Console.ReadLine());
            if (n % 2 == 0) Console.WriteLine("Черных: {0}", n / 2 * n);
            else Console.WriteLine("Черных: {0}", (n - 1) / 2 * (n-1) + n);
yach:       Console.Write("k = ");
            int k = int.Parse(Console.ReadLine());
            Console.Write("m = ");
            int m = int.Parse(Console.ReadLine());
            switch(colour(k,m,n)){
                case 0: Console.WriteLine("Ячейка {0}x{1} - черная", k, m); break;
                case 1: Console.WriteLine("Ячейка {0}x{1} - белая", k, m); break;
                case 2: Console.WriteLine("Такой ячейки нет, попробуйте ввести другую");
                    goto yach;
            }
            Console.WriteLine("k1 = ");
            int k1 = int.Parse(Console.ReadLine());
            Console.WriteLine("m1 = ");
            int m1 = int.Parse(Console.ReadLine());
yach2:      if !(k1 <= n && m1 <= n)
{
                Console.WriteLine("Такой ячейки нет, попробуйте ввести другую");
                goto yach2;
            }
            if (colour(k, m, n) == colour(k1, m1, n)) Console.WriteLine("Ячейки {0}x{1} и {2}x{3} одинаковые");
            else Console.WriteLine("Ячейки {0}x{1} и {2}x{3} разные");
        }
        static int colour(int k, int m, int n)
        {
            if (k <= n && m <= n)
            {
                if ((k + m) % 2 == 0) return 0;
                else return 1;
            }
            else return 2;
        }
    }
}

Как убрать ошибки в коде?

@Элд Хасп · 28.09.2018, 13:25

Сообщение от Lidasag

Как убрать ошибки в коде?

Ошибки какие? Компиляции, исполнения, алгоритма? Конкретнее напишите какие ошибки, где и в каких случаях возникают.

@Lidasag · 28.09.2018, 13:30 **[ТС]**

Все выводится , но консоль сразу закрывается. Пробовала уже и врайтлай и кей, ридлайн, ридкей...

@PoPsa85 · 28.09.2018, 13:45

Lidasag, Это что у вас ?

C#

1	if !(k1 <= n && m1 <= n)

@V_Monomax · 28.09.2018, 14:06

мне больше всего интересно что делает вот этот код

Сообщение от Lidasag

yach:

и вот этот

Сообщение от Lidasag

yach2:

это новая функция в C# или просто дичь от автора кода?

@PoPsa85 · 28.09.2018, 14:10

V_Monomax, Это далеко не новая функция https://msdn.microsoft.com/ru-... (v=vs.120)

@V_Monomax · 28.09.2018, 14:22

я конечно понимаю что ":" - это наследование, либо в литералах его можно встретить, но такое его применение здесь и не оправдано, да и код засоряет, так что это здесь лишнее.

Добавлено через 8 минут
Это я к чему, использование иди туда, ну вообще не оправдано, только код засоряет, это конечно привет бейсикам, и до свидания ООП. Зачем вам нужен C# с его подходом ООП, если вы его не используете? Зачем вам нужен тогда C#, если из бейсика вырасти не можете? Конечно применять все средства заложенные в языке хорошо, но только в тех случаях когда это оправдано. А при вашем подходе, очень похоже, существует проблема зацикливания (вываливаетесь по переполнению стека, либо подобная ей проблема). Так что ваш код надо рефакторить, что-бы в первую очередь вы могли перечитывать его). Не говорю уж о других (хотя другим вы будете тогда когда через более чем месяц будете сидеть и читать свой же код, и не понимать что к чему).

@Lidasag · 28.09.2018, 14:47 **[ТС]**

Код не мой. Мне без разницы что там написано, мне надо чтоб окно не уходило, после окончания программы

@Элд Хасп · 28.09.2018, 14:58

Сообщение от Lidasag

Все выводится , но консоль сразу закрывается. Пробовала уже и врайтлай и кей, ридлайн, ридкей...

То есть сам участок с этим кодом работает? Нет только ожидания перед закрытием окна консоли. Но в этом участке кода нет ни каких ожиданий. Вы пишите пробовали "врайтлай и кей, ридлайн". А как Вы пробовали?
Если вставить в конец Вашего кода после всех команд

C#

1 2	Console.Write(" Для выхода нажмите любую клавишу ..... "); Console.ReadKey();

будет работать или нет?
Вы проверяли в Debug до Ваших команд ожидания доходит исполнение или происходит аварийное закрытие окна? Но при аварийном должно выдать исключение.

@PoPsa85 · 28.09.2018, 15:10

Элд Хасп, Да конечно он не будет работать. там синтаксические ошибки. Тут

C#

1	!(k1 <= n && m1 <= n)

тут

C#

1	Console.WriteLine("Ячейки {0}x{1} и {2}x{3} разные");

тут вообще зацикливание

C#

yach2:      if !(k1 <= n && m1 <= n)
{
                Console.WriteLine("Такой ячейки нет, попробуйте ввести другую");
                goto yach2;

@griffon88 · 28.09.2018, 15:16

C#

            Console.Write("Размерность доски: ");
            int n = int.Parse(Console.ReadLine());
            if (n % 2 == 0)
            {
                Console.WriteLine("Черных: {0}", n / 2 * n);
            }
            else
            {
                Console.WriteLine("Черных: {0}", (n - 1) / 2 * (n - 1) + n);
            }
      yach: Console.Write("k = ");
            int k = int.Parse(Console.ReadLine());
            Console.Write("m = ");
            int m = int.Parse(Console.ReadLine());
            switch (colour(k, m, n))
            {
                case 0:
                {
                    Console.WriteLine("Ячейка {0}x{1} - черная", k, m);
                    break;
                }
                case 1:
                {
                    Console.WriteLine("Ячейка {0}x{1} - белая", k, m);
                    break;
                }
                case 2:
                {
                    Console.WriteLine("Такой ячейки нет, попробуйте ввести другую");
                    goto yach;
                }
            }
     yach2: Console.WriteLine("k1 = ");
            int k1 = int.Parse(Console.ReadLine());
            Console.WriteLine("m1 = ");
            int m1 = int.Parse(Console.ReadLine());
            if (!(k1 <= n && m1 <= n))
            {
                Console.WriteLine("Такой ячейки нет, попробуйте ввести другую");
                goto yach2;
            }
 
            if (colour(k, m, n) == colour(k1, m1, n))
            {
                Console.WriteLine("Ячейки {0}x{1} и {2}x{3} одинаковые", k, m, k1, m1);
            }
            else
            {
                Console.WriteLine("Ячейки {0}x{1} и {2}x{3} разные", k, m, k1, m1);
 
            }
            Console.ReadKey();
        }
        static int colour(int k, int m, int n)
        {
            if (k <= n && m <= n)
            {
                if ((k + m) % 2 == 0)
                {
                    return 0;
                }
                return 1;
            }
            return 2;
        }

Вроде всё работает

@Элд Хасп · 28.09.2018, 15:18

Сообщение от PoPsa85

Да конечно он не будет работать. там синтаксические ошибки. Тут

Я не смотрел код. Так как автор Lidasag утверждаете, что всё работает как ему надо.
Только проблема с ожиданием перед закрытием окна.

@Элд Хасп Модератор 8603 / 5932 / 1690 Регистрация: 21.04.2018 Сообщений: 17,638 Записей в блоге: 2
	28.09.2018, 13:25	2
	Сообщение от Lidasag Как убрать ошибки в коде? Ошибки какие? Компиляции, исполнения, алгоритма? Конкретнее напишите какие ошибки, где и в каких случаях возникают. 0

@Lidasag 1 / 1 / 0 Регистрация: 25.12.2017 Сообщений: 40
	28.09.2018, 13:30 [ТС]	3
	Все выводится , но консоль сразу закрывается. Пробовала уже и врайтлай и кей, ридлайн, ридкей... 0

@V_Monomax 1405 / 1260 / 20 Регистрация: 09.08.2011 Сообщений: 2,321 Записей в блоге: 1
	28.09.2018, 14:06	5
	мне больше всего интересно что делает вот этот код Сообщение от Lidasag yach: и вот этот Сообщение от Lidasag yach2: это новая функция в C# или просто дичь от автора кода? 0

@PoPsa85 799 / 548 / 237 Регистрация: 01.07.2018 Сообщений: 1,810
	28.09.2018, 14:10	6
	V_Monomax, Это далеко не новая функция https://msdn.microsoft.com/ru-... (v=vs.120) 0

@V_Monomax 1405 / 1260 / 20 Регистрация: 09.08.2011 Сообщений: 2,321 Записей в блоге: 1
	28.09.2018, 14:22	7
	я конечно понимаю что ":" - это наследование, либо в литералах его можно встретить, но такое его применение здесь и не оправдано, да и код засоряет, так что это здесь лишнее. Добавлено через 8 минут Это я к чему, использование иди туда, ну вообще не оправдано, только код засоряет, это конечно привет бейсикам, и до свидания ООП. Зачем вам нужен C# с его подходом ООП, если вы его не используете? Зачем вам нужен тогда C#, если из бейсика вырасти не можете? Конечно применять все средства заложенные в языке хорошо, но только в тех случаях когда это оправдано. А при вашем подходе, очень похоже, существует проблема зацикливания (вываливаетесь по переполнению стека, либо подобная ей проблема). Так что ваш код надо рефакторить, что-бы в первую очередь вы могли перечитывать его). Не говорю уж о других (хотя другим вы будете тогда когда через более чем месяц будете сидеть и читать свой же код, и не понимать что к чему). 0

@Lidasag 1 / 1 / 0 Регистрация: 25.12.2017 Сообщений: 40
	28.09.2018, 14:47 [ТС]	8
	Код не мой. Мне без разницы что там написано, мне надо чтоб окно не уходило, после окончания программы 0

@Элд Хасп Модератор 8603 / 5932 / 1690 Регистрация: 21.04.2018 Сообщений: 17,638 Записей в блоге: 2
	28.09.2018, 15:18	12
	Сообщение от PoPsa85 Да конечно он не будет работать. там синтаксические ошибки. Тут Я не смотрел код. Так как автор Lidasag утверждаете, что всё работает как ему надо. Только проблема с ожиданием перед закрытием окна. 0