プログラムの勉強でわからないところがあります

質問

明解C言語の入門編、演習4-3
二つの整数値を読み込んで、小さい方の数以上で大きい方の数以下の整数を全て加えた値を表示するプログラムを作成するものなのですが、うまくいかなく困っています。
他の書き方でやれば普通にいけると思うのですが、どうしてこれだとできないのかがわからないとすっきりしなくって・・・

以下のようなプログラムで試しています。

#include <stdio.h>

int main(void)
{
int n1,n2,n3,n4;

puts("二つの整数を入力してください");
printf("整数1:"); scanf("%d",&n1);
printf("整数2:"); scanf("%d",&n2);
n3=(n1>n2) ? n2 : n1;
n4=(n1>n2) ? n1 : n2;
printf("%d以上%d以下の全整数の和は", n3,n4);

int num=n1;
int sub=n2;
int wa;
printf("num=%d sub=%dで",num,sub);
do{
if (n1>n2)
{
wa=num+sub; /* 大きいほうの数(num)にsub(小さい方の数+0,2,3,4・・・)を足していく */
sub=sub+1;/* ここを通るたびにsubに+1 */
printf("%d",wa);}

else
{
wa=sub+num;
num=num+1;
printf("%d",wa);}
}while(sub!=n1 || num !=n2); /* 小さい方の数に1足していったものが大きい方の数と等しくなったら0を生成し終了 */

printf ("です\n");

return 0;
}

このプログラムで試すと、延々と計算をし続けていってしまうのです。

whileあたりがおかしいのかと思ったのですが、どこがおかしいのかわからなくって・・
whileは、中の数字が0を生成する場合止まる、というものだったと思うので、小さい方の数字に1ずつ足していって、大きい方の数字と等しくならない限り続けるように作ったつもりなのですが・・・

どこがおかしいのか教えて頂けると嬉しいです。

それとワガママなお願いなのですが、できれば、正答のようなものは書かないでもらえると助かります、どこが間違いなのかだけ教えて頂いて、後はなんとか自分の力でこなしてみたいので・・・

自分勝手なお願いですが、よろしくお願いします。

回答No.1

麻野 なぎ（@AsanoNagi） · ベストアンサー

> 大きい方の数字と等しくならない限り続けるように作ったつもりなのです

そうなってないですね。
while() の中身は、「大きい方の数字と等しくないか、小さい方の数字と等しくないかどちらかの条件が満足される限り」です。

n1 と n2 の大小関係は、プログラムの実行中変化しませんから、num と sub のいずれかは「変化しない」ことになります。
つまり、大きいほうか小さいほうかは、永久に等しくなりません。

実際、n1 と n2 の大小関係は不明なのですが、わざわざ、n3 と n4 に「大小関係がはっきりするように」再設定しています。
これがあるのに、「大小関係が不明」という前提でプログラムを書くのが間違い（と言わないまでも、無駄）です。

あと、wa が初期化してない気がします。

回答No.2

sepgp34

はじめまして。
No1さんの回答に同意です。

間違いではありませんが、補足です。
せっかくn3、n4という小さい数値と大きい数値を設定しているのであれば、
do-while文中のif文を使用しなくてよいかと思われます。
あと、No1さんがおっしゃるように初期化漏れが多いですね。
変数はきちんと初期化しておきましょう。

amagatsu はdo-whileを使用しておりますが、
単純にfor文を使用する方法ではだめですか？
for(min値からmax値まで){
和変数に加算していく
}
の方が簡単な気がします。