ランダムな整数の配列をつくりたい

Question

1から100までの整数をランダムで配列に指定数，入力していく関数を作成しているのですが，
以下のコードで試したところ100を超える数字が入力されてしまいます．この関数の不備と，また，同じ数字が必ず配列に含まれないようにするにはどうすればいいですか．

#include <stdio.h>
#include <stdlib.h>

void make_num(int ary[], int x, int max)
{
    int i, m;
    for (i = 1; i <= x; i++)
    {
        m = rand() % max;
        ary[i] = m;
        i++;
    }
}

int main(void)
{
    int i, a[100];
    make_num(a, 5, 100);
    for (i = 1; i <= 5; i++)
        printf("a[%d]=%d\n", i, a[i]);
}

Comments

[BeatStar]

試していませんが、値渡しなっているからでは?

[cosinfake]

a[1]=41
a[2]=13
a[3]=67
a[4]=7799484
a[5]=34
実行結果はこうなってしまいます．

Answer 1

    for (i = 1; i <= x; i++)
    {
        m = rand() % max;
        ary[i] = m;
        i++;
    }

i が二回 ++ されています。

Comments

[cosinfake]

ほんとですね，お恥ずかしいミスを．ご指摘ありがとうございます。

Answer 2

基本、episteme さんが書いてる
＞i が二回 ++ されています。
が全てだと思いますが、
乱数生成は、
a[100] 全部埋めるのもありだと思います。

#include <stdio.h>
#include <stdlib.h>
#include <time.h>
void different_Fisher_Yates_shuffle(int p[],size_t max)
{
     for (size_t i = 0; i < max; i++) {
	  size_t no = rand() % max;
	  int tmp = p[no];
	  p[no] = p[i];
	  p[i] = tmp;
     }			
}
void make_num(int ary[], int x, int max) {
    int i, m;
    for (i = 0; i < max; i++) {
        ary[i] = i;
    }
    different_Fisher_Yates_shuffle(ary, max);
// x は必要ないので未使用
}
int main(void)
{
     int i, a[100];
     srand(time(NULL));
     make_num(a, 5, 100);
     for (i = 1; i <= 5; i++)
	  printf("a[%d]=%d\n", i, a[i]);
}

Answer 3

同じ数字が必ず配列に含まれないようにするにはどうすればいいですか．

乱数を出して、今まで出した数値と比較していき、同じ数字があれば乱数を出し直してやり直せばいいです

Answer 4

同じ数字が必ず配列に含まれないようにするにはどうすればいいですか．

#include <stdio.h>
#include <stdlib.h>
#include <stdbool.h>

// ary[1～size]内に val があるなら true
bool contains(int ary[], int size, int val) {
  for ( int i = 1; i <= size; ++i ) {
    if ( ary[i] == val ) return true;
  }
  return false;
}

void make_num(int ary[], int x, int max) {
    int i, m;
    for (i = 1; i <= x; i++) {
        do {
            m = rand() % max;
        } while ( contains(ary,i-1,m)); // 重複したらもう一度
        ary[i] = m;
    }
}

int main(void)
{
    int i, a[100];
    make_num(a, 5, 100);
    for (i = 1; i <= 5; i++)
        printf("a[%d]=%d\n", i, a[i]);
}

Comments

[cosinfake]

サンプルを作成していただきありがとうございます．参考になりました．