ある値入力まで入力繰り返しプログラム

Question

関数double fact(int n)を作成し、main で呼び出し、
入力が負になるまで number=を繰り返すプログラムを作成しようとしています。
以下のプログラムではnを入力すると出力はされますが
負の値がでるまで繰り返しnumber=が表示されません。
どのように変更すればよいかご存じの方ご教示お願い致します。

C言語コード

#include<stdio.h>
double fact(int n);
int main(){
int number;
double n;
int k;

while(1){
printf("number="); scanf("%d",&n);
if(n<0) break;
k = k + 1.0/fact(n);}
printf("e=%f\n",k);
}
return 0;

}
double fact(int n){
n=1;
int i;
for(i=1;i<=n;i++){
n = n*i;
}

Comments

[SHOMI]

main()の{}の数があっていませんし、fact()は途中で途切れています。
実際にコンパイルしたコードを貼ってください。
kは不定値のまま使用されています。

[cateye]

上↑のコード、インデントも付けないで、自分でも見にくくないですか？
・・・ちゃんとインデント付けましょう。
fact()のnって何に使ってます？・・・デバッグしましょう・・・

[Daregada]

おそらく、「本当にやりたいこと」と「書きたいと考えているコード」と「実際に書いているコード」がすべて食い違っています。
「キー入力した値 N から 0 までの 1/n! を加算して、自然対数の底 e の近似値を求める」ということが、「本当にやりたいこと」なのでは?

[KeiD]

こめんとありがとうございます。おっしゃる通りです。すいません。

Answer 1

プログラムを組むならインデントを付ける癖をつけましょう
まずは、そこから始めた方がいいです。

下記のサイトを見て可読性の違いを認識してください。
書いてある言語は違いますが基本的には一緒です
http://wordpress.ideacompo.com/?p=11459

Answer 2

現時点で貼られているコードはmain()の{}の対応があっていませんし、fact()は途中で途切れています。
コンパイルが通らないので正しいコードを貼りなおしてもらいたいですが、main()で明らかにおかしい点として

• scanfでdoubleを読み取るなら書式文字列は"%lf"です。

もしくは、fact()の引数はintなので整数を読み取りたいという事であれば、nをdoubleではなくintにしてください。

• kが不定値のまま使用されています。

Comments

[KeiD]

ありがとうございます。
#include<stdio.h>
double fact(int n);
int main(){
int n;
double k;

while(1){
printf("number="); scanf("%lf",&n);
if(n<0) break;
k = k + 1.0/fact(n);}
printf("e=%f\n",k);
return 0;
}

double fact(int n){
n=1;
int i;
for(i=1;i<=n;i++){
n = n*i;
}}
でC++で実行しましたが入力数がそのまま出力されてしまいます。

[cateye]

fact()のfor(i=1;i<=n;i++)が1回しか回らないのは分かりますか？
また、戻り値がありません。

[KeiD]

ありがとうございます。初心者ですいません。

[dodox86]

それと、SHOMIさんの回答を良く読んでください。nがdouble型ならscanfの書式制御文字列は"%lf"にすべき、との指摘ですが、"%d"のままで使うならnをint型にせよ、ということです。

[KeiD]

ありがとうございます。

[SHOMI]

fact()にreturnがありませんし、kは相変わらず初期化しておらず不定値です。