初心者です。お助けください。

質問

とりあえず関数を説明します。
NumberCOM：0475で初期化した重複のない３０個の乱数をとり配列COM[30]に格納した後、表示する。
NumberMAN：適当に６個の整数をとって、配列MAN[6]に格納した後、表示する。
Sort：配列COMとMANの要素をそれぞれ小さい順にならべて、表示する。
（但し、下にやったように配列Work[100]を設けて考えないといけないという縛りつき）

実行したら
NumberCOMとNumberMANはうまく実行できるんですが、
配列COMのSortがうまくいきません。

Sortした後のCOM配列の要素数がNumberCOM関数で作ったCOM配列の要素よりも少なく、重複して表示されてしまいます。

どうしたらうまくSortできるのでしょうか。見にくくて申し訳ありませんが、お教えください。

#include<stdio.h>
#include<stdlib.h>
#define N 30
#define n 6

int NumberCOM(int *com){
int a,b;
srand(0475u);
for(a=0;a<N;a++){
com[a]=(rand()/(RAND_MAX +1.0)*100);
for(b=0;b<a;b++){ /*重複を避ける*/
if(com[a]==com[b]){
com[b]=(rand()/(RAND_MAX +1.0)*100);
com[a]=com[b];
}
}
printf("%d,",com[a]);
}
printf("\n");
}

int NumberMAN(int *man){
int c;
man[0]=8;man[1]=70;man[2]=58;man[3]=89;man[4]=92;man[5]=23;
for(c=0;c<n;c++){
printf("%d,",man[c]);
}
printf("\n");
}

int Sort(int *a,int ko){
int d,e,f=0,Work[100]={0};
for(d=0;d<ko;d++){
Work[a[d]]=1;
}
for(e=0;e<100;e++){
if(Work[e]==1){
a[f]=e;
f=f+1;
}
}
for(f=0;f<ko;f++){
printf("%d,",a[f]);
}

printf("\n");
}

int main(void){
int COM[N],MAN[n];
NumberCOM(COM);
NumberMAN(MAN);
Sort(COM,N);
Sort(MAN,n);

return 0;
}

回答No.2

Oh-Orange · ベストアンサー

★重複を避ける部分がおかしいです。
・重複を避ける処理を入れているのにその中で発生させた乱数は重複のチェックを
　行っていません。また、その乱数を com[a]、com[b] の２つに代入しています。
　この時点で重複データを避ける処理になっていません。無意味でしょうが…。
・私なら下のようにしますけど。参考に。

サンプル:
// 重複チェック用
int DoubleCheck( int *com, int size, int check )
{
　int i;

　for ( i = 0 ; i < size ; i++ ){
　　if ( com[i] == check ){
　　　return 1;　// 重複あり
　　}
　}
　return 0;　// 重複なし
}

// 乱数
int NumberCOM( int *com, int size )
{
　int i;
　srand( 0475u );

　for ( i = 0 ; i < size ; i++ ){
　　do {
　　　com[ i ] = (rand() / (RAND_MAX + 1.0) * 100);
　　} while ( DoubleCheck(com,i,com[i]) );

　　printf( "%d,", com[i] );
　}
　printf( "\n" );
　return 0;　←これ忘れずに(戻り値が int で宣言しているから)
}

その他:
・上記のサンプルのように『重複チェック関数』を作った方が処理がわかりやすくなると思います。
　重複の処理は NumberCOM() 関数の do-while 文で行っています。
　重複があると DoubleCheck() 関数で 1 が返されるため do-while 文を繰り返して新しい乱数を
　com[i] にセットします。その後に重複しなくなるまで do-while 文を繰り返します。
・以上。参考に。

お礼

恥ずかしながら、関数に関して間違った解釈をしてました。
ご指摘ありがとうございます。
それにgood ideaまで出していただいて。

参考になりました。
これからは、複雑になったら分けるというのを基本にします。

回答No.3

yaemon_2006

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

int main(void)
{
int work[100];
int i, j, temp;

srand((unsigned)time(NULL));
for(i = 0; i < 100; i ++) work[i] = i;
for(i = 99; i > 1; i --){
j = ((double)rand() / RAND_MAX) * i ;
temp = work[j];
work[j] = work[i];
work[i] = temp;
}
for(i = 0; i < 100; i ++) printf("%d ", work[i]);
return 0;
}

回答No.1

επιστημη（@episteme）

NumberCOMが生成した値に重複があります。