Ошибка else не может запускать оператор

Asked
3 years, 9 months ago

Viewed
17k times

Оператор else не работает, если ввод в консоль не «Камень, ножницы или бумага», сообщение об исключении не отображается. Что является причиной этого.

using System;

namespace Rock__Paper__Scissors_
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.WriteLine("Lets play a game of Rock, Paper, Scissors.");
            Console.Write("Enter Rock, Paper or Scissors:");
            string userChoice = Console.ReadLine();

            Random r = new Random();
            int computerChoice = r.Next(3);

            //0 = Scissors
            if (computerChoice == 0)
            {
                if (userChoice == "Scissors")
                {
                    Console.WriteLine("Computer chose scissors!");
                    Console.WriteLine("TIE!");
                }

                else if (userChoice == "Rock")
                {
                    Console.WriteLine("Computer chose Scissors!");
                    Console.WriteLine("You WIN!");
                }

                else if (userChoice == "Paper")
                {
                    Console.WriteLine("Computer chose Scissors");
                    Console.WriteLine("You LOSE!");

                }
            }

            //1 = Rock
            else if (computerChoice == 1)
            {
                if (userChoice == "Scissors")
                {
                    Console.WriteLine("Computer chose Rock!");
                    Console.WriteLine("You LOSE!");
                }

                else if (userChoice == "Rock")
                {
                    Console.WriteLine("Computer chose Rock!");
                    Console.WriteLine("TIE!");
                }

                else if (userChoice == "Paper")
                {
                    Console.WriteLine("Computer chose Rock");
                    Console.WriteLine("You WIN!");
                }
            }

            //2 = Paper
            else if (computerChoice == 2)
            {
                if (userChoice == "Scissors")
                {
                    Console.WriteLine("Computer chose Paper!");
                    Console.WriteLine("You WIN");
                }

                else if (userChoice == "Rock")
                {
                    Console.WriteLine("Computer chose Paper!");
                    Console.WriteLine("You LOSE!");
                }

                else if (userChoice == "Paper")
                {
                    Console.WriteLine("Computer chose Paper");
                    Console.WriteLine("TIE!");
                }
            }

            //3 = Exception Handling
            else
            {
                Console.WriteLine("You must enter Rock, Paper or Scissors");
            }


        }
    }
}

Прежде чем продолжить, проверьте значение или userChoice … Я бы предпочел использовать цикл while

using System;

namespace Rock__Paper__Scissors_
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.WriteLine("Lets play a game of Rock, Paper, Scissors.");
            Console.Write("Enter Rock, Paper or Scissors:");
            string userChoice = Console.ReadLine();

            //Check it here in a while loop, until the user gets it 
            //right, the program will not proceed and loop here
            while (userChoice != "Scissors" || userChoice != "Rock" || userChoice != "Paper")
            {
                Console.Write("You must enter Rock, Paper or Scissors");
                userChoice = Console.ReadLine();
            }

            Random r = new Random();
            int computerChoice = r.Next(3);

            //0 = Scissors
            if (computerChoice == 0)
            {
                if (userChoice == "Scissors")
                {
                    Console.WriteLine("Computer chose scissors!");
                    Console.WriteLine("TIE!");
                }

                else if (userChoice == "Rock")
                {
                    Console.WriteLine("Computer chose Scissors!");
                    Console.WriteLine("You WIN!");
                }

                else if (userChoice == "Paper")
                {
                    Console.WriteLine("Computer chose Scissors");
                    Console.WriteLine("You LOSE!");

                }
            }

            //1 = Rock
            else if (computerChoice == 1)
            {
                if (userChoice == "Scissors")
                {
                    Console.WriteLine("Computer chose Rock!");
                    Console.WriteLine("You LOSE!");
                }

                else if (userChoice == "Rock")
                {
                    Console.WriteLine("Computer chose Rock!");
                    Console.WriteLine("TIE!");
                }

                else if (userChoice == "Paper")
                {
                    Console.WriteLine("Computer chose Rock");
                    Console.WriteLine("You WIN!");
                }
            }

            //2 = Paper
            else if (computerChoice == 2)
            {
                if (userChoice == "Scissors")
                {
                    Console.WriteLine("Computer chose Paper!");
                    Console.WriteLine("You WIN");
                }

                else if (userChoice == "Rock")
                {
                    Console.WriteLine("Computer chose Paper!");
                    Console.WriteLine("You LOSE!");
                }

                else if (userChoice == "Paper")
                {
                    Console.WriteLine("Computer chose Paper");
                    Console.WriteLine("TIE!");
                }
            }
        }
    }
}

Добавлено еще несколько операторов if / if else вместо else. Теперь он выплевывает ошибку исключения, которую я хотел.

Цель создания этого заключалась в том, чтобы практиковать / применять методы if, if else и else, поскольку я пытаюсь изучить C # и прорабатываю некоторые учебные пособия в Интернете. Конечно, есть, наверное, лучшие способы сделать эту игру.

-Нужно добавить какие-то петли (когда я научусь их делать). — Компьютер, кажется, генерирует случайное число в предсказуемой последовательности и не кажется таким уж случайным, поэтому мне нужно работать над улучшением этого.

Используя Систему;

namespace Rock__Paper__Scissors_
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.WriteLine("Lets play a game of Rock, Paper, Scissors.");
            Console.Write("Enter Rock, Paper or Scissors:");
            string userChoice = Console.ReadLine();

            Random r = new Random();
            int computerChoice = r.Next(2);


            //0 = Scissors
            if (computerChoice == 0)
            {
                if (userChoice == "Scissors")
                {
                    Console.WriteLine("Computer chose scissors!");
                    Console.WriteLine("TIE!");
                }

                else if (userChoice == "Rock")
                {
                    Console.WriteLine("Computer chose Scissors!");
                    Console.WriteLine("You WIN!");
                }

                else if (userChoice == "Paper")
                {
                    Console.WriteLine("Computer chose Scissors");
                    Console.WriteLine("You LOSE!");

                }
            }

            //1 = Rock
            else if (computerChoice == 1)
            {
                if (userChoice == "Scissors")
                {
                    Console.WriteLine("Computer chose Rock!");
                    Console.WriteLine("You LOSE!");
                }

                else if (userChoice == "Rock")
                {
                    Console.WriteLine("Computer chose Rock!");
                    Console.WriteLine("TIE!");
                }

                else if (userChoice == "Paper")
                {
                    Console.WriteLine("Computer chose Rock");
                    Console.WriteLine("You WIN!");
                }
            }

            //2 = Paper
            else if (computerChoice == 2)
            {
                if (userChoice == "Scissors")
                {
                    Console.WriteLine("Computer chose Paper!");
                    Console.WriteLine("You WIN");
                }

                else if (userChoice == "Rock")
                {
                    Console.WriteLine("Computer chose Paper!");
                    Console.WriteLine("You LOSE!");
                }

                else if (userChoice == "Paper")
                {
                    Console.WriteLine("Computer chose Paper");
                    Console.WriteLine("TIE!");
                }
            }


            //Exception Handling
            if (userChoice != "Scissors")
            {
                Console.WriteLine("Choose Rock, Paper or Scissors");
            }

            else if (userChoice != "Rock")
            {
                Console.WriteLine("Choose Rock, Paper or Scissors");
            }

            else if (userChoice != "Paper")
            {
                Console.WriteLine("Choose Rock, Paper or Scissors");
            }


        }
    }
}

let swiperNav = document.querySelectorAll('.work__stages-nav')
let number = document.querySelectorAll('.number')
let swiperNextBtn = document.querySelector('.swiper-button-next')
let swiperPrevtBtn = document.querySelector('.swiper-button-prev')
let swiperTitle = document.querySelectorAll('.swiper__item-info__title')

swiperNextBtn.addEventListener('click', function (e) {
  let swiperTitle = document.querySelectorAll('.swiper__item-info__title')
  let number = document.querySelectorAll('.number')
  for (let i = 0; i < swiperTitle.length; i++) {
    if (swiperTitle[i].innerHTML == 'Выбор материала и цвета') {
      swiperNav[0].style.color = '#373E46'
      number[0].style.color = '#373E46'
      number[0].style.background = 'white'
      swiperNav[1].style.color = '#621F2A'
      number[1].style = `
      background: #621F2A;
      color: white;
      `
    } else if (swiperTitle[i].innerHTML == 'Дизайн проект и изготовление') {
      swiperNav[1].style.color = '#373E46'
      number[1].style.color = '#373E46'
      number[1].style.background = 'white'
      swiperNav[2].style.color = '#621F2A'
      number[2].style = `
      background: #621F2A;
      color: white;
      `
      
    }  else if (swiperTitle[i].innerHTML == 'Установка гриль кухни') {
      swiperNav[2].style.color = '#373E46'
      number[2].style.color = '#373E46'
      number[2].style.background = 'white'
      swiperNav[3].style.color = '#621F2A'
      number[3].style = `
      background: #621F2A;
      color: white;
      `
    }
  }
})

• 
  Вопрос задан

  27 янв.

• 93 просмотра

Пригласить эксперта

Трудно определить точную проблему без дополнительной информации о коде и контексте, в котором он используется. Однако одна из потенциальных проблем может заключаться в том, что оператор else if не связан должным образом с начальным оператором if. Для того чтобы оператор else if работал как положено, он должен непосредственно предшествовать оператору if, без какого-либо дополнительного кода между ними. Другая потенциальная проблема может заключаться в том, что условия в операторах if и else if не выполняются, в результате чего код в этих блоках не выполняется.

• 
  Показать ещё
  Загружается…
  

Обновлено:
Моя проблема была решена благодаря пользователю Крису Ларабеллу, спасибо всем, что откликнулись.

Проблема, возникающая с моим кодом, заключается в том, что, когда указанный файл отсутствует в каталоге Desktop, консоль закроется и не перейдет к оператору else, что происходит, когда файл отсутствует. Однако, когда файл присутствует, консоль будет работать полностью нормально, это просто инструкция else.
Вот мой код, который используется.

if (inputDrive == "search.system")
        {
            try
            {
                string Desktop = Environment.GetFolderPath(Environment.SpecialFolder.DesktopDirectory);
                string DeleteFile = @"delete.txt";
                string[] fileList = System.IO.Directory.GetFiles(Desktop, DeleteFile);
                foreach (string file in fileList)
                {
                    if (System.IO.File.Exists(file))
                    {
                        System.IO.File.Delete(file);
                        Console.WriteLine("File has been deleted");
                        Console.ReadLine();
                    }
                    else
                    {
                        Console.Write("File could not be found");
                        Console.ReadLine();
                    }
                }
            }
            catch (System.IO.FileNotFoundException)
            {
                Console.WriteLine("search has encountered an error");
                Console.ReadLine();
            }
        }

Я пытаюсь найти в каталоге Desktop файл с именем ‘delete.txt’ и удалить его, когда пользователь вводит search.system. тогда консоль ответит вам, что файл был удален. Если файл не был найден, он ответит вам через консоль, что «файл не может быть найден». Если произойдет ошибка, он перейдет к перехвату и скажет «поиск обнаружил ошибку».

Я также хочу сказать, что мне очень жаль, если этот код запутан и / или если это совершенно не соответствует тому, что я пытаюсь выполнить. Я новичок в C# и новичок в кодировании в целом.

Вы могли бы поместить оператор if, чтобы проверить, что длина fileList равна > 0. Если длина файла равна нулю, файл не найден. В противном случае вы можете продолжить удаление файла.

Кроме того, не расстраивайтесь, как новый программист. Установите точку останова в строке, где вы используете метод GetFiles(), и шаг (F11) к следующей строке. Наведите курсор на переменную fileList и посмотрите, равно ли нулю количество элементов в массиве.

System.IO.Directory.GetFiles ()

Похоже, вы просто ищете определенный файл по имени и удаляете его, если он существует. Вы можете упростить свой код, выполнив следующие действия:

if (inputDrive == "search.system")
{
    try
    {
        string Desktop = Environment.GetFolderPath(Environment.SpecialFolder.DesktopDirectory);
        string DeleteFile = @"delete.txt";
        string filePath = System.IO.Path.Combine(Desktop, DeleteFile);

        if (System.IO.File.Exists(filePath))
        {
            System.IO.File.Delete(filePath);
            Console.WriteLine("File has been deleted");
            Console.ReadLine();
        }
        else
        {
            Console.Write("File could not be found");
            Console.ReadLine();
        }
    }
    catch (System.Exception ex)
    {
        Console.WriteLine($"search has encountered an error: {ex}");
        Console.ReadLine();
    }
}