switch文が正常に作動しません。c言語

Question

switch文のcase２の動作が正常に作動しません。printf("Enter a new employee name:");このパートがうまくいってないみたいで下記の出力のような次のprintfに繋がった出力になってしまいます。
case２を独立させて動かしてみたところ問題なく期待通りの出力を得られました。

問題のあるプログラム

#define _CRT_SECURE_NO_WARNINGS
#include <stdio.h>
#include <string.h>

void search();
void assing();

int main(void)
{
	int ch;

	while (1)
	{

		printf("==================Employee Management System=============\n\n");

		printf("1. Search\n\n");
		printf("2. assign\n\n");
		printf("0. Exit\n\n");

		printf("========================================================\n\n");

		printf("\nPlease enter your Choice:");
		scanf("%d", &ch);

		switch (ch)
		{
		case 1: search();
			break;
		case 2: assing();
			break;
		case 0: exit(0);
		}


	}
	return 0;
}

void search()
{
	FILE* fp;
	fp = fopen("tel.txt", "r");
	char str[1000];
	char id[100];
	char name[100];
	int key;

	printf("Do you want to search by 1.ID or 2.name?(Enter number 1 or 2): ");
	scanf("%d", &key);

	if (key == 1)
	{
		printf("Enter search ID: ");
		scanf("%s", id);
		while (1)
		{

			if (fgets(str, sizeof(str), fp) == NULL) break;

			if (strstr(str, id) != NULL)
			{

				fgets(str, sizeof(str), fp); printf("%s", str);


			}
		}
	}
	else if (key == 2)
	{
		printf("Enter search Name: ");
		scanf("%s", name);
		while (1)
		{
			if (fgets(str, sizeof(str), fp) == NULL) break;

			if (strstr(str, name) != NULL)
			{

				fgets(str, sizeof(str), fp);
				fgets(str, sizeof(str), fp); printf("%s", str);
			}
		}


	}
	fclose(fp);
}

void assing()
{
	int cnt = 0；
	char name[50];
	char id[10];
	char tel[15];
	char department[30];
	int c;
	int count = 0;


	FILE* fp;
	fp = fopen("tel.txt", "a+");

	while ((c = getc(fp)) != EOF)
	{
		if (c == '\n') cnt++;
	}
	count = cnt / 5 + 1;

	rewind(fp);
	
	printf("Enter a new employee name:");
	fgets(name, sizeof(name), stdin);

	printf("Enter a new employee tel number:");
	fgets(tel, sizeof(tel), stdin);

	printf("Enter a new employee department:");
	fgets(department, sizeof(department), stdin);



	fprintf(fp, "NAME: %s", name);
	fprintf(fp, "ID: TP0%d\n", count);
	fprintf(fp, "TEL: %s", tel);
	fprintf(fp, "DEP: %s", department);
	fprintf(fp, "========================\n");

	fclose(fp);

}

出力

==================Employee Management System=============

1. Search

2. assign

0. Exit

========================================================


Please enter your Choice:2
Enter a new employee name:Enter a new employee tel number:12345678
Enter a new employee department:1234567

case2の独立したプログラム

　　#define _CRT_SECURE_NO_WARNINGS
	#include <stdio.h>
	#include <string.h>
	

	char name[50];
	char str[100];
	char tel[15];
	char department[30];
	int c;
	int count = 0;
	int cnt;

	
	int main()

	{
		FILE* fp;
		fp = fopen("tel.txt", "a+");

		while ((c = getc(fp)) != EOF)
		{
			if (c == '\n') cnt++;
		}
		count = cnt / 5 + 1;


		rewind(fp);

		printf("Enter a new employee name:");
		fgets(name, sizeof(name), stdin);

		printf("Enter a new employee tel number:");
		fgets(tel, sizeof(tel), stdin);

		printf("Enter a new employee department:");
		fgets(department, sizeof(department), stdin);



		fprintf(fp, "NAME: %s", name);
		fprintf(fp, "ID: TP0%d\n", count);
		fprintf(fp, "TEL: %s", tel);
		fprintf(fp, "DEP: %s", department);
		fprintf(fp, "========================\n");

		fclose(fp);

		return 0;

	}

Enter a new employee name:xxx
Enter a new employee tel number:07000000000
Enter a new employee department:asia

Answer 1

C言語のコードを書くなら、デバッグできる環境を整えましょう。
Eclipseや、WindowsならVisualStudioなど。
コードの任意の場所で実行を止め、変数のナカミを見ることができます。そこから１行づつ実行して、コードの流れを見れるようになります
そうすれば、アテズッポでコードを書かなくて済むようになります。

Comments

[kananananana]

VisualStudio2019を使用しています。

[y_waiwai]

なら、そこのところでブレークポイント設定して、そこで実行を止め、変数の中身を見てみましょう。
なにがはいってるでしょうか。

[kananananana]

name 0x006ffab0 "\n" char[50]
[0] 10 '\n' char
　　　[1] 0 '\0' char
が入っていました。

[y_waiwai]

別回答の通り、改行コードがはいってますね

[y_waiwai]

scanfのあと、fgetsで１行空読みするとか、ですね。

scanfとfgetsってのはあんまし相性は良くないので、できればどちらかに統一したほうがいいですが。

[kananananana]

なぜここに改行コードが入ってしまったんでしょうか？

空読することで解決しました！

なぜ相性が良くないのでしょうか？
沢山聞いてしまってごめんなさい。

[y_waiwai]

scanfってのは文字単位、数値単位で読むので、この場合のように処理残りが問題になります
fgetsが１行単位で読んでくるもんなんで、そこらへんなにが起きるのか分かりづらいですね

そうでなくとも、scanfという関数は、想定してない入力でフリーズしたり暴走さえするようになるような危険な関数です。実務では禁忌になってますんで、そのつもりで。

[kananananana]

なるほど！理解出来ました。
ありがとうございます！

Answer 2

assingに飛ぶ前のscanf("%d", &ch);を入力した際に残っていた改行文字を、Enter a new employee name:直後のfgetsで拾ってしまったのではないでしょうか？

Comments

[kananananana]

その通りでした！ありがとうございます！