C言語：簡単な構造体について

Question

前提・実現したいこと

C言語での構造体で、入力したデータ（男か女）に当てはまる時にそれを取り出し表示させたいです。

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

性別は？> 男
1 渡辺一夫 16 男 
0  0  
3 篠田浩二 17 男 
4 遠藤雄一 19 男 

性別は？> 女
0  0  
2 浅田裕子 20 女 
0  0  

のように、必要ないデータも0として表示されてしまいます。

該当のソースコード

#include<stdio.h>
#include<string.h>
struct student{
	int no;
	char name[20];
	int nenrei;
	char seibetu[20];
};

int main()
{
	int i, cnt = 0;
	char str[20];
	struct student kekka[256];
	struct student data[5] = {
		{1, "渡辺一夫", 16, "男"},
		{2, "浅田裕子", 20, "女"},
		{3, "篠田浩二", 17, "男"},
		{4, "遠藤雄一", 19, "男"},
		{5, "堀田美香", 18, "女"}
	};
	printf("性別は？> ");
	gets(str);
	
	for(i=0; i<5; i++){
		if(! strcmp(str, data[i].seibetu)){
			kekka[i] = data[i];
			cnt++;
		}
	}
	for(i=0; i<=cnt; i++){
		printf("%d ", kekka[i].no);
		printf("%s ", kekka[i].name);
		printf("%d ", kekka[i].nenrei);
		printf("%s \n", kekka[i].seibetu);
	}
	return 0;
}

試したこと

どうしてこうなるのかはなんとなくは分かりますが、どう対応したらいいのか分かりません。

よろしくお願いします。

Answer 1

ぱっと見これで動きそうですが未検証です。

修正１

kekka[cnt] = data[i];

修正２

for(i=0; i<cnt; i++){

Comments

[YOshim]

これだけでOKですね。プログラムって面白い。

Answer 2

2か所ほど間違いがあります。

1か所は、dice142さんが指摘している箇所です。
残りの箇所は、結果を表示する為のfor文です。

コメントへの回答

悪い方へ修正しているようですね。
他回答者が、すでに回答していますので
デバッグする方法を紹介します。

kekka配列の期待値があると思います。
その期待値になるために、どのような添え字で
kekka配列に設定するかと思いますので

iとcntの値を確認すれば、kekka[i]が不向きであることが
わかると思います。

    for(i=0; i<5; i++){
        if(! strcmp(str, data[i].seibetu)){
            printf("i: %d cnt:%d\n", i, cnt);
            kekka[i] = data[i];
            cnt++;
        }
    }

printfを入れてデバッグすれば良い。。。

Comments

[yutarokon]

ありがとうございます。
for(i=0; i<5; i++){
for(j=i+1; j<5; j++){
if(! strcmp(str, data[i].seibetu)){
kekka[j] = data[i];
cnt++;
}
}
}
for(j=1; j<cnt; j++){
printf("%d ", kekka[j].no);
printf("%s ", kekka[j].name);
printf("%d ", kekka[j].nenrei);
printf("%s \n", kekka[j].seibetu);
}
return 0;
このようにしてみたのですがダメみたいです...

Answer 3

    for(i=0; i<5; i++){
        if(! strcmp(str, data[i].seibetu)){
            kekka[i] = data[i];
            cnt++;
        }
    }

この部分で、dataの要素番号と同じ番号のkekkaに入れているのが原因です。
dataはそれぞれ見ていくのに対し、kekkaは一致したときだけ要素番号が進む必要があるので
同じ変数iを使わずに、別々の変数を使う必要があります。

---

[追記]

** data[] **

要素番号	no	name	nenrei	seibetu
0	1	渡辺一夫	16	男
1	2	浅田裕子	20	女
2	3	篠田浩二	17	男
3	4	遠藤雄一	19	男
4	5	堀田美香	18	女

↑このようにあるものから

** kekka[] **

要素番号	no	name	nenrei	seibetu
0	1	渡辺一夫	16	男
1	3	篠田浩二	17	男
2	4	遠藤雄一	19	男

このように別な配列に格納されますが、
2つの配列の要素番号は一致しているものもあれば一致していないものもあります。
適当に並び替えたデータで考えると、data配列とkekka配列の各要素の番号に
数列的な規則性はありません。
つまり、不規則に格納されるkekka配列の要素番号に
for文のカウンタ変数は使うべきではありません。

kekka配列は条件に合うときだけ要素が追加されるもので、
そのときにkekka配列の次の要素を指す要素番号を変えていきます。
その値を使用すれば良いのです。

Comments

[yutarokon]

回答ありがとうございます。
しばらく考えてみたのですが...分かりませんでした。
もう少し詳しく教えていただいてもよろしいでしょうか？

[dice142]

回答に追記しました。