Изменение кода

@DayZzZ · Регистрация: 02.10.2018

Author24 — интернет-сервис помощи студентам

Помогите модернизировать код. Желательно убрать goto;

C#

 while (true) //включаем бесконечный цикл который будем прерывать по требованию
            {
                ///
                /// первый ферзь
                ///
            f8: //метка для возврата
                clearBoard(); //нажимаем на кнопку очистки поля
                z = 1;                
                g = rand.Next(1, 9); //заносим случайное значение от 1 до 8
                while (z < 9) //цикл перебора всех вертикалей для ферзя
                {
                    (Controls[board[g] + z] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем рандома с типом PictureBox и замена его картинкой с красной точкой
                    z++; //следующий элемент вертикали
                }
                ///
                /// перебираем только вертикали потомучто горизонталь нам незачем так как на горизонталь нельзя ставить второго ферзя
                ///
                ///
                /// диагонали
                ///
                v = g - 1; //присваиваем переменной вертикали рандом
                h = 8 - 1; //присваиваем переменной горизонтали
                while (v > 0 && h > 0) //цикл перебора диагонали от позиции ферзя до левого верхнего предела, цикл вплоть до конца доски по вертикали или горизонтали
                {
                    (Controls[board[v] + h] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем элемента массива с индексом v + цифра горизонтали h
                    v--; //уменьшаем вертикаль на 1 для перехода на след элемент диагонали
                    h--; //увеличиваем горизонталь на 1 для перехода на след элемент диагонали
                }
                v = g + 1; //присваиваем переменной вертикали рандом
                h = 8 - 1; //присваиваем переменной горизонтали
                while (v < 9 && h > 0) //цикл перебора диагонали от позиции ферзя до правого верхрего предела, цикл вплоть до конца доски по вертикали или горизонтали
                {
                    (Controls[board[v] + h] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем элемента массива с индексом v + цифра горизонтали h
                    v++; //увеличиваем вертикаль на 1 для перехода на след элемент диагонали
                    h--; //увеличиваем горизонталь на 1 для перехода на след элемент диагонали
                }
                (Controls[board[g] + "8"] as PictureBox).Image = Properties.Resources.chf; //вывод первого ферзя на поле                
 
                ///
                ///второй ферзь
                ///
            f7: //метка для возврата
                z = 1;
                g = rand.Next(1, 9); //заносим случайное значение от 1 до 8
                if ((Controls[board[g] + "7"] as PictureBox).Image == null) //если выбранное поле пустое то можно ставить ферзя
                {
                    while (z < 8) //цикл перебора всех вертикалей для ферзя
                    {
                        (Controls[board[g] + z] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем рандома с типом PictureBox и замена его картинкой с красной точкой
                        z++; //следующий элемент вертикали
                    }
                    ///
                    /// диагонали
                    ///
                    v = g - 1; //присваиваем переменной вертикали рандом
                    h = 7 - 1; //присваиваем переменной горизонтали
                    while (v > 0 && h > 0) //цикл перебора диагонали от позиции ферзя до левого верхнего предела, цикл вплоть до конца доски по вертикали или горизонтали
                    {
                        (Controls[board[v] + h] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем элемента массива с индексом v + цифра горизонтали h
                        v--; //уменьшаем вертикаль на 1 для перехода на след элемент диагонали
                        h--; //увеличиваем горизонталь на 1 для перехода на след элемент диагонали
                    }
                    v = g + 1; //присваиваем переменной вертикали рандом
                    h = 7 - 1; //присваиваем переменной горизонтали
                    while (v < 9 && h > 0) //цикл перебора диагонали от позиции ферзя до правого верхрего предела, цикл вплоть до конца доски по вертикали или горизонтали
                    {
                        (Controls[board[v] + h] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем элемента массива с индексом v + цифра горизонтали h
                        v++; //увеличиваем вертикаль на 1 для перехода на след элемент диагонали
                        h--; //увеличиваем горизонталь на 1 для перехода на след элемент диагонали
                    }
                }
                else //иначе если выбранное поле заполнено то перебираем заного
                {
                    count++;
                    if (count > 10)
                    {
                        sch = 0; //обнуляем счетчик
                        goto f8; //идем в начало цикла к первому ферзю и ставим его заного
                    }
                    goto f7; //идем к метке
                }
                (Controls[board[g] + "7"] as PictureBox).Image = Properties.Resources.chf; //выводим позицию второго ферзя
                count = 0;
 
            ///
            ///третий ферзь
            ///
            f6: //метка для возврата
                z = 1;
                g = rand.Next(1, 9); //заносим случайное значение от 1 до 8
                if ((Controls[board[g] + "6"] as PictureBox).Image == null) //если выбранное поле пустое то можно ставить ферзя
                {
                    while (z < 7) //цикл перебора всех вертикалей для ферзя
                    {
                        (Controls[board[g] + z] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем рандома с типом PictureBox и замена его картинкой с красной точкой
                        z++; //следующий элемент вертикали
                    }
                    ///
                    /// диагонали
                    ///
                    v = g - 1; //присваиваем переменной вертикали индекс рандом
                    h = 6 - 1; //присваиваем переменной горизонтали индекс
                    while (v > 0 && h > 0) //цикл перебора диагонали от позиции ферзя до левого верхнего предела, цикл вплоть до конца доски по вертикали или горизонтали
                    {
                        (Controls[board[v] + h] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем элемента массива с индексом v + цифра горизонтали h
                        v--; //уменьшаем вертикаль на 1 для перехода на след элемент диагонали
                        h--; //увеличиваем горизонталь на 1 для перехода на след элемент диагонали
                    }
                    v = g + 1; //присваиваем переменной вертикали индекс введенной буквы из массива
                    h = 6 - 1; //присваиваем переменной горизонтали индекс введенной цифры из текстбокса
                    while (v < 9 && h > 0) //цикл перебора диагонали от позиции ферзя до правого верхрего предела, цикл вплоть до конца доски по вертикали или горизонтали
                    {
                        (Controls[board[v] + h] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем элемента массива с индексом v + цифра горизонтали h
                        v++; //увеличиваем вертикаль на 1 для перехода на след элемент диагонали
                        h--; //увеличиваем горизонталь на 1 для перехода на след элемент диагонали
                    }
                }
                else //иначе если выбранное поле заполнено то перебираем заного
                {
                    count++;
                    if (count > 10)
                    {
                        sch = 0; //обнуляем счетчик
                        goto f8; //идем в начало цикла к первому ферзю и ставим его заного
                    }
                    goto f6; //идем к метке
                }
                (Controls[board[g] + "6"] as PictureBox).Image = Properties.Resources.chf; //выводим позицию третьего ферзя
                count = 0;
 
            ///
            ///четвертый ферзь
            ///
            f5: //метка для возврата
                z = 1;
                g = rand.Next(1, 9); //заносим случайное значение от 1 до 8
                if ((Controls[board[g] + "5"] as PictureBox).Image == null) //если выбранное поле пустое то можно ставить ферзя
                {
                    while (z < 6) //цикл перебора всех вертикалей для ферзя
                    {
                        (Controls[board[g] + z] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем рандома с типом PictureBox и замена его картинкой с красной точкой
                        z++; //следующий элемент вертикали
                    }
                    ///
                    /// диагонали
                    ///
                    v = g - 1; //присваиваем переменной вертикали рандом
                    h = 5 - 1; //присваиваем переменной горизонтали
                    while (v > 0 && h > 0) //цикл перебора диагонали от позиции ферзя до левого верхнего предела, цикл вплоть до конца доски по вертикали или горизонтали
                    {
                        (Controls[board[v] + h] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем элемента массива с индексом v + цифра горизонтали h
                        v--; //уменьшаем вертикаль на 1 для перехода на след элемент диагонали
                        h--; //увеличиваем горизонталь на 1 для перехода на след элемент диагонали
                    }
                    v = g + 1; //присваиваем переменной вертикали индекс введенной буквы из массива
                    h = 5 - 1; //присваиваем переменной горизонтали индекс введенной цифры из текстбокса
                    while (v < 9 && h > 0) //цикл перебора диагонали от позиции ферзя до правого верхрего предела, цикл вплоть до конца доски по вертикали или горизонтали
                    {
                        (Controls[board[v] + h] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем элемента массива с индексом v + цифра горизонтали h
                        v++; //увеличиваем вертикаль на 1 для перехода на след элемент диагонали
                        h--; //увеличиваем горизонталь на 1 для перехода на след элемент диагонали
                    }
                }
                else //иначе если выбранное поле заполнено то перебираем заного
                {
                    count++;
                    if (count > 10)
                    {
                        sch = 0; //обнуляем счетчик
                        goto f8; //идем в начало цикла к первому ферзю и ставим его заного
                    }
                    goto f5; //идем к метке
                }
                (Controls[board[g] + "5"] as PictureBox).Image = Properties.Resources.chf; //выводим позицию четвертого ферзя
                count = 0;
 
            ///
            ///пятый ферзь
            ///
            f4: //метка для возврата
                z = 1;
                g = rand.Next(1, 9); //заносим случайное значение от 1 до 8
                if ((Controls[board[g] + "4"] as PictureBox).Image == null) //если выбранное поле пустое то можно ставить ферзя
                {
                    while (z < 5) //цикл перебора всех вертикалей для ферзя
                    {
                        (Controls[board[g] + z] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем рандома с типом PictureBox и замена его картинкой с красной точкой
                        z++; //следующий элемент вертикали
                    }
                    ///
                    /// диагонали
                    ///
                    v = g - 1; //присваиваем переменной вертикали рандом
                    h = 4 - 1; //присваиваем переменной горизонтали
                    while (v > 0 && h > 0) //цикл перебора диагонали от позиции ферзя до левого верхнего предела, цикл вплоть до конца доски по вертикали или горизонтали
                    {
                        (Controls[board[v] + h] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем элемента массива с индексом v + цифра горизонтали h
                        v--; //уменьшаем вертикаль на 1 для перехода на след элемент диагонали
                        h--; //увеличиваем горизонталь на 1 для перехода на след элемент диагонали
                    }
                    v = g + 1; //присваиваем переменной вертикали индекс введенной буквы из массива
                    h = 4 - 1; //присваиваем переменной горизонтали индекс введенной цифры из текстбокса
                    while (v < 9 && h > 0) //цикл перебора диагонали от позиции ферзя до правого верхрего предела, цикл вплоть до конца доски по вертикали или горизонтали
                    {
                        (Controls[board[v] + h] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем элемента массива с индексом v + цифра горизонтали h
                        v++; //увеличиваем вертикаль на 1 для перехода на след элемент диагонали
                        h--; //увеличиваем горизонталь на 1 для перехода на след элемент диагонали
                    }
                }
                else //иначе если выбранное поле заполнено то перебираем заного
                {
                    count++;
                    if (count > 10)
                    {
                        sch = 0; //обнуляем счетчик
                        goto f8; //идем в начало цикла к первому ферзю и ставим его заного
                    }
                    goto f4; //идем к метке
                }
                (Controls[board[g] + "4"] as PictureBox).Image = Properties.Resources.chf; //выводим позицию пятого ферзя
                count = 0;
 
            ///
            ///шестой ферзь
            ///
            f3: //метка для возврата
                z = 1;
                g = rand.Next(1, 9); //заносим случайное значение от 1 до 8
                for (int i = 1; i < 9; i++) //цикл перебора элементов горизонтали
                {
                    if ((Controls[board[i] + "3"] as PictureBox).Image != null) //если есть красная точка
                    {
                        sch++;//увеличиваем счетчик
                    }
                }
                if (sch == 8) //если счетчик после цикла равен 8 тоесть все клетки заняты
                {
                    sch = 0; //обнуляем счетчик
                    goto f8; //идем в начало цикла к первому ферзю и ставим его заного
                }
                sch = 0; //обнуляем счетчик
                if ((Controls[board[g] + "3"] as PictureBox).Image == null) //если выбранное поле пустое то можно ставить ферзя
                {
                    while (z < 4) //цикл перебора всех вертикалей для ферзя
                    {
                        (Controls[board[g] + z] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем рандома с типом PictureBox и замена его картинкой с красной точкой
                        z++; //следующий элемент вертикали
                    }
                    ///
                    /// диагонали
                    ///
                    v = g - 1; //присваиваем переменной вертикали рандом
                    h = 3 - 1; //присваиваем переменной горизонтали
                    while (v > 0 && h > 0) //цикл перебора диагонали от позиции ферзя до левого верхнего предела, цикл вплоть до конца доски по вертикали или горизонтали
                    {
                        (Controls[board[v] + h] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем элемента массива с индексом v + цифра горизонтали h
                        v--; //уменьшаем вертикаль на 1 для перехода на след элемент диагонали
                        h--; //увеличиваем горизонталь на 1 для перехода на след элемент диагонали
                    }
                    v = g + 1; //присваиваем переменной вертикали индекс введенной буквы из массива
                    h = 3 - 1; //присваиваем переменной горизонтали индекс введенной цифры из текстбокса
                    while (v < 9 && h > 0) //цикл перебора диагонали от позиции ферзя до правого верхрего предела, цикл вплоть до конца доски по вертикали или горизонтали
                    {
                        (Controls[board[v] + h] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем элемента массива с индексом v + цифра горизонтали h
                        v++; //увеличиваем вертикаль на 1 для перехода на след элемент диагонали
                        h--; //увеличиваем горизонталь на 1 для перехода на след элемент диагонали
                    }
                }
                else //иначе если выбранное поле заполнено то перебираем заного
                {
                    count++;
                    if (count > 10)
                    {
                        sch = 0; //обнуляем счетчик
                        goto f8; //идем в начало цикла к первому ферзю и ставим его заного
                    }
                    goto f3; //идем к метке
                }
                (Controls[board[g] + "3"] as PictureBox).Image = Properties.Resources.chf; //выводим позицию шестого ферзя
                count = 0;
 
            ///
            ///седьмой ферзь
            ///
            f2: //метка для возврата
                z = 1;
                g = rand.Next(1, 9); //заносим случайное значение от 1 до 8
                for (int i = 1; i < 9; i++) //цикл перебора элементов горизонтали
                {
                    if ((Controls[board[i] + "2"] as PictureBox).Image != null) //если есть красная точка
                    {
                        sch++; //увеличиваем счетчик
                    }
                }
                if (sch == 8) //если счетчик после цикла равен 8 тоесть все клетки заняты
                {
                    sch = 0; //обнуляем счетчик
                    goto f8; //идем в начало цикла к первому ферзю и ставим его заного
                }
                sch = 0; //обнуляем счетчик
                if ((Controls[board[g] + "2"] as PictureBox).Image == null) //если выбранное поле пустое то можно ставить ферзя
                {
                    while (z < 3) //цикл перебора всех вертикалей для ферзя
                    {
                        (Controls[board[g] + z] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем рандома с типом PictureBox и замена его картинкой с красной точкой
                        z++; //следующий элемент вертикали
                    }
                    ///
                    /// диагонали
                    ///
                    v = g - 1; //присваиваем переменной вертикали рандом
                    h = 2 - 1; //присваиваем переменной горизонтали
                    while (v > 0 && h > 0) //цикл перебора диагонали от позиции ферзя до левого верхнего предела, цикл вплоть до конца доски по вертикали или горизонтали
                    {
                        (Controls[board[v] + h] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем элемента массива с индексом v + цифра горизонтали h
                        v--; //уменьшаем вертикаль на 1 для перехода на след элемент диагонали
                        h--; //увеличиваем горизонталь на 1 для перехода на след элемент диагонали
                    }
                    v = g + 1; //присваиваем переменной вертикали индекс введенной буквы из массива
                    h = 2 - 1; //присваиваем переменной горизонтали индекс введенной цифры из текстбокса
                    while (v < 9 && h > 0) //цикл перебора диагонали от позиции ферзя до правого верхрего предела, цикл вплоть до конца доски по вертикали или горизонтали
                    {
                        (Controls[board[v] + h] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем элемента массива с индексом v + цифра горизонтали h
                        v++; //увеличиваем вертикаль на 1 для перехода на след элемент диагонали
                        h--; //увеличиваем горизонталь на 1 для перехода на след элемент диагонали
                    }
                }
                else //иначе если выбранное поле заполнено то перебираем заного
                {
                    count++;
                    if (count > 10)
                    {
                        sch = 0; //обнуляем счетчик
                        goto f8; //идем в начало цикла к первому ферзю и ставим его заного
                    }
                    goto f2; //идем к метке
                }
                (Controls[board[g] + "2"] as PictureBox).Image = Properties.Resources.chf; //выводим позицию седьмого ферзя
                count = 0;
 
            ///
            ///восьмой ферзь
            ///
            f1: //метка для возврата
                z = 1;
                g = rand.Next(1, 9); //заносим случайное значение от 1 до 8
                for (int i = 1; i < 9; i++) //цикл перебора элементов горизонтали
                {
                    if ((Controls[board[i] + "1"] as PictureBox).Image != null) //если есть красная точка
                    {
                        sch++; //увеличиваем счетчик
                    }
                }
                if (sch == 8) //если счетчик после цикла равен 8 тоесть все клетки заняты
                {
                    sch = 0; //обнуляем счетчик
                    goto f8; //идем в начало цикла к первому ферзю и ставим его заного
                }
                sch = 0; //обнуляем счетчик
                if ((Controls[board[g] + "1"] as PictureBox).Image == null) //если выбранное поле пустое то можно ставить ферзя
                {
                    while (z < 2) //цикл перебора всех вертикалей для ферзя
                    {
                        (Controls[board[g] + z] as PictureBox).Image = Properties.Resources.bad; //поиск на форме контрола с именем рандома с типом PictureBox и замена его картинкой с красной точкой
                        z++; //следующий элемент вертикали
                    }
                    ///
                    /// диагонали
                    ///
                    v = g - 1; 
                    h = 1 - 1; 
                    while (v > 0 && h > 0) 
                    {
                        (Controls[board[v] + h] as PictureBox).Image = Properties.Resources.bad; 
                        v--; 
                        h--; 
                    }
                    v = g + 1; 
                    h = 1 - 1; 
                    while (v < 9 && h > 0) /
                    {
                        (Controls[board[v] + h] as PictureBox).Image = Properties.Resources.bad; //
                        v++; 
                        h--; 
                    }
                }
                else //иначе если выбранное поле заполнено то перебираем заного
                {
                    count++;
                    if (count > 10)
                    {
                        sch = 0; //обнуляем счетчик
                        goto f8; //идем в начало цикла к первому ферзю и ставим его заного
                    }
                    goto f1; //идем к метке
                }
 
                (Controls[board[g] + "1"] as PictureBox).Image = Properties.Resources.chf; //выводим позицию восьмого ферзя
 
                for (int i = 1; i < board.Length; i++)
                {
                    for (int j = 1; j < 9; j++)
                    {
                        if ((Controls[board[i] + j.ToString()] as PictureBox).Image == null)
                            (Controls[board[i] + j.ToString()] as PictureBox).Image = Properties.Resources.bad;
                    }
                }

Someone007 · 04.03.2019, 20:56

Вам бы сначала от дублирования кода избавиться...

Новые блоги и статьи
Как проверить, что файл НЕ существует с помощью Bash bytestream 25.01.2025 В системном администрировании проверка существования файлов является фундаментальной операцией, особенно при написании скриптов на Bash. Правильная обработка ситуаций, когда файл отсутствует,. . .	Как проверить, что строка пустая, неопределена или null в JavaScript bytestream 25.01.2025 При разработке веб-приложений на JavaScript разработчики регулярно сталкиваются с необходимостью проверки строковых значений на различные состояния. Валидация данных является критически важным. . .	Как проверить существование ключа в объекте или массиве в JavaScript bytestream 25.01.2025 При разработке веб-приложений на JavaScript программисты постоянно сталкиваются с необходимостью работать с объектами и массивами. Эти структуры данных являются фундаментальными элементами языка и. . .	Как создать ArrayList из массива (array) в Java bytestream 25.01.2025 При разработке программного обеспечения на языке Java программисты часто сталкиваются с необходимостью эффективного управления наборами данных. Массивы и ArrayList являются двумя фундаментальными. . .	Как сгенерировать случайные целые числа (int) в указанном диапазоне в Java bytestream 25.01.2025 В программировании генерация случайных чисел является важнейшим инструментом для решения множества практических задач. От создания игровых механик до реализации алгоритмов машинного обучения, от. . .	Как сделать, чтобы grep выводил строки до и после найденной hw_wired 25.01.2025 Утилита grep стала незаменимым инструментом для поиска и фильтрации информации в Unix-подобных операционных системах. Эта командная утилита позволяет осуществлять поиск строк, соответствующих. . .
Как клонировать определенную ветку в Git bytestream 24.01.2025 Одной из ключевых функций Git является возможность клонирования веток, что позволяет создавать локальные копии удаленных репозиториев и работать с определенными версиями проекта. Этот механизм. . .	Как в цикле обойти строки DataFrame в Pandas Python bytestream 24.01.2025 DataFrame представляет собой одну из основных структур данных в библиотеке Python Pandas, которая организует информацию в виде двумерной таблицы с строками и столбцами. Эта структура данных особенно. . .	Как получить имя текущей ветки в Git bytestream 24.01.2025 При работе с Git часто возникает необходимость определить имя текущей ветки, в которой ведется разработка. Знание текущей ветки является критически важным аспектом для эффективного управления. . .	Как отсортировать массив объектов по значению поля объекта в JavaScript bytestream 24.01.2025 При разработке веб-приложений на JavaScript разработчики часто сталкиваются с необходимостью работать с массивами объектов. Эти структуры данных представляют собой упорядоченные наборы элементов, где. . .	Ошибка "src refspec master does not match any" при пуше коммита в Git bytestream 24.01.2025 При работе с системой контроля версий Git разработчики нередко сталкиваются с различными ошибками, одной из которых является сообщение "src refspec master does not match any". Эта ошибка возникает. . .	Как округлить не более двух цифр после запятой в JavaScript bytestream 24.01.2025 При работе с числами в JavaScript разработчики часто сталкиваются с необходимостью округления десятичных значений до определенного количества знаков после запятой. Это особенно важно при работе с. . .

Someone007 6512 / 4088 / 1606 Регистрация: 09.05.2015 Сообщений: 9,554
	04.03.2019, 20:56	2
	Вам бы сначала от дублирования кода избавиться... 0