检查输入是否在数字范围之间 - 在 C# 中

Question

在控制台应用程序中，我试图检查年份的输入值是否在 1-10 之间。

目前我也在使用以下方法检查输入是否为有效的 int。

int numberOfYears;

Console.WriteLine("Enter the number of years:");
string numberOfYearsInput = Console.ReadLine();

while (!Int32.TryParse(numberOfYearsInput, out numberOfYears))
{
    Console.WriteLine("Not a valid ammount, try again.");

    numberOfYearsInput = Console.ReadLine();
}

我已尝试将其更改为：

while (!Int32.TryParse(numberOfYearsInput, out numberOfYears))
{
    Console.WriteLine("Not a valid ammount, try again.");

    numberOfYearsInput = Console.ReadLine();

    if (numberOfYears < 1 || numberOfYears > 10)
    {
        Console.WriteLine("Not a valid ammount, try again.");

        numberOfYearsInput = Console.ReadLine();   
    }
}

但是，这会导致输入值不正确。我怎样才能把它结合起来让它工作？

编辑

已更改为以下，仍然允许 11、12 等作为有效输入

Console.WriteLine("Enter the number of years:");
string numberOfYearsInput = Console.ReadLine();

while (!Int32.TryParse(numberOfYearsInput, out numberOfYears))
{
    Console.WriteLine("Not a valid ammount, try again.");

    numberOfYearsInput = Console.ReadLine();

    if (numberOfYears >= 1 && numberOfYears <= 10)
    {
         Console.WriteLine("Not a valid ammount, try again.");

         numberOfYearsInput = Console.ReadLine();
    }
}

Answer 1

我认为这种任务应该由 oop 标准来完成，例如

    public static bool BetweenRanges(int a,int b,int number)
    {
        return (a <= number && number <= b);
    }
    static void Main(string[] args)
    {

        Console.WriteLine("Enter the number of years:");
        int numberOfYearsInput = Convert.ToInt16(Console.ReadLine());

        while (!BetweenRanges(1,10, numberOfYearsInput))
        {
            Console.WriteLine("Not a valid ammount, try again.");
            numberOfYearsInput = Convert.ToInt16(Console.ReadLine());
        }
    }

在这种情况下，您可以在更多地方使用BetweenRange，并使用任何类型的数字。代码更干净，更易于理解，在这种情况下可能并不重要，但它是最佳实践。 （当你的代码增长时会很容易理解）

如果是文本输入，您可以检查它并告诉用户不允许输入文本：

    public static bool BetweenRanges(int a, int b, int number)
    {
        return (a <= number && number <= b);
    }
    static void Main(string[] args)
    {
        bool notValid = true;

        while (notValid)
        {
            try
            {
                Console.WriteLine("Enter the number of years:");
                int numberOfYearsInput = Convert.ToInt16(Console.ReadLine());

                while (!BetweenRanges(1, 10, numberOfYearsInput))
                {
                    Console.WriteLine("Not a valid ammount, try again.");
                    numberOfYearsInput = Convert.ToInt16(Console.ReadLine());
                }

                notValid = false;
            }
            catch
            {
                Console.WriteLine("Text is not valid ammount, try again.\n");
            }
        }
    }

Answer 2

在 if 语句中，您使用“||”，但这是一个布尔 OR，您需要使用“AND”语句。此外，您的较小/较大运算符是倒置的，因此给您以下语句：

如果 numberOfYears 小于 1 或 numberOfYears 大于 10

如果你想检查数字是否在 1 到 10 之间，该行应该是这样的：

if (numberOfYears >= 1 && numberOfYears <= 10)

'&&' 是布尔值'AND' 运算符 '>=' 和 '<=' 分别大于/小于或等于。

因此它检查 numberOfYears 是否大于或等于 1 并且小于或等于 10。

Answer 3

试试这个

        int numberOfYears;
        Console.WriteLine("Enter the number of years:");
        string numberOfYearsInput = Console.ReadLine();
        bool result = false;

        while (!result)
        {
            if (Int32.TryParse(numberOfYearsInput, out numberOfYears))
            {
                if (numberOfYears < 1 || numberOfYears > 10)
                {
                    Console.WriteLine("Not a valid ammount, try again.");

                    numberOfYearsInput = Console.ReadLine();
                }
                else
                {
                    result = true;
                    Console.WriteLine("Correct Number");
                    Console.ReadKey(true);
                    //Do your code here
                }
            }
            else
            {
                Console.WriteLine("Not a valid number, try again.");
                numberOfYearsInput = Console.ReadLine();
            }
        }

Answer 4

问题出在这行代码while (!Int32.TryParse(numberOfYearsInput, out numberOfYears))，因为它传递了所有有效数字并仅在输入不是有效数字时才进入循环。

尝试使用这段代码，让我知道它是否适合你

       int numberOfYears;

        while (true)
        {
            Console.WriteLine("Not a valid ammount, try again.");
            if (!Int32.TryParse(Console.ReadLine(), out numberOfYears))
                continue;

            if (numberOfYears >= 1 && numberOfYears <= 10)
            {
                Console.WriteLine("Valid value");
                break;
            }
        }

Answer 5

它允许错误输入的原因是因为您使用的是 OR 操作数 (||)。

这基本上告诉编译器年份需要高于 1 OR 低于 10。

所以如果你添加一个数字，比如 100，它实际上被评估为真，因为它在 if 语句条件的左侧成立。 （大于 1）。

所以改成：

public static void Main()
{
    int numberOfYears = GetYears();

    Console.WriteLine("Good work, years = " + numberOfYears);
}

public static int GetYears()
{
    Console.WriteLine("Enter the number of years:");

    string inputYears = Console.ReadLine();
    int years;

    while(!(Int32.TryParse(inputYears, out years)) 
          || (years < 1  || years > 10))               
    {
        Console.WriteLine("Not a valid ammount, higher than 1, lower than 10!"); 
        Console.WriteLine("Please enter a new number");
        inputYears = Console.ReadLine(); 
    } 

    return years;
}

这是一个小提琴，其中包含 cmets 中所要求的问题的解决方案：Fiddle