recipsumの値が常に1になってしまい、
その原因がわからなかった人は
結局third.cを本質的に理解していなかったということである。
本質的な理解をせずに、試行錯誤してうまくいけばOKという学習態度では、 プログラミングの能力は身につかない。
次のようなプログラム:
double recipsum(int n) {
double sum=0;
for( ; n>0; n--) {
sum += 1/n;
}
return sum;
}
がうまく動作しなければ、原因を探るために次のように
printfを挿入してみる。
double recipsum(int n) {
double sum=0;
for( ; n>0; n--) {
sum += 1/n;
printf("n=%dのとき、sum=%f\n", n, sum);
}
return sum;
}
sumが増えていかないことがわかるので、
1/nの値を表示してみる。
double recipsum(int n) {
double sum=0;
for( ; n>0; n--) {
double tmp = 1/n;
sum += tmp;
printf("n=%dのとき、1/n=%f, sum=%f\n", n, tmp, sum);
}
return sum;
}
このように問題点を絞りこんでいけば、
うまく動作しない原因を突き止めることができる。
ここまで自分で突き止めて、
「1/nが常に0になります。なぜですか？」
という質問なら答えることができる。
仮引数の他に変数を1つだけという条件が満たされない解答が多かった。
double recipsum(int n) {
int i;
double sum=0;
for(i=1; i<=n; n++) {
sum+=1.0/i;
}
return sum;
}
問題を良く読んでいないのか、それとも未完成でも出さないよりは
良いということか？
後者の場合は、必ずコメントの形で未完成である旨を明示して欲しい。
recipsumの型は、
double recipsum(int n)と指定されているので、
もちろんこれを勝手に変えてはいけない。
double recipsum(int n, int i) { double sum=0; for(i=1; i<=n; n++) { sum+=1.0/i; } return sum; }
do〜while文を使う場合、
do {
…
} while (--n>0);
フツーは whileのまえに空白を入れる。
ただし、この問題の場合 do〜whileよりも
whileを使う方が良い。
下の例のようにnを減算するタイミングを間違えると、
while(n-- > 0) {
sum += 1.0/n;
}
1.0/0.0を計算することになり、プログラムが異常終了をする。