我如何打破循环并排除C＃中的最后一个输入？

Question

我必须编写一个代码，询问用户将要输入多少部电影。输入电影数量后，应要求用户输入电影名称。所有名称将添加到数组movies。如果用户输入“退出”，则程序应停止并显示用户已经输入的电影。

我写了这段代码：

Console.WriteLine("Enter number of movies:");
int num1 = int.Parse(Console.ReadLine());
string[] movies = new string[num1];

for (i = 0; i < num1; i++)
{
    Console.WriteLine("Enter movie name:");
    movies[i] = Console.ReadLine();
    if(movies[i] == "Exit" || movies[i] == "exit")
    {
        break;
    }
}
for (i = 0; i < num1; i++)
{
    Console.WriteLine("Movie {0} is {1}", i + 1, movies[i]);
}

问题是：如果用户写exit，程序将exit显示为电影。例如：用户想要输入4部电影，并输入first作为第一部电影，并输入exit作为第二部电影。在这种情况下，程序的输出如下：

movie 1 is first 
movie 2 is exit
movie 3 is
movie 4 is

但是该程序仅应显示输入的第一部电影。我在做什么错？

Answer 1

首先，让我们看看您拥有什么：在检查是否退出之前，请先添加值。

[此外，您可以通过忽略字母大小写进行比较（Broots Waymb建议的答案）。

最后，您将打印整个数组，而不检查是否存在某些值。

Console.WriteLine("Enter number of movies:");
int num1 = int.Parse(Console.ReadLine());
string[] movies = new string[num1];

for (i = 0; i < num1; i++)
 {
    Console.WriteLine("Enter movie name:");
    movies[i] = Console.ReadLine(); // you always add the value
    if(movies[i] == "Exit" || movies[i] == "exit") 
    {
      break;
    }
}
for (i = 0; i < num1; i++) // there is no condition to stop the iteration.
  Console.WriteLine("Movie {0} is {1}", i + 1, movies[i]);

为了解决打印问题，我们可以添加一个索引，该索引仅在输入电影时才会增加。或者，您可以在位置或数组中使用列表，然后使用foreach循环进行迭代。另一个解决方案是保留阵列，但将其转换为打印任务列表的可能性。

Console.WriteLine("Enter number of movies:");
int num1 = int.Parse(Console.ReadLine());
string[] movies = new string[num1];
int enteredMovies = 0;

for (i = 0; i < num1; i++)
{
    Console.WriteLine("Enter movie name:");
    string movie = Console.ReadLine(); 

    if(movie.Equals("exit", StringComparison.OrdinalIgnoreCase)
    {
       break;
    }
    else
    {
       movies[i] = movie;
       enteredMovies++;

    }

}

for (i = 0; i < enteredMovies; i++)
{
    Console.WriteLine("Movie {0} is {1}", i + 1, movies[i]);
}

或者您也可以使用我提到的列表。

Answer 2

将输入的影片设置为临时变量。如果不是您的退出条件，则仅添加它。

//Rest of the code left out for simplicity
Console.WriteLine("Enter movie name:");
string movie = Console.ReadLine();
if(movie.Equals("exit", StringComparison.OrdinalIgnoreCase)
    break;
else
    movies[i] = movie;  

您可以进行一些其他改进，例如使用List。这样，您就无需管理大小并提示用户。

Answer 3

使用List<T>可能比使用数组容易，除非分配明确要求使用数组。将使用以下内容：

1. 读取电影数量
2. 创建电影列表
3. 循环播放电影数量
   1. 从控制台读取一行并存储在一个临时变量中
   2. 检查临时变量是否等于出口
   3. 将临时变量添加到电影列表中，或中断循环
4. 现在，电影列表应该包含所有输入的电影，不要少了。

Answer 4

您有几个问题：

• 您首先输入数据，然后检查它。
• 您需要将输入转换为小写，以便无论用户输入exit, Exit, EXIT, ExIt，您的程序都将终止。

您的代码可以改进，但由于您是初学者，可能不了解高级知识，因此我将直接离开：

using System;

public class Test
{ 

    public static void Main(String[] args)
    {
        Console.WriteLine("Enter number of movies:");
        int num1 = int.Parse(Console.ReadLine());
        string[] movies = new string[num1];

        for (int i = 0; i < num1; i++)
        {
            Console.WriteLine("Enter movie name:");
            String movie = Console.ReadLine();
            if (movie.ToLower() == "exit")
                break;
            else
                movies[i] = movie;
        }
        for (int i = 0; i < num1; i++)
            Console.WriteLine("Movie {0} is {1}", i + 1, movies[i]);
    }
}

Answer 5

谢谢大家的关注。这是一个不错的社区。

我花了每个答案一点点，并设法解决了这个问题。我可以学到很多。

谢谢