c++ - 阵列循环无输出

我要从24个中抽出3个球，并加总这些球所代表的折扣。我不允许在抽奖后更换球，因此不能重复绘制球。折扣为1、2、3、4、5、6、7、8，每个都有3。

我对这个问题的解决方案是创建多个while循环，从24个循环开始，然后重新滚动，直到没有重复为止。过去我做过这样的循环，它将循环随机数，直到选择了一个非重复项，并且效果很好，所以我以相同的方式设置了该程序。

#include <iostream>
#include <cstdlib>
#include <ctime>

using namespace std;

int draw(int urn[], int draws[]);

int main()
{
    int discountSum,discount;
    int urn[24]={1,1,1,2,2,2,3,3,3,4,4,4,5,5,5,6,6,6,7,7,7,8,8,8};
    int draws[1000]={0};
    draw(urn,draws);
    cin >> discount;
    int averageDiscount=discount/1000;


    return 0;
}

int draw(int urn[], int draws[])
{
    static int discountSum=0;
    int i; //simulations counter
    int ball1, ball2, ball3;
    int score1, score2, score3;
    int totalDiscount;
    srand(time(0));

    for(i=0;i<1000;i++)
    {
        ball1=rand()%24;
        while (ball2==ball1)
        {
            ball2=rand()%24;
        }
        while ((ball3==ball2)||(ball3==ball1))
        {
            ball3=rand()%24;
        }
        score1=urn[ball1];
        score2=urn[ball2];
        score3=urn[ball3];
        totalDiscount=score1+score2+score3;
        draws[i]=totalDiscount;
        cout << "1) " << draws[i] << " percent discount.\n";
        discountSum+=totalDiscount;
    }
    return discountSum;
}

当我运行此代码时，程序不会输出错误，而不会收到错误，而是返回负垃圾值而不是0。我希望看到每次折扣直至第1000个循环的输出。

最佳答案

似乎错误在于ball2和ball3在被设置为任何东西之前已经被比较了。它们甚至从未被初始化，这可能是“垃圾价值”的原因。如果在比较之前“绘制”每个球，则可能会起作用，例如：

ball1=rand()%24;
ball2=rand()%24;
while (ball2==ball1)
{
    ball2=rand()%24;
}
ball3=rand()%24;
while ((ball3==ball2)||(ball3==ball1))
{
    ball3=rand()%24;
}

更好的方法是改为使用do ... while循环，该循环始终运行一次，如下所示：

ball1=rand()%24;
do
{
    ball2=rand()%24;
} while (ball2==ball1);
do
{
    ball3=rand()%24;
} while ((ball3==ball2)||(ball3==ball1));