while文でつまづいています

質問

Ｃ言語はじめたばかりのものです。

単細胞生物が翌日（2日目）には分裂して2細胞になるとする。1匹もしなないとすると
１万匹を超えるのは何日目になるかを算出するプログラムを作成しています。
「15日目に16384個の細胞数になります。」という結果を出したいのですが、
「１日目に１個の細胞数になります。」と表示され、１日目までしか計算されません。

まだscanfやiprintf、ifやwhileぐらいしか習っていません。
どう直したらよいでしょうか？

#include<stdio.h>
#include<math.h>

main()
{
int n; /*細胞数*/
int d = 1; /*日数*/

while(n > 10000)
{
n = pow(2,d-1);
d = d + 1;

if(n > 10000)
break;
}

printf("%d日目に%d個の細胞数になります。",d,n);

}

回答No.4

KEIS050162 · ベストアンサー

いくつか突っ込みどころがありますのでひとつひとつ行きましょう。

１）条件式が逆

while( n > 10000 ) は、　「ｎが１００００より大きい時に、次の処理を実行せよ」という記述なので、質問者殿の記述では、何もしないで抜けてしまう。（ｎを初期化していないので、最初のｎが不明ですが、恐らく１００００より小さい値が入っているのだと思われる）

それから、変数は必ず初期化するクセをつけましょう。この記述だと、int n　の定義の後、n > 10000　と、初期化されていないｎを判定することになります。

これが、根本原因ですね。以下は、補足。

２）while文のbreak条件が冗長
最初のwhile("条件")を書くなら、途中のif文は要りません。

while文の書き方の例：
・条件の判定を最初にする場合
　　　　while("条件") {
　　　　　　　演算
　　　　}

・演算後に条件判定をしたい場合
　　　　while( TRUE ) { （もしくは、while( 1 )　でも良い）
　　　　　　　　演算
　　　　　　　　if ("条件" ) break;
　　　　}

・演算後に条件判定をしたい場合（その２）
　　　　do {
　　　　　　　　演算

　　　　} while( "条件" )

３）なぜｐｏｗ（）？
日ごとに、倍々に増えて行くので、結果は、２の（日数－１）乗　で良いのですが、
毎回、２の（日数ー１）乗を計算するのではなく、
この場合は、ｎが１ループする度に倍になる記述の方がシンプルで良いと思います。

例）
int d = 1;
int n = 1; (初期化）

while ( n <= 10000 ) {
　　　n = n * 2;
　　　d++;
}

ご参考に。

お礼

とてもわかりやすく説明してくださり、ありがとうございます。
１週間前に始めましたが、この辺りから難しくなってきました。
もう一度基本を確認します。

回答No.3

jam-best

while()文は「()内が真である場合の間処理を実行」です。

1日目にn=1で　1>10000の処理結果は偽ですのでwhile文が1回で抜けています。

他の方もおっしゃっていますが、nも初期化しましょう。
プログラムの書き方は人それぞれですが、
個人的に、途中でif文で判定してbreakにするのであれば、while(1)で無限ループにするなどのほうがいいのではとも思います。

お礼

ありがとうございます。
もう一度勉強しなおします。
無限ループという方法もあるのですね。参考になります。

回答No.2

harawo

＞ while(n > 10000)

これは、「while (n <= 10000)」では？
nに初期値が代入されていないのも、気になります。

お礼

ありがとうございます。
初期化わすれてました。頭が混乱してますね。

回答No.1

Tacosan

・n を初期化せずに使っているということに気づいてる?
・わざわざ pow を使う理由が分からない.
・while 文の意味を復習してください.

お礼

ありがとうございます。
基本からやり直します。