Do while循环在不满足条件的情况下移动到下一步

Question

这周我刚刚开始学习C#并且正在尝试运行一个简单的代码，当用户输入文本时，它会提示用户输入一个数字，或者当用户输入一个负数时，它会提示用户输入一个正数（因此，对于文本是一个布尔运算，对于负数是一个if语句）。如果他们输入了一个有效的（正）数字，程序将继续进行其余的步骤。

然而，在此代码中，如果用户输入一个负数，然后输入一个文本，然后输入另一个负数，依此类推，似乎会中断循环并继续下一个操作。

代码是一个更大的程序的一部分，所以我把它缩小了，只抽出最关键的部分来运行。有人能看出我错过了什么吗？

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace IncomeTaxCalculator
{
    class IncomeTax
    {
        public static void Main()
        {
            double income;
            income = income_input();
            show_output(income);
        }
        public static double income_input()
        {
            double income; string income_string; bool bad_value = true;
            do
            {
                Console.Write("What is your total income: ");
                income_string = Console.ReadLine();
                if (double.TryParse(income_string, out income))
                {
                    bad_value = false;
                }
                else
                {
                    Console.WriteLine("Enter your income as a whole-dollar numeric figure.");
                }
                if (income < 0)
                {
                    Console.WriteLine("Your income cannot be a negative");
                }
            } while (bad_value || income < 0);
            return income;
        }
              public static void show_output(double income)
        {
            Console.WriteLine("Your income is " + income);
            Console.WriteLine("\n\n Hit Enter to exit.");
            Console.ReadLine();
        }
    }
}

Answer 1

我知道这已经被接受了，但这可以在一个简单得多的循环中完成。为什么不创建一个无限循环，并在满足值时创建break/return。而不是检查无效的输入，搜索有效的输入。

我不会详细说明为什么这是一个更容易接受的解决方案，考虑一下给出的指令，如果您输入了一个无效的输入，那么您的指令就是错误的。相反，检查积极的结果。看这个！！

static double income_input()
{
    double income = double.NaN;
    while (true)
    {
        Console.WriteLine("What is your income?:");
        if (double.TryParse(Console.ReadLine(), out income) && income > 0)
            return income;
        Console.WriteLine("Invalid input. Please enter a valid number greater than zero.");
    }
}

实际上，我们在这里所做的只是使用while(true)创建了一个无限循环。所以现在循环永远不会结束，除非我们明确地告诉它结束。

如果您想要尽可能短的代码，可以使用一些C#7语法来处理内联变量。

static double income_input()
{
    while (true)
    {
        Console.WriteLine("What is your income?:");
        if (double.TryParse(Console.ReadLine(), out double income) && income > 0)
            return income;
        Console.WriteLine("Invalid input. Please enter a valid number greater than zero.");
    }
}

编码快乐！。