編集履歴

質問編集履歴

質問者の返信を元に「自分で一部書き換えたプログラム」を再度書き換えました。

2019/01/10 09:03

投稿

Akihiro_O

スコア20

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -224,25 +224,29 @@
 {
-	int i, j, k;
+	int i,j, k;
 	char sex[16];
 	do
 	{
 		printf("名前を入力してください：");
-		scanf_s("%255s", &data->name,256);
+		scanf_s("%s", data->name,256);
-		printf("%sでよろしいですか？？（合ってます→1、間違いました→2）\n",data->name);
+		printf("%sでよろしいですか？？（「合ってます」：「1」を入力、「間違いました」：「1以外」を入力）\n", data->name);
 		scanf_s("%d", &i);
-	} while (i == 2);
+	} while (i != 1);
 	do
@@ -252,17 +256,19 @@
 		scanf_s("%d", &data->age);
-		printf("%dでよろしいですか？？（合ってます→1、間違いました→2）\n", data->age);
+		printf("%dでよろしいですか？？（「合ってます」：「1」を入力、「間違いました」：「1以外」を入力）\n", data->age);
 		scanf_s("%d", &j);
-	} while (j == 2);
+	} while (j != 1);
 	do
 	{
-		printf("性別を入力してください（男性→1、女性→2）：");
+		printf("性別を入力してください（「男性」：「1」を入力、「女性」：「2」を入力）：");
 		scanf_s("%d", &data->sex);
@@ -274,19 +280,27 @@
 		}
+		else if(data->sex == 2)
+		{
+			strcpy_s(sex, 16, "女性");
+		}
 		else
 		{
-			strcpy_s(sex, 16, "女性");
+			continue;
 		}
-		printf("%sでよろしいですか？？（合ってます→1、間違いました→2）\n", sex);
+		printf("%sでよろしいですか？？（「合ってます」：「1」を入力、「間違いました」：「1以外」を入力）\n", sex);
 		scanf_s("%d", &k);
-	} while (k == 2);
+	} while (k != 1);
@@ -332,10 +346,10 @@
 }
 ```
 **「自分で一部書き換えたプログラム」**に穴や間違いがありましたら、教えて頂けると助かります。

名前を入力する部分のscanf_s関数の引数を変更しました。%s→%255s

2019/01/10 09:03

投稿

Akihiro_O

スコア20

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -236,7 +236,7 @@
 		printf("名前を入力してください：");
-		scanf_s("%s", &data->name,256);
+		scanf_s("%255s", &data->name,256);
 		printf("%sでよろしいですか？？（合ってます→1、間違いました→2）\n",data->name);

コードブロックを採用しました。質問内容を一部追記しました。質問回答を受け、構造体内の変数宣言部でunsignedを使いました。

2019/01/07 10:24

投稿

Akihiro_O

スコア20

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -6,7 +6,19 @@
+#前提
+情報を入力する「InputPeople関数」内において、正しく入力されたか確認を取るために、著者の解答を一部書き換えました。
+#実現したいこと
+より安全で正確な（穴のない）プログラムが書けたらと思っています。
-##問題
+#問題
 「3人分の、名前、年齢、性別、を入力して表示するプログラムを作りなさい。
@@ -18,6 +30,8 @@
 #著者の解答
+```ここに言語を入力
 ＃include<stdio.h>
 ＃include<string.h>
@@ -140,10 +154,14 @@
+```
 #自分で一部書き換えたプログラム
+```ここに言語を入力
 ＃include<stdio.h>
 ＃include<string.h>
@@ -152,11 +170,11 @@
 {
-	char name[256];
+	unsigned char name[256];
-	int age;
+	unsigned int age;
-	int sex;
+	unsigned int sex;
 }People;
@@ -186,6 +204,8 @@
 	}
 	for (i = 0; i < 3; i++)
 	{
@@ -204,25 +224,71 @@
 {
-	printf("名前：");
+	int i, j, k;
-	scanf_s("%s", &data->name, 256);
-	printf("年齢：");
+	char sex[16];
-	scanf_s("%d", &data->age);
 	do
 	{
-		printf("性別(1-男性、2-女性)：");
+		printf("名前を入力してください：");
+		scanf_s("%s", &data->name,256);
+		printf("%sでよろしいですか？？（合ってます→1、間違いました→2）\n",data->name);
+		scanf_s("%d", &i);
+	} while (i == 2);
+	do
+	{
+		printf("年齢を入力してください：");
+		scanf_s("%d", &data->age);
+		printf("%dでよろしいですか？？（合ってます→1、間違いました→2）\n", data->age);
+		scanf_s("%d", &j);
+	} while (j == 2);
+	do
+	{
+		printf("性別を入力してください（男性→1、女性→2）：");
 		scanf_s("%d", &data->sex);
-	} while (data->sex != 1 && data->sex != 2);
+		if (data->sex == 1)
+		{
+			strcpy_s(sex, 16, "男性");
+		}
+		else
+		{
+			strcpy_s(sex, 16, "女性");
+		}
+		printf("%sでよろしいですか？？（合ってます→1、間違いました→2）\n", sex);
-//この部分を書き換えました。
+		scanf_s("%d", &k);
+	} while (k == 2);
 	printf("\n");
@@ -244,10 +310,6 @@
 	printf("年齢：%d\n", data.age);
 	if (data.sex == 1)
 	{
@@ -270,11 +332,7 @@
 }
+```