0

配列に繰り返されないインデックス (0-51) を生成し、いくつかの整数を格納しようとしています。しかし、出力は奇妙に思えます。以下は私のコードです：

#include <iostream>     // std::cout
#include <algorithm>    // std::random_shuffle
#include <vector>       // std::vector
#include <ctime>        // std::time
#include <cstdlib>      // std::rand, std::srand

// random generator function:
int myrandom (int i)
{
    return std::rand()%i;
}

int main ()
{

  int* temp = new int[52];

  std::srand ( unsigned ( std::time(0) ) );
  std::vector<int> myvector;

  // set some values:
  for (int i=0; i<52; ++i) myvector.push_back(i);

  // using built-in random generator:
  std::random_shuffle ( myvector.begin(), myvector.end() );

  // using myrandom:
  std::random_shuffle ( myvector.begin(), myvector.end(), myrandom);

  std::vector<int>::iterator it=myvector.begin();

  for(int i = 1;i<=24;i++)
  {
      temp[*it] = i;
      it++;

  }

  for(int i = 0;i<52;i++)
  {
      std::cout << temp[i];
      std::cout << '\n';
  }

  delete[]temp;
  temp = NULL;



  return 0;
}

ここで何がうまくいかないのですか？

出力には、1998373 のような非常に膨大な数が含まれています。

4

1 に答える 1

2

new int[52]初期化されていないメモリを返します。これらの値のうち 24 個のみを設定していて、残りは初期化されていません。多分これ：

for(int i = 1;i<=24;i++)
{
  temp[*it] = i;
  it++;
}

するつもりだった

for(int i = 1;i<52;i++)
{
  temp[*it] = i;
  it++;
}

? 0または、割り当て後にすべての値を明示的に設定します。

于 2013-10-12T20:17:54.333 に答える