为什么我的If和Else语句不起作用？(C++)

Question

我刚开始学C++,我试着做一个骰子游戏,用户输入一个1到6之间的数字,然后代码打印一个在这个范围内的随机数,如果y和z相同,你就赢了。

这是我的代码，但当我输入一个数组中没有的数字时，它的工作方式就好像它在数组中一样。

#include <iostream>
#include <stdlib.h>
#include <time.h>

int main() {
    for (; ; ) {
        int x[6] = { 1,2,3,4,5,6 }; //dice numbers
        int y;                      //user choice
        int z;                      // random number generated

        std::cout << "Pick a number between 1 and 6\n";
        std::cin >> y;

        if (y >= 1 || y <= 6) {     //if y is greater then or = to 1 or less then 
            std::cout << "Rolling dice, if your number is the correct number you win\n";
            srand(time(NULL));      //this makes rand() actually random.
            z = rand() % 6;
            std::cout << "The number is " << x[z] << "\n" << "The game will repeat..\n";
        }

        else {                      //if the num generated isn't within 1 or 6 this is printed
            std::cout << "ERROR: Generated number is not within 1 nor 6, try again.";

        }

        if (y == z) {
            std::cout << "Congratulations you rolled the right number";
        }
    }

（输入是y）（数组是x）（你需要赢的数字是z）

此外，我可能会更改它，使它只读取数组，这样用户甚至可以放入骰子的边数，如果这样做顺利的话。

Answer 1

我发现你的代码有五个问题。

首先，最好执行while(true)而不是for（；；）。

其次，您的else块应该从终止if块的同一行开始。

第三，您应该在赢得游戏时插入break语句。

第四，您应该将条件if(y>=1y<=6)更改为if(y>=1&&y<=6)。两者的区别在于是OR逻辑运算符。如果y大于或等于1，或者y小于或等于6，这是真的，这本质上对每个整数都起作用。999会通过，因为它大于或等于1，而-999会通过，因为它小于或等于6。而且1到6之间的任何数都可以通过，因为它们同时通过y>=1和y<=6。如果您要插入一个AND运算符&来代替OR运算符，那么条件将通过，因为只有当y介于1和6之间时，条件才会为真。

第五，您应该将条件y==z的if块移动到嵌套在y>=1&&y<=6中。下面的代码是我所做的每一个更改：

    while (true) { //1
        int x[6] = { 1,2,3,4,5,6 };
        int y;
        int z;

        std::cout << "Pick a number between 1 and 6\n";
        std::cin >> y;

        if (y >= 1 && y <= 6) { //4
            std::cout << "Rolling dice, if your number is the correct number you win\n";
            srand(time(NULL));
            z = rand() % 6;
            std::cout << "The number is " << x[z] << "\n" << "The game will repeat..\n";
            if (y == z) { //5
                std::cout << "Congratulations you rolled the right number";
                break; //3
            }
        } else { //2
            std::cout << "ERROR: Generated number is not within 1 nor 6, try again.";

        }
    }

Answer 2

此条件：

if (y >= 1 || y <= 6)

对于y的所有值将为true。每个整数要么大于等于1，要么小于等于6。

你需要一个连词，像这样：

if (y >= 1 && y <= 6)

您还需要在y==z时中断；，否则将导致无限循环。