第一次迭代跳过循环中的第一步

Question

我正在使用 Rob Miles 的 C# 编程黄皮书学习 C#。在本文中，他讨论了一个计算器示例，用于确定所需的总玻璃面积（以平方米为单位）。我重新创建了这段代码，并想尝试用对我有意义的东西来改进它。例如，在他的代码中，他使用了很多看起来对我没有吸引力的空行。我想试着让它看起来更干净一点。我主要是想去掉第一个空行，但要确保空行出现在下一行。

这是原始代码。

double width, height, woodLength, glassArea;

        const double MAX_WIDTH = 5.0;
        const double MIN_WIDTH = 0.5;
        const double MAX_HEIGHT = 3.0;
        const double MIN_HEIGHT = 0.75;

        string widthString, heightString;

        do {
            Console.Write("\nGive the width of the window between " + MIN_WIDTH + " and " + MAX_WIDTH + ": ");
            widthString = Console.ReadLine();
            width = double.Parse(widthString);

            if (width < MIN_WIDTH){
                Console.WriteLine("Width is too small.");
            }

            if (width > MAX_WIDTH){
                Console.WriteLine("Width is too large.");
            }
        } while (width < MIN_WIDTH || width > MAX_WIDTH);

我尝试使用 goto，但因为循环是不同的方法（？）它不起作用。通过一些谷歌搜索，我发现 goto 是非常糟糕的做法，但我想不出任何其他方法可以使用。 （我也不知道）

            Console.Write("Give the width of the window between " + MIN_WIDTH + " and " + MAX_WIDTH + ": ");

        goto first_loop;

        do {
            Console.Write("\nGive the width of the window between " + MIN_WIDTH + " and " + MAX_WIDTH + ": ");
            first_loop:
            widthString = Console.ReadLine();
            width = double.Parse(widthString);

            if (width < MIN_WIDTH){
                Console.WriteLine("Width is too small.");
            }

            if (width > MAX_WIDTH){
                Console.WriteLine("Width is too large.");
            }
        } while (width < MIN_WIDTH || width > MAX_WIDTH);

Answer 1

我主要想删除第一个空行，但要确保 下一行会出现空行。

从提示中删除\n，并将其添加到两条错误消息的末尾：

do
{
    Console.Write("Give the width of the window between " + MIN_WIDTH + " and " + MAX_WIDTH + ": ");
    widthString = Console.ReadLine();
    width = double.Parse(widthString);

    if (width < MIN_WIDTH)
    {
        Console.WriteLine("Width is too small.\n");
    }
    else if (width > MAX_WIDTH)
    {
        Console.WriteLine("Width is too large.\n");
    }
} while (width < MIN_WIDTH || width > MAX_WIDTH);

但是，如果我自己编写此代码，我会使用布尔值来表示总体成功，并在 while 循环条件下检查它。我也会使用double.TryParse()，如下所示：

bool valid = false;
do
{
    valid = true; // assume good until proven otherwise
    Console.Write("Give the width of the window between " + MIN_WIDTH + " and " + MAX_WIDTH + ": ");
    widthString = Console.ReadLine();
    if (double.TryParse(widthString, out width))
    {
        if (width < MIN_WIDTH || width >MAX_WIDTH )
        {
            valid = false;
            Console.WriteLine("Width is too " + ((width<MIN_WIDTH) ? "small" : "large") + ".");
        }
    }
    else
    {
        Console.WriteLine("Invalid width entry. Please try again.");
        valid = false;
    }
    
    if (!valid)
    {
        Console.WriteLine();
    }
} while (!valid);

Answer 2

如果您也不想像@Idle_Mind 的解决方案那样在末尾添加额外的空行，您可以这样做

bool isFirstRun = true;
do
{
    if (!isFirstRun) Console.WriteLine();
    isFirstRun = false;

    Console.Write("Give the width of the window between " + MIN_WIDTH + " and " + MAX_WIDTH + ": ");
    widthString = Console.ReadLine();
    width = double.Parse(widthString);

    if (width < MIN_WIDTH)
    {
      Console.WriteLine("Width is too small.");
    }

    if (width > MAX_WIDTH)
    {
      Console.WriteLine("Width is too large.");
    }
} while (width < MIN_WIDTH || width > MAX_WIDTH);