t秒後のロケットの速度vを求める関数

Question

ｔ秒後のロケットの速度を求める関数を作れという課題で、作って実行してみたところ、
05-3.exe は動作を停止しました。
問題が発生したため、プログラムが正しく動作しなくなりました。プログラムは閉じられ、解決策がある場合はWindows から通知されます。
と、表示され実行ができません。
どこが間違っているのでしょうか？よろしくお願いします。

#include<stdio.h>

float a = 0;

float velocity(float F, float m, int t){

float a =  F/m;
float v = a*t;

return v;

}

float main(void)
{
float m = 0;
float F = 0;
int t = 0;
printf("速度0で静止しているロケットに点火して打ち上げる。\n打ち上げからt秒後の速度V(t)を求める。\n");
printf("ロケットの質量 m=");
scanf("%f,&m");
printf("ロケットの推力 F=");
scanf("%f",&F);
printf("時間 t=");
scanf("%d",&t);

printf("打ち上げからt秒後の速度は%fです。",velocity(F,m,t));

}

Comments

[cateye]

環境（コンパイラなど）を教えてください。デバッグ時にはどこでエラーが出ますか？

[jo-jo]

Borland C++ Compiler 5.5 を使っています。コマンドプロンプトで値を入れて実行するとエラーが出ます。デバックのやり方がわからないです、、、

Answer 1

1回目のscanfがおかしいですね。
他は大丈夫だと思います。

	scanf("%f,&m"); 
	↓
	scanf("%f",&m); 

Answer 2

２回目以降のscanf()「scanf("%f",&F);、scanf("%d",&t)」は正常に読み込めていますか？←printf()などを挿入して、結果を表示してみましょうｄ＾＾

Comments

[cateye]

scanf()は使い方が難しい関数です。私の場合は、fgets()→sscanf()を使います。

Answer 3

計算方法が正しいかについてはご本人に考えていただくとして、
文法上おかしな所を指摘してみました。

#include <stdio.h>

// velocity関数のプロトタイプ宣言    <= C++では無いとエラーになる
float velocity(float F, float m, int t);

// float a = 0;    <= velocity関数内でaが宣言されているのでこれは不要 
float velocity(float F, float m, int t)
{
    float a = F / m; 
    float v = a * t;

    return v; 
}

int main()    // <= main関数の戻り値はint, 引数を使わないなら空ける(voidは間違い) 
{ 
    float m = 0.0f;
    float F = 0.0f; 
    int t = 0; 

    printf("速度0で静止しているロケットに点火して打ち上げる。\n打ち上げからt秒後の速度V(t)を求める。\n");
    printf("ロケットの質量 m = "); 
    scanf("%f", &m);    // <= format(%f)と変数(&m)は別の引数
    ...(中略)

    printf("打ち上げからt秒後の速度は%fです。\n", velocity(F, m, t));
}

変数名はmとかtみたいに１文字だとわかりにくいのでmassとかsecとか意味のある単語に変える
特に理由が無ければfloatと%fじゃなくdoubleと%lfを使ったほうが良いかと思います。

Comments

[jo-jo]

指摘ありがとうございます！！
引数や戻り値の使い方がまだまだあやふやで基本的なことができていないので、指摘されたところを改善していきます。