测试用户输入是否在 1-100 之间不起作用？

Question

我正在调试一个介于 1-100 之间的“猜数字”程序。第一个用户创建一个 secretNumber，第二个用户猜测这个数字。

我正在尝试检查第二个用户输入是否是 1-100 之间的数字，但由于某种原因它仍然接受负数，我不知道为什么。

请注意，我对编码很陌生，所以我还不熟悉行话。在我的课上，我们刚刚学习了 while 循环和关系运算符以及 if/else 语句，所以我只能使用这些。

(secretNumber == 0) 是为了确保int.TryParse() 在用户输入字母而不是数字时不会返回 0。

if 语句检查：

( secretNumber >= 1 && secretNumber <= 100 )

还有

( secretNumber >= 1 || secretNumber <= 100 )

这是我的代码：

//Create a number variable to hold the whole number
int secretNumber;

//Validate that the number is a whole number AND between 1 and 100
if ( int.TryParse(secretNumberString, out secretNumber)
  || ( secretNumber >= 1 && secretNumber <= 100 )
  || secretNumber == 0 )
{

  // Tell the user the problem
  Console.WriteLine("Sorry, please only type in whole numbers and it must be between 1 and 100!");

  // Re-ask the question
  Console.WriteLine("What is your secret number?");

  // Re-catch the response in the same variable
  secretNumberString = Console.ReadLine();

  // Re-assign the new input
  int.TryParse(secretNumberString, out secretNumber);

} 

如果用户输入负值，则 if 语句应该运行。

如果用户输入的值大于 100，它就会运行，但如果值低于 100，它就不会运行。否则，如果数字在 1-100 之间正确，它就不应该运行。

Answer 1

如果输入为非数字或超出 1~100 范围，则该语句必须为真。对吗？

//if string is NOT integral (notice the !)   Or less than 1      Or bigger than 100
if (!int.TryParse(str, out int secretNumber) || secretNumber < 1 || secretNumber > 100)

此外，由于未经验证，用户在第二次尝试时仍可以输入非整数、超出范围的数字。理想情况下，我认为最好将验证逻辑作为循环变量：

string input;
int secretNumber;
do
{
    Console.WriteLine("Input an integer between 1 and 100, inclusive.");
    input = Console.ReadLine();
}
while (!int.TryParse(input, out secretNumber) || secretNumber < 1 || secretNumber > 100);

Answer 2

问题是你的||（或），这只是一个简单的布尔代数（命题逻辑）错误，它必须是&&（和）

然而，一个类似的更简洁的模式是一个精简的验证循环

var num = 0;

Console.WriteLine("What is your secret number between 1 and 100 inclusively?");
var str = Console.ReadLine();

// while its not a number, "or" the number is less or equal to 0 "or" greater than 100
// annoy them with a message 
while (!int.TryParse(str, out num ) || num  <= 0 || num  > 100)
{
   Console.WriteLine("Omg you had one job, a number between 1 and 100 inclusively... Try again");
   str= Console.ReadLine()
}

Console.WriteLine($"You won, your number is {num } ");

Answer 3

您可能想阅读int.TryParse() 的文档——它返回true 如果字符串被成功解析为整数，否则返回false。你的测试，int.TryParse( secretNumberString, out secretNumber ) || ( secretNumber >= 1 && secretNumber <= 100 ) || secretNumber == 0 ) 只会在解析失败时测试secretNumber 的值（此时它的值将是0）。

这可能会让你开始：

static int GetAnswer( string prompt, int min , int max )
{
  if (min > max) throw new ArgumentException("min must be less than or equal to max");
  int    value   ;
  bool   valid = false ;

  do
  {
    Console.WriteLine(prompt);
    string answer  = Console.ReadLine().Trim() ;

    valid =  int.TryParse(answer, out value)
          && value >= min
          && value <= max
          ;

    if (!valid)
    {
      Console.WriteLine($"Sorry, please only type in whole numbers in the range {min}-{max}.");
    }

  } while (!valid );

  return value;
}