while文で、値を代入したい。

Question

30000を超える値を入力したら、再度値を入力してもらうプログラムを作成しています。
しかし、一度目の値入力から更新されずにDの値がマイナスになってしまいます。
どうしたらよいでしょうか。。。

#include<stdio.h>
int main(void){
int A,B,C,D,Z;
Z=30000;
printf("%d人の兵力を割り振ってください。\n",Z);
printf("グーの兵力："); scanf("%d", &A);
D = Z - A;
while (D<0) {
system("cls");
printf("兵は%d人しかいません。\n",Z);
printf("%d人の兵力を割り振ってください。\n",Z);
printf("グーの兵力："); scanf("%d", &A);
if(A<Z)
{
break;}
}

printf("残りの兵力は%dです。\n", D);
printf("チョキの兵力："); scanf("%d", &B);
	C = D - B;
	while (C < 0){
system("cls");
printf("残りの兵は%d人しかいません。\n", D);
    printf("チョキの兵力："); scanf("%d", &B);
    if(B<D)
{
	break;
}
	}
printf("パーの兵力：%d\n",C);

return 0;
}

Comments

[y_waiwai]

このままではコードが読みづらいので、質問を編集し、<code>ボタンを押し、出てくる’’’の枠の中にコードを貼り付けてください

[otn]

数字を全角で入力しているとか？

Answer 1

再入力後にDやCを更新していません。
ifは不要です。

#include<stdio.h> 
+#include <stdlib.h>
int main(void) {
	int A, B, C, D, Z;
	Z = 30000;

	printf("%d人の兵力を割り振ってください。\n", Z);
	printf("グーの兵力："); scanf("%d", &A);
	D = Z - A;
	while (D < 0) {
		system("cls");
		printf("兵は%d人しかいません。\n", Z);
		printf("%d人の兵力を割り振ってください。\n", Z);
		printf("グーの兵力："); scanf("%d", &A);
+		D = Z - A;
-		if (A < Z)
-		{
-			break;
-		}
	}

	printf("残りの兵力は%dです。\n", D);
	printf("チョキの兵力："); scanf("%d", &B);
	C = D - B;
	while (C < 0) {
		system("cls");
		printf("残りの兵は%d人しかいません。\n", D);
		printf("チョキの兵力："); scanf("%d", &B);
+		C = D - B;
-		if (B < D)
-		{
-			break;
-		}
	}
	printf("パーの兵力：%d\n", C);
	return 0;
}

Answer 2

SHOMIさんの回答で良いと思います。最初の回答は読み違ってました。

Z…全兵力
A,B,C…３つそれぞれの兵力
D…一時変数

A,B,Cに割り振る際に全兵力Zを引いていくような感じにするとわかりやすくなると思います。
ちょっと書き換えてみました。ついでに変数名も変えてみました。

#include<stdio.h>
#include<stdlib.h>

int main(void){
    int RemainPerson;
    int APerson,BPerson,CPerson;

    /* 残り人数(初期値は合計人数) */
    RemainPerson=30000;

    /* 兵力Aの入力 */
    printf("%d人の兵力を割り振ってください。\n",RemainPerson);
    printf("グーの兵力："); scanf("%d", &APerson);
    while (RemainPerson - APerson < 0) {
        /* 残り兵力を越えてしまっている場合、正しい値が入力されるまでやり直す。*/
        system("cls");
        printf("兵は%d人しかいません。\n",RemainPerson);
        printf("%d人の兵力を割り振ってください。\n",RemainPerson);
        printf("グーの兵力："); scanf("%d", &APerson);
    }
    /* 兵力Aを除いたものが残り人数となる。*/
    RemainPerson -= APerson;

    /* 兵力Bの入力 */
    printf("残りの兵力は%dです。\n", RemainPerson);
    printf("チョキの兵力："); scanf("%d", &BPerson);
    while (RemainPerson - BPerson < 0) {
        /* 残り兵力を越えてしまっている場合、正しい値が入力されるまでやり直す。*/
        system("cls");
        printf("残りの兵は%d人しかいません。\n", RemainPerson);
        printf("チョキの兵力："); scanf("%d", &BPerson);
    }
    /* さらに兵力Bを除いたものが残り人数となる。*/
    RemainPerson -= BPerson;

    /* 兵力Cの決定(残り人数がそのまま兵力Cとなる。) */
    CPerson = RemainPerson;
    printf("パーの兵力：%d\n",CPerson);

    return 0;
}

↓↓ 以下最初に書いた間違い ↓↓

残り兵力を保持している変数は「Z」ではないでしょうか？
「D」は一時的に使用される変数のようです。兵力がマイナスであるのかを判断するには「Z」を対象としなければいけないのではないでしょうか？

あと、変数がAとかBとか英数字1文字だと読むのがつらいので、もっとわかりやすい変数名にすることをおすすめします。
(実際自分もリファクタリングせずにソースコードを見ているのでなんか読み間違っているような気がしています。)