我的“ while”陈述出了什么问题

Question

int monthentered = 0;
int dayentered = 0;
int year = 0000;
int [] month = new int [12];
int [] day = new int [31];
bool leap = false;

for (int x = 0; x <= 11; x++)
{
    month[x] = x+1;
}

for (int x = 0; x <= 30; x++)
{
    day[x] = x+1;
}
Console.WriteLine("Please enter a year...");
year = (Convert.ToInt16(Console.ReadLine()));
Console.WriteLine("{0}", year);

Console.WriteLine("Please enter a month...");
monthentered = (Convert.ToInt16(Console.ReadLine()));

Console.WriteLine("Please enter a day...");
dayentered = (Convert.ToInt16(Console.ReadLine()));

while (monthentered == 01 || 03 || 05 || 07 || 08 || 10 || 12)
{
    while (dayentered == 31)
    {
        Console.WriteLine("There are only 30 days in this month please re-enter your day...");
        dayentered = (Convert.ToInt16(Console.ReadLine()));
    }
}

while (monthentered == 02)
{
    while (dayentered > 28)
    {
        Console.WriteLine("There are only 28 days in this month please re-enter your day...");
        dayentered = (Convert.ToInt16(Console.ReadLine()));
    }
}

Console.WriteLine("{0}/{1}/{2}", dayentered, monthentered, year);
Console.ReadKey();

while (monthentered == 01 || 03 || 05 || 07 || 08 || 10 || 12)时行有什么问题吗？ 我收到一个我不明白的错误。 "Operator '||' cannot be applied to operands of type 'bool' and 'int'"

请帮忙。

Answer 1

while(monthentered == 01 || monthentered == 03 || monthentered == 05 || monthentered == 07 || monthentered == 08 || monthentered == 10 || monthentered == 12)

||之间的每个操作 和&&必须渲染为布尔值（即完整测试），因为每个操作都是独立计算的

Answer 2

现在就像while(bool || int || int) ，我想您想要这样的东西：

while (monthentered == 1 || monthentered == 3 || monthentered == 5 /*|| ...*/)

您需要每次检查变量的值，因此它会是while(bool || bool || bool)等

您还可以创建有效月份的集合，并检查其中输入的数字。

Answer 3

运算符'||' 不能应用于'bool'和'int'类型的操作数

您不能使用|| 在数字上-您正在使用的每个部分|| 与应该评估为bool 。

条件应如下所示：

while (monthentered == 01 || 
       monthentered == 03 || 
       monthentered == 05 || 
       monthentered == 07 || 
       monthentered == 08 || 
       monthentered == 10 || 
       monthentered == 12)

尽管更易读的选项是：

var validMonths = new int[] { 1, 3, 5, 7, 8, 10, 12 };

while(validMonths.Contains(monthentered))

Answer 4

它应该是

while (monthentered  == 01 ||monthentered  == 03 || monthentered  ==05 || monthentered  ==07 || monthentered  ==08 || monthentered  ==10 || monthentered  ==12)

Answer 5

您不能将||的bool与int进行比较。 操作员 。 monthentered == 01是bool （true / false），其余均为ints 。

因此，将while更改为：

int[] allAllowedMonths = new[]{ 1, 3, 5, 7, 8, 10, 12 };
while (allAllowedMonths.Contains( monthentered ))
{
    // ...
}

Answer 6

C＃同时仅接受true / false表达式！

while (monthentered == 01 || monthentered ==03 || monthentered ==05 || monthentered ==07 || monthentered ==08 || monthentered ==10 || monthentered ==12) 

Answer 7

它应该是

while (monthentered == 01 || 
       monthentered == 03 || 
       monthentered == 05 || 
       monthentered == 07 || 
       monthentered == 08 || 
       monthentered == 10 || 
       monthentered == 12)

 {  
   //code here
 }

Answer 8

直接答案会导致代码可编译但不起作用 ：您需要使用

(monthentered == 01 || monthentered == 03 || monthentered == 05 ||
 monthentered == 07 || monthentered == 08 || monthentered == 10 ||
 monthentered == 12)

进一步的问题：

• 您的代码在大多数月份中需要32天以上的时间。
• 您月份的while循环永远不会终止，因为您永远不会更改月份的值。 改用if语句。

额外提示：您可以使用DateTime.DaysInMonth来获取每个月和每年允许的最大数字，而不是手动检查月份数。 这也将适应leap年。

Answer 9

这是乏味的语法，但您需要将while语句编写为：

while (monthentered == 01 || monthentered == 02 || monthentered == 3 ... etc

您的语句将首先对布尔条件monthentered == 01进行求值，然后尝试对其他整数进行逻辑运算或得出结果，从而得出错误。

但是，我将看一下您要尝试执行的操作，看看在这里是否真的需要while循环。