C言語の配列と関して

Question

整数型の配列変数dataを作成し、キーボードから入力させ、値を表示させてから3の倍数とそれ以外の数にわけて表示するプログラムです。
思うように動きません。

#include<stdio.h>
int main()
{
	int i, data[10], b[10], sum = 0, sum2 = 0;
	double a[10];
	printf("1から10までの整数を10個入力してください。\n");
	//数値の入力
	for(i=0; i<10; i++){
		scanf("%d", &data[i]);
	}
	
	//配列の表示
	for(i=0; i<10; i++){
		printf("%d ", data[i]);
	}
	printf("\n");
	
	//3の倍数かどうか
	for(i=0; i<10; i++){
		if(data[i]%3==0){
			a[i] = data[i];
			sum = sum + 1;
		}else b[i] = data[i];
		sum2 = sum2 + 1;
	}
	
	//3の倍数表示
	printf("3の倍数：");
	for(i=0; i<sum; i++){
		printf("%d ", a[i]);
	}
	printf("\n");
	
	//3の倍数以外表示
	printf("3の倍数以外：");
	for(i=0; i<sum2; i++){
		printf("%d ", b[i]);
	}
	printf("\n");
	
	return 0;
}

↓10個の整数を1,2,3,4,5,6,7,8,9,3としたときの結果です。

c:\prgc>kadai16

1から10までの整数を10個入力してください。
1
2
3
4
5
6
7
8
9
3
1 2 3 4 5 6 7 8 9 3
3の倍数：8 1997138198 0 6356644
3の倍数以外：1 2 1995729546 4 5 4199136 7 8 6356704 6356760

なぜでしょうか？

Answer 1

    printf("%d ", a[i]);

%d はint型のフォーマット文字ですねー

---

で、全部の変数がAUTO変数で初期化していないため、これらの変数はなにかの値を代入しない限り、でたらめな値になってます
３の倍数でないときのa[i]、３の倍数のときのb[i]、ともになにも代入されないため、でたらめな値になってます
ということで、該当する個所の出力がでたらめになる、ということですねー

Comments

[退会済みユーザー]

ありがとうございます！
これでおかしな数字がでないようになりました！

[y_waiwai]

んで、回答に触れられてないけど気になるところとして、

a[]はべつに int でいいぢゃん、

ってのもあります。
まあ、これはバグではないですがｗ

[退会済みユーザー]

そうでしたか、参考になります！

Answer 2

まず、ぱっと目に付いた点、

    //3の倍数かどうか
    for(i=0; i<10; i++){
        if(data[i]%3==0){
            a[i] = data[i];
            sum = sum + 1;
        }else b[i] = data[i];
        sum2 = sum2 + 1;
    }

sum2 = sum2 + 1

は、else に含まれません。

        }else {
          b[i] = data[i];
          sum2 = sum2 + 1;
        }

かなって気がします。　(これを忘れた結果は、環境依存)
括弧で括る習慣をつけた方が良いと思います。
その他はこれから。
それと、何が上手くいかないか具体的に書かれた方が、確認が早くなります。

-- 追記。
大物がいました。

a[i] = data[i];

ここ、 a[i]じゃなくて、 a[sum] でしょう。
同様に、

b[i] = data[i];

ここも b[sum2] ですね。

追加 (4/16):
他の方も指摘されてますが、念のため。

double a[10]

ここは、int a[10] ですね。そうしないと、 doubleのデータを intで出力しようとするので、不正な出力となります。

Comments

[退会済みユーザー]

迂闊でした...
ご指摘ありがとうございます！なおしてみます！

[退会済みユーザー]

ありがとうございます！
指摘の部分をなおしと、3の倍数はちゃんと3,6,9,3と出たんですが、
それ以外のものに1,3,6,9,3,8と出てしまいます...

[pepperleaf]

if 文の else 以降を {} で括っただしょうか?
あと、本文に追加

Answer 3

Cなど不要。そう、シェル芸ならね！

$ echo 1 2 3 4 5 6 7 8 9 10|xargs -n1|awk '{if($1%3){a=a" "$1}else{b=b" "$1}}END{print "3の倍数:"b"\n3の倍数以外:"a}'
3の倍数: 3 6 9
3の倍数以外: 1 2 4 5 7 8 10

Answer 4

こんにちは、
さっそく、気になった点を述べたいと思います。
1.配列の要素数の"10"は記号定数にすべきです。
2.入力はscanfより、fgetsをおすすめします。
3.ただの表示だけならば,わざわざ新たな配列に格納する必要はありません。
(あくまでも、表示だけならの話です。そのあと他の処理があるときは別の話です。)

以下、私が書いたソースを載せておきます。(大したことはしてません)

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

#define BUFSIZE 256
#define MAX_NUMBER 10

int main(void)
{
	char buf[BUFSIZE];
	int data[MAX_NUMBER];
	int i;
	
	for(i = 0; i < MAX_NUMBER; i++){
		fgets(buf,sizeof(buf),stdin);
		data[i] = atoi(buf);
	}
	
	printf("3の倍数:");
	for(i = 0; i < MAX_NUMBER; i++){
		if(data[i] % 3 == 0){
			printf("%d ",data[i]);
		}
	}
        printf("\n");
	printf("3の倍数以外");
	for(i = 0; i < MAX_NUMBER; i++){
		if(data[i] % 3 != 0){
			printf("%d ",data[i]);
		}
	}
	return 0;
}

追記

どこが原因かわかりました。とりあえず、以下のソースで試してください。

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

#define BUFSIZE 256
#define MAX_NUMBER 10

int main(void)
{
	char buf[BUFSIZE];
	int data[MAX_NUMBER];
	int i;
	int a[MAX_NUMBER];
	int b[MAX_NUMBER];
	int count = 0;
	int count2 = 0;
	
	for(i = 0; i < MAX_NUMBER; i++){
		fgets(buf,sizeof(buf),stdin);
		data[i] = atoi(buf);
	}
	
	for(i = 0; i < MAX_NUMBER; i++){
		if(data[i] % 3 == 0){
			a[count] = data[i];
			count++;
		}
		else{
			b[count2] = data[i];
			count2++;
		}
	}
	printf("3の倍数:");
	for(i = 0; i < count; i++){
		printf("%d ",a[i]);
	}
	printf("\n");
	printf("3の倍数以外:");
	for(i = 0; i < count2; i++){
		printf("%d ",b[i]);
	}
	
	return 0;
}

Comments

[退会済みユーザー]

好みによりますが, sum = sum + 1のところはsum++のほうが簡略化しています。

Answer 5

#include<stdio.h>
int main()
{
    int i, data[10];
    int a[10], b[10], sum = 0, sum2 = 0;

    printf("1から10までの整数を10個入力してください。\n");
    // 数値の入力
    for(i = 0; i < 10; i++){
      scanf("%d", &data[i]);
    }

    //配列の表示
    for(i = 0; i < 10; i++){
      printf("%d ", data[i]);
    }
    printf("\n");

    // 3の倍数かどうか
    for(i = 0; i < 10; i++){
      if (data[i] % 3 == 0){
        a[sum] = data[i];
        sum = sum + 1;
      }else{
        b[sum2] = data[i];
        sum2 = sum2 + 1;
      }
    }

    // 3の倍数表示
    printf("3の倍数：");
    for(i = 0; i < sum; i++){
      printf("%d ", a[i]);
    }
    printf("\n");

    // 3の倍数以外表示
    printf("3の倍数以外：");
    for(i = 0; i < sum2; i++){
      printf("%d ", b[i]);
    }
    printf("\n");

    return 0;
}

実行例

$ gcc x.c
$ ./a.out
1から10までの整数を10個入力してください。
1 2 3 4 5 6 7 8 9 10
1 2 3 4 5 6 7 8 9 10 
3の倍数：3 6 9 
3の倍数以外：1 2 4 5 7 8 10 

3の倍数かどうか の部分をみなおすとよいと思います。