回答編集履歴
1
oles => roles
answer
CHANGED
|
@@ -43,7 +43,7 @@
|
|
|
43
43
|
scanf("%d", &utagai);
|
|
44
44
|
suspicion[utagai]++; // 疑われた人の疑惑値を +1 する
|
|
45
45
|
```
|
|
46
|
-
名前がかぶらないようにするのは、エディタ等で、特定の変数を探す・変数名を一括変換する、といった作業がしやすく
|
|
46
|
+
名前がかぶらないようにするのは、エディタ等で、特定の変数を探す・変数名を一括変換する、といった作業をがしやすくするためです。
|
|
47
47
|
|
|
48
48
|
変数名や関数名は、何のためにあるのか、何をするものか、という意味が伝わりやすい名前にするよう、心がけてください（私にとっても難しいですけどね）。
|
|
49
49
|
ここで配列にした場合、その値が何を意味するか（例：疑惑値）、配列の添え字（インデックス）は何を意味するか（例：プレーヤーを特定する）を整理してください。配列の添え字は、Ｃでは０から始まります。ユーザに０〜３を入力させるか、１〜４を入力させるか、も大事です。
|
|
@@ -105,7 +105,7 @@
|
|
|
105
105
|
case DIVINER:
|
|
106
106
|
```
|
|
107
107
|
|
|
108
|
-
・ここで、CITIZEN1 と CITIZEN2 の区別が要るのかどうか。同じ値でも良いのでは？それは私には判断できませんが、区別が要らない可能性があります。その場合、配列名 values（値）は他にも使いたくなりそうなので
|
|
108
|
+
・ここで、CITIZEN1 と CITIZEN2 の区別が要るのかどうか。同じ値でも良いのでは？それは私には判断できませんが、区別が要らない可能性があります。その場合、配列名 values（値）は他にも使いたくなりそうなので roles（役）に変えた上で、こんな風にするでしょう。
|
|
109
109
|
```C
|
|
110
110
|
enum { EXECUTED, JINROU, CITIZEN, DIVINER };
|
|
111
111
|
int roles[4] = { JINROU, CITIZEN, CITIZEN, DIVINER };
|