配列の数を順番に表示していく際に、最後に表示された数だけ","をつけないようにしたい

Question

思った通りに実行することはできましたが、もっと簡単に書くことはできないのでしょうか。
最初の数を表示して、それ以降を,%dと置いていて、for文が2個ずつに分かれてしまいコードが長く感じてしまいます。

実行例↓
Input Data No. 1 : -1
Input Data No. 2 : 1
Input Data No. 3 : 2
Input Data No. 4 : 4
Input Data No. 5 : 4
Input Data No. 6 : 7
Input Data No. 7 : 7
Input Data No. 8 : 10
Input Data No. 9 : 10
Input Data No.10 : 15

Input a search value: 7

<< Results >>
search value: 7
data smaller than search value: -1,1,2,4,4
searched data: 7,7
data larger than search value: 10,10,15

#include <stdio.h>
#define SIZE 10
int main(void){
  int i,i1,i2,i3,search;
  int array[SIZE];

  for(i = 0;i<SIZE;i++){
    printf("Input Data No.%2d : ",i+1);
    scanf("%d",&array[i]);**ボールドテキスト**
  }
  printf("\n");
  printf("Input a search value: ");
  scanf("%d",&search);
  printf("\n");

  printf("<< Results >>\n");
  printf("search value: %d\n",search);
  printf("data smaller than search value: ");
  for(i = 0;i<SIZE;i++){
    if(search > array[i]){
      printf("%d",array[i]);
      break;
    }
  }
    i1 = i+1;
    for(i = i1;i<SIZE;i++){
      if(search > array[i]){
      printf(",%d",array[i]);
      }
    }
    printf("\n");

  printf("searched data: ");
  for(i = 0;i<SIZE;i++){
    if(search == array[i]){
      printf("%d",array[i]);
      break;
    }
  }
    i2 = i+1;
    for(i = i2;i<SIZE;i++){
      if(search == array[i]){
      printf(",%d",array[i]);
      }
    }
   printf("\n");

  printf("data larger than search value: ");
  for(i = 0;i<SIZE;i++){
    if(search < array[i]){
      printf("%d",array[i]);
      break;
    }
  }
    i3 = i+1;
    for(i = i3;i<SIZE;i++){
      if(search < array[i]){
      printf(",%d",array[i]);
      }
    }
   printf("\n");
  return 0;
}

Answer 1

試してないですが、

for(int i=0, j=1; i<SIZE; i++){ //j の 1 はセパレータの文字(バイト)数
  if(search > array[i]){
    printf(",%d"+j, array[i]);
    j = 0;
  }
}

は如何でしょう。

i と j のように同じ型であれば for の初期化に書けますので、ローカルな変数のスコープをループ外まで広げなくて良くなります。

Comments

[jimbe]

かぶってしまいました(^_^;

[melian]

なるほど！

[bafubafu_]

めっちゃスッキリしました！

Answer 2

こんなのはいかがでしょうか？

#include <stdio.h>

#define SIZE 10

void print(int search, int array[], int comp(int, int), const char *msg)
{
	printf("%s", msg);
	for (int j = 1, i = 0; i < SIZE; i++)
		if (comp(array[i], search)) printf(",%d" + j, array[i]), j = 0;
	putchar('\n');
}

int gt(int a, int b) { return a > b; }
int eq(int a, int b) { return a == b; }
int lt(int a, int b) { return a < b; }

int main(void)
{
	int search, array[SIZE];

	for (int i = 0; i < SIZE; i++) {
		printf("Input Data No.%2d : ", i + 1);
		scanf("%d", &array[i]);
	}
	printf("\n"
	       "Input a search value: ");
	scanf("%d", &search);
	printf("\n"
		   "<< Results >>\n"
		   "search value: %d\n", search);
	print(search, array, lt, "data smaller than search value: ");
	print(search, array, eq, "searched data: ");
	print(search, array, gt, "data larger than search value: ");
}

Comments

[bafubafu_]

行数が半分になっててめちゃくちゃスッキリしてますね...
まだまだわからないことがいっぱいなので勉強します...

Answer 3

char *sep;
printf("data smaller than search value: ");
sep = "";
for (i = 0; i < SIZE; i++) {
  if (search > array[i]) {
    printf("%s%d", sep, array[i]);
    sep = ",";
  }
}
printf("\n");

とか。

Comments

[bafubafu_]

凄くスッキリしました！