我正在尝试编写为彩票生成三位数的程序,用户输入三位数。
但我没有得到任何正确的结果。
h1=rand() % 10;
int h2=rand() % 10;
int h3=rand() % 10;
cout<<"Enter three digite number\n";
cin>>n1>>n2>>n3;
if(n1==h1&&n2==h2&&n3==h3)
cout<<"you win 10.000 award\n";
else
if(n1==h1||n1==h2||n1==h3&&n2==h1||n2==h2||n2==h3&&n3==h1||n3==h2||n3==h3)
cout<<"you win 3,000\n";
else
if(n1==h1||n1==h2||n1==h3||n2==h1||n2==h2||n2==h3||n3==h1||n3==h2||n3==h3)
cout<<"you win 1,000\n";
else
cout<<"you don't win anything sorry\n";
答案 0 :(得分:1)
第二个和第三个if块中的测试条件不正确。
if(n1==h1||n1==h2||n1==h3&&n2==h1||n2==h2||n2==h3&&n3==h1||n3==h2||n3==h3)
在这种测试条件下,存在许多问题。
首先,这很容易被短路。如果n1与h1或h2匹配,则其余条件不会被测试。这同样适用于每次比较之后,请记住&amp;&amp;其中两个条件都需要在短路时才是真的。
假设我们使用括号来阻止这种情况,那么您的条件将适用于任何顺序的所有三个匹配。
看到这两个问题,最好重新修改逻辑,因为这会使第三个块的操作变得多余。您的第三个区块将针对特定的输入组合运行,而不是您期望的那样
我认为编写程序逻辑的更好方法是单独比较每个字符,并为找到的匹配数保持一个计数器。
int c=0, win;
if(n1==h1||n1==h2||n1==h3) c++;
if(n2==h1||n2==h2||n2==h3) c++;
if(n3==h1||n3==h2||n3==h3) c++;
if(c==3)
{
//Check order of digits
if(n1==h1&&n2==h2&&n3==h3) win = 10000;
else win=3000 //All digits same but not in order
}
else if(c==2)
win=3000
else if(c==1)
win=1000
else
win=0
if(win) cout<<"You win "<<win<<"\n";
else cout<<"you dont win anything\n";