1

我正在尝试制作一个生成随机数的程序，直到它找到一组预定义的数字（例如，如果我有一组我最喜欢的 5 个数字，我需要玩多少次才能让计算机随机找到相同的数字） . 我写了一个简单的程序，但不理解结果，这似乎与我的预期有点无关，例如结果不一定包含所有预定义的数字，有时它会包含（即使这样也不会停止循环从运行）。我认为问题出在逻辑运算符“&&”上，但不确定。这是代码：

const int one = 1;
const int two = 2;
const int three = 3;

使用命名空间标准；

int main()
{

    int first, second, third;
    int i = 0;

    time_t seconds;
    time(&seconds);

    srand ((unsigned int) seconds);

    do

    {


    first = rand() % 10 + 1;
    second = rand() % 10 + 1;
    third = rand() % 10 + 1;


    i++;

    cout << first<<","<<second<<","<<third<< endl;
    cout <<i<<endl;
    } while (first != one && second != two && third != three); 

    return 0;
 }

以下是可能的结果：

3,10,4
1 // itineration variable
7,10,4
2
4,4,6
3
3,5,6
4
7,1,8
5
5,4,2
6
2,5,7
7
2,4,7
8
8,4,9
9
7,4,4
10
8,6,5
11
3,2,7
12

我还注意到，如果我使用 || 运算符而不是 && 循环将执行，直到它找到符合变量设置顺序的确切数字（此处为：1、2、3）。这更好，但是即使顺序不同，只有数字，我该怎么做才能使循环停止？感谢您的回答和帮助。

4

2 回答 2

4

问题在于您的情况：

} while (first != one && second != two && third != three);

你继续，而他们都不是平等的。但是一旦其中至少一个相等，您就会停止/离开循环。

要解决此问题，请使用逻辑或 ( ||) 而不是逻辑和 ( &&) 来链接测试：

} while (first != one || second != two || third != three);

现在只要它们中的任何一个不匹配，它就会继续。

编辑 - 进行更高级的比较：

我将使用一个简单的宏来使其更易于阅读：

#define isoneof(x,a,b,c) ((x) == (a) || (x) == (b) || (x) == (c))

请注意，您可以使用不同的方法。

} while(!isoneof(first, one, two, three) || !isoneof(second, one, two, three) || !isoneof(third, one, two, three))
于 2012-08-28T10:16:55.403 回答
1

您的逻辑条件有误：这意味着“所有数字都不相等”。要打破这个条件，一对变得相等就足够了。

您需要构建一个不同的条件 - 要么在它前面加上“不”

!(first==one && second==two && third==three)

或使用德摩根定律转换：

first!=one || second!=two || third!=three
于 2012-08-28T10:17:15.043 回答