C言語：配列を使ったループ出力で０を出さない方法

Question

初心者の学生です。どうぞ宜しくお願いいたします。

従業員情報を入力、表示する課題を解いているのですが、データが無い時も０が表示されてしまいます。

教授からは「SIZEを使うから4回繰り返すのは当然だ」と言われ、他の変数（numなど）を作ったり if文やwhileループと組み合わせてみたのですが上手くいかず...条件文をどう書いたらいいかわからず、質問させていただきました。

表示（case 1）を選択した場合、SIZE分（4回分）ループされてしまっているためこれ以上追加ができません。
入力（case 2）を選択した場合も「0 0 0.00」という表示が出てしまいます。

お知恵をお借りできれば幸いです。お時間ありがとうございます。

ソースコード

#include <stdio.h>
#define SIZE 4

struct Employee {
	int id;
	int age;
	double salary;
};

int main(void) {
	int option, i;

	struct Employee emp[SIZE] = { {0, 0, 0.00} };

	printf("---=== 従業員データ ===---\n\n");

	do {
		printf("1. 従業員情報表示\n");
		printf("2. 従業員追加\n");
		printf("0. 終了\n\n");
		printf("上からメニューを選んでください: ");

		scanf("%d",&option);
		printf("\n");
		
		switch (option) {
		case 0:
			break;

		case 1: // 従業員表示
			printf("従業員ID 年齢 給料\n======  ======= ==========\n");   
	        for(i = 0; i < SIZE; i++) {
                printf("%6d%9d%11.2lf\n", emp[i].id, emp[i].age, emp[i].salary);
	        }
		    printf("\n");
			break;

		case 2:	// 従業員追加
			printf("従業員追加\n===============\n");
            if(i >= SIZE) {
			    printf("\n");
			    printf("ERROR: 登録できる最大数に達しました\n\n");
			    break;
			}
			printf("従業員IDを入力してください: "); scanf("%d", &emp[i].id);
			printf("年齢を入力してください: "); scanf("%d", &emp[i].age);
			printf("給料を入力してください: "); scanf("%lf", &emp[i].salary);
            printf("\n");
            i++;
			break;

		default:
			printf("ERROR: 予期せぬオプションです: もう一度\n\n");
		}

	} while (option != 0);

    printf("プログラムを終了します。\n");
	return 0; 
}

出力

試したこと

• forループ内の継続条件式用に別の変数（num）を作りインクリメントを試みた ⇒ 上手くできませんでした
• ①if(emp[i].id == 0)、②while(num >= SIZE)、③forと入れ子にした ⇒ 上手くできませんでした

補足情報

このサイトでコンパイルしました。
https://www.onlinegdb.com/online_c_compiler

Answer 1

まず、

for(i = 0; i < SIZE; i++) {
    printf("%6d%9d%11.2lf\n", emp[i].id, emp[i].age, emp[i].salary);
}

の継続条件i < SIZEにより、必ずSIZE回(現在の設定では4回)この繰り返しが行なわれます。

この継続条件を変更してもよいのですが、その中のprintfを「条件を満たしたときだけ実行する」ことを考えてみたらいかがでしょうか。

問題文には明示されていませんが、「従業員IDは 0 ではない」という前提があるので、それを使えばいいかと思います。念のために「従業員追加」のほうのコードに、「入力された従業員IDが 0 なら、追加作業を行なわない」処理を追加すると確実です。

余談: このあと「3. 従業員情報を削除」みたいなのを追加する指示が出ることが予想できますが。

Comments

[aluminium]

仰るとおりです！質問させていただいたコードは演習の一部で、削除もあります。
ご丁寧に解説いただき、どこに条件を設定すべきかよくわかりました。ありがとうございました！

Answer 2

登録した数分だけ回ればいいのですよね？
0を出さないとかではなく。
変数名がわかりづらいですが、iが登録数と思われるので、その数分回ればよいのです。
iはわかりづらいののでnumなどの名前に変えてください。
そもそも従業員数のカウントと、表示ループ時のカウンタに同じ変数iを使ってしまっているのも問題です。
最初に表示してしまうともう従業員の登録はできなくなります（iが4になるため）。
変数名の追加と修正を行ったうえで表示処理を次のようにすればよいでしょう。

            for(i = 0; i < num; i++) {
                printf("%6d%9d%11.2lf\n", emp[i].id, emp[i].age, emp[i].salary);
            }

Comments

[aluminium]

iが他とバッティングしていて使えないとは思ってもみなかったです！
jにしてみたら他も上手く動きましたので、ベストアンサーにさせていただきます。
まだまだロジックの理解が足りないですね... 練習します！ありがとうございました！

Answer 3

表示を行うforループの中で表示すべき条件かチェックを行って、不要ならcontinueで飛ばしてしまう、でいいのではないでしょうか。

Comments

[aluminium]

早々にご回答いただいたのに御礼が遅くなり申し訳ありません。
if文をかける位置をご教示いただきありがとうございました！forループの中に条件設定したら上手くいきました！