C++Srand值的问题

Question

所以我用随机值创建了一张彩票，然后对它进行排序。我不担心排序技术，因为老师在这项作业中没有寻找类等，但它很有效，我生成的票证值-尽管使用srand（time（0）），然后使用rand（）%40+1-我认为这应该使我的randoms在1-40之间…但mainTicket[0]总是等于0。有什么想法吗？很抱歉格式化，让我添加了额外的空格，并弄乱了缩进。

#include <iostream>
#include <cstdlib>
#include <time.h>
using namespace std;
void mainLotto (int main[]);
void lottoSort (int ticketArr[]);
int main()
{
int mainTicket[5];
srand (time(0));
mainLotto(mainTicket);
do
{
    lottoSort(mainTicket);
} while (mainTicket[0] > mainTicket[1] || mainTicket[1] > mainTicket[2] || mainTicket[2] > mainTicket[3] || mainTicket[3] > mainTicket[4] || mainTicket[4] > mainTicket[5]);
for (int i = 0; i < 5; i++)
{
    cout << mainTicket[i] << "nn";
}
return 0;
}
///
/// <> Creates the actual lottery ticket
///
void mainLotto (int main[])
{
// Creating the ticket
for (int i = 0; i < 5; i++)
{
    main[i] = rand() % 40 + 1;
}
}
///
/// <> Sorts the actual lottery ticket
///
void lottoSort (int ticketArr[])
{
 // Sorting the ticket
for (int j = 0; j < 5; j++)
{
    if (ticketArr[j] > ticketArr[j+1])
    {
        int temp;
        temp = ticketArr[j+1];
        ticketArr[j+1] = ticketArr[j];
        ticketArr[j] = temp;
    }
}
}

Answer 1

我看到您的数组被越界访问时出现了两个问题：

此处：

int main()
{
    int mainTicket[5];
    srand(time(0));
    mainLotto(mainTicket);
    do
    {
        lottoSort(mainTicket);
    }
    while(mainTicket[0] > mainTicket[1] || mainTicket[1] > mainTicket[2]
        || mainTicket[2] > mainTicket[3] || mainTicket[3] > mainTicket[4]);
//      || mainTicket[4] > mainTicket[5]); // OUT OF BOUNDS!!!
    for(int i = 0; i < 5; i++)
    {
        cout << mainTicket[i] << "nn";
    }
    return 0;
}

此处：

void lottoSort(int ticketArr[])
{
    // Sorting the ticket
    for(int j = 0; j < 4; j++) // j < 4 NOT 5!!! <== WAS OUT OF BOUNDS
    {
        if(ticketArr[j] > ticketArr[j + 1])
        {
            int temp;
            temp = ticketArr[j + 1];
            ticketArr[j + 1] = ticketArr[j];
            ticketArr[j] = temp;
        }
    }
}

排序例程很可能是从数组边界外拖动了一个零。

Answer 2

我刚刚打印了您声称始终为零的项目，在mainLotto()之后，它产生了30。

我怀疑问题出在你告诉我们不要看的地方

在排序功能中：

for (int j = 0; j < 5; j++) {
    if (ticketArr[j] > ticketArr[j + 1]) {

数组大小为5。您的j最终将取一个等于4的值。

然后执行ticketArr[j + 1]，这实际上是一个越界访问。

解决方法是循环到4，而不是5。

正如Galik所说，mainTicket[4] > mainTicket[5]也是一种越界访问，在阅读了我的答案后，你应该能够理解为什么。：）