主页 / 问题 / 1226152
Accepted
aclui dll
Asked:2022-01-01 07:40:05 +0000 UTC2022-01-01 07:40:05 +0000 UTC

代码不起作用，循环混乱

  • 772

我想编写一个程序，将 20 个数字随机化并将它们添加到一个数组中（每个数组单元一个数字），但该数组不应包含重复的数字。结果，我想查看 20 个不同的数字，但没有任何反应。在我看来，问题出在第 11 行之后。我检查了 rand 和 srand 函数，它们与 c++ 标准库一起使用。很可能代码对您来说似乎很糟糕，但如果您能帮助我，我将不胜感激。以这种方式解决了问题

using namespace std;
int main ()
{
  int SIZE = 20;
  int arr[SIZE];
  srand (time (NULL));
  bool value;
  int i, j;
  value = true;
  for (i = 0; i < SIZE; i++)
    {
      while (value = true)
    {
      arr[i] = rand () % 20;
      if (i > 0)
        {
          for (j = i; j >= 0; j--)
        {
          if (arr[i] = arr[j])
            {
              value = true;
            }
        }
        }
    }

      cout << arr[i] << "\n";
    }
  return 0;
}
c++

1 个回答

  1. Best Answer

    嵌套循环是不可取的。这是一种算法，其中数字在一个循环中简单地打乱 - 不需要唯一性比较。代码看起来更短更清晰；

    #include <iostream>
using namespace std;

#define SIZE 20    

int main () {
    int i, arr[SIZE];
    srand (time (NULL));

    for (i = 0; i < SIZE; i++) arr[i] = i;  // [0...SIZE-1]

    for (i = 0; i < SIZE; i++){
        int t = arr[0];          // временно сохраняем число arr[0]
        int idx = rand() % SIZE; // генерим случайный индекс
        arr[0] = arr[idx];       // из arr[idx] число помещаем в arr[0]
        arr[idx] = t;            // в arr[idx] число из [0]
    }

    for (i = 0; i < SIZE; i++) cout << arr[i] << endl;     

    return 0;
}
    • 3

相关问题