c言語 配列に数字を代入する

Question

前提・実現したいこと

c言語で配列　number[10][2]にnumber[n][0]には０～２の数字をnumber[n][1]には０～９の数字をnumber[n][0]とnumber[n][1]の組み合わせがすべて異なるように格納したいです。
(nは、0<=n<=9の整数)

発生している問題・エラーメッセージ

下記の様にプログラムを組みましたが動きません。おかしい所やプログラムとして汚いところ詳しく解説していただけるとありがたいです。

該当のソースコード

int main( void )
{
	int i,j,k;
	int count=0;
	int retry;
	int number[10][2];
	srand(time(NULL));
	while(count<10){
		retry=0;
		do{
			i=rand()%3;
			j=rand()%10;
			number[count][0]=i;
			number[count][1]=j;
			for(k=0;k<count;k++){
				if(number[count][0]==number[k][0]&&number[count][1]==number[k][1])
					retry=1;
			}
		}while(retry==1);
		count++;
	}
	for(i=0;i<10;i++){
		printf("%d  %d\n",number[i][0],number[i][1]);
	}
	return 0;
}

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[jimbe]

>動きません
とはどういうことでしょう.
何も表示もされないのでしょうか.

Answer 1

0～2 と 0～9 の組み合わせというのは, つまり 0～29 の値ということです.
ひとまず 2 次元配列というのを(面倒なので)無視すれば, 0～29 の 30 個の数値の中から 10 個出すということになります.

#include <stdio.h>
void suffle(int a[],int size) {
  int i, j, t;
  srand(time(NULL));
  for(i=size-1; i>0; i--){
    j = rand() % (i+1);
    t = a[i]; a[i] = a[j]; a[j] = t; /* a[i]<->a[j] 交換*/
  }
}
#define COUNT 30
#define SIZE 10
int main(void) {
  int i, values[COUNT];
  for(i=0; i<COUNT; i++) values[i] = i; /*0～29*/
  suffle(values, COUNT); /*ランダムに並び替え*/
  for(i=0; i<SIZE; i++) printf("%02d\n", values[i]); /*10の位=0～2, 1の位=0～9*/
  return 0;
}

オリジナルに近くするとこんな感じ...ちょっと収まりが良くない感じがします.

#include <stdio.h>
#define COUNT 30
#define SIZE 10
int main(void){
  int i,j,v;
  int retry;
  int number[SIZE][2];
  srand(time(NULL));
  for(i=0;i<SIZE;i++){
    do{
      v=rand()%COUNT;
      for(j=0,retry=0;j<i&&!retry;j++) retry=(number[j][0]==v);
    }while(retry);
    number[i][0]=v;
  }
  for(i=0;i<SIZE;i++){
    number[i][1]=number[i][0]%10; number[i][0]/=10;
    printf("%d  %d\n",number[i][0],number[i][1]);
  }
  return 0;
}

Comments

[kelt22]

この発想はなかったです。ありがとうございます。

Answer 2

retry = 0; を do の中に書けばよいのではありませんか？

rand() を 10回しか呼び出さないようにすると、

#include <stdio.h>   // printf
#include <stdlib.h>  // srand, rand
#include <time.h>    // time

int main(void)
{
	#define T(x) x, x+1, x+2, x+3, x+4
	static int a[] = { T(0), T(5), T(10), T(15), T(20), T(25) };
	int number[10][2], n = 30;
    srand(time(0));
	for (int i = 0; i < 10; i++) {
		int j = rand() % n;
		printf("%d  %d\n", number[i][0] = a[j] / 10, number[i][1] = a[j] % 10);
		a[j] = a[--n];
	}
}

Answer 3

動かす方の回答はあるようなので・・・

number[count][0]=rand()%30;
として同じ数値を撥ねて、あとで、
number[count][1] = number[count][0] % 10;
number[count][0] /= 10;
とした方が、わかりやすくないですか？

配列の2次元目が2個だけなら、numberを div_t 型の構造体の１次元配列にしておいて、
そのまま div 関数の返却値を使うという手もありそうです。