コンパイルエラーが出ているが原因がわからない

Question

ラザフォードの散乱のプログラミングを書いているのですが、コンパイルエラーへの対処がわからないです。
特に、7，8，9，10行目で「 initializer element is not a compile-time constant」と出ます。
15行目で「note: 'g' declared here」と出ます。
24、25，26で「 too few arguments to function call, expected 4, have 2」と出ます。
プログラム初心者なので、よければエラーの原因と対処法を教えてもらいたいです。
コンパイルは「paiza.LO」のサイトで行いました。
よろしくお願いします。

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

#define N 10

double x=-2000, b=2000,y[N]={10,50,100,150,200,300};
double n,h,t,vx,vy,q=5.30*2,ma=6.664*pow(10,-27),M;

double  M=q/ma;
double  vx=pow(M,0.5);
double  n=40000;
double  h=(b-x)/n;   


double g(double x, double y, double ma, double r);

int main() {
        double　i;
        for(i = 0 ; i <=n ; i++)
        {
            double k1,k2,k3,k4;

            t+=h;
            k1 = g(t,vy);
            k2 = g(t+h*0.5,vy+k1*0.5);
            k3 = g(t+h*0.5,vy+k2*0.5);
            k4 = g(t+h, y+k3);
            vy = vy + (k1+2.0*k2+2.0*k3+k4)/6.0;
            y=vy*t;
            x=vx*t;
            printf("%2f %f %f\n",t,x,y);

            return 0;
        }
}  

double g(double x, double y, double ma, double r){
　　   double z=79,m=3.141592,e=8.854*pow(10,-12),r=sqrt(pow(x,2)+pow(y,2));

        g =2*z*z*y/(4*m*e*ma*pow(r,3));

        return(g);
}

Answer 1

特に、7，8，9，10行目で「 initializer element is not a compile-time constant」と出ます。

宣言と同時に初期化する場合の値は、コンパイル時に計算出来る物だけです。
当たり前ですが、pow()のような関数の呼び出しは、コンパイル時じゃなくて実行時に行われますので、書けません。

24、25，26で「 too few arguments to function call, expected 4, have 2」と

自分で、関数gを引数4つの関数であると宣言したのに、それを忘れて引数2つで呼び出そうとしているからです。
自分で書いたコードは、少なくともコードを書いている間は覚えていましょう。

15行目で「note: 'g' declared here」と出ます。

は、上記に付随しての情報です。

以上がコンパイルエラーについてですが、他の方の指摘にもあるとおり、他にも色々間違っています。
よく考えて書きましょう。書いたら読み直しましょう。

Comments

[kisara11235]

ありがとうございます。
教本を確認しながらもう一度作り直します。

Answer 2

ma=6.664*pow(10,-27)

コンパイル時に値が決まらない（pow関数の呼び出し）のでエラー

double M=q/ma;

上記が原因で値が決まっていない。

Comments

[cateye]

double vx=pow(M,0.5);　もそうですね、

[kisara11235]

ありがとうございます。
教本を確認しながらもう一度作り直します。

Answer 3

こんにちは。
パッと見た感じですが、ループ文の中に変数宣言とreturnはよろしくないかと思いました。
それが根本的な解決になるかはわかりませんが、とりあえず、for文の中にdouble k1;やreturn 0;があるのはおかしいのでmain(){の下に書くといいと思います。
return 0;はmain文の最後で良いと思います。

Comments

[syalpon]

あとサブ関数g()の返値がおかしいですね。
gは関数なのにあたかも変数のようにg = ...となっています。違うdouble型の変数名で宣言して返すといいと思います

[syalpon]

さらに
'''
double x=-2000, b=2000,y[N]={10,50,100,150,200,300};
double n,h,t,vx,vy,q=5.30*2,ma=6.664*pow(10,-27),M;

double M=q/ma;
double vx=pow(M,0.5);
double n=40000;
double h=(b-x)/n;
'''
の部分では変数宣言が二度されています。ので
double n,h,t,vx,vy,q=5.30*2,ma=6.664*pow(10,-27),M;
とdouble型でn,h,t...Mを宣言したのであれば下の
double M=q/ma; は
M = q/ma; で良いかと思います。

さらに細かい点を付け加えるならfor文でのカウンタ変数 i はint型にしておくといいと思います。
浮動小数点型はあくまで近似値として小数を扱うため、数値に誤差がありループによって積み重なると意図しない値を取る可能性があるからです。

[thkana]

回答としては、まずは、質問にある「エラーの原因と対処法」に集中するべきではありませんか。

それ以外にも問題があることの指摘も必要になってくるでしょうが、それに終始するのもいかがなものかと思います。

[kisara11235]

ありがとうございます。
教本を確認しながらもう一度作り直します。

[syalpon]

>>thkanaさん
仰る通りですね。
少しずれた回答になってしまって申し訳ございません。
言い訳にはなりますが、自分はがコンパイルエラーが指す原因のことよりも間違った文法の方でエラーが起きているものだと思い、エラー文の対処よりも先に文法の指摘を書いてしまいました。
例え話になるのですが自分の環境ですと文末の；を忘れた場合などはその直前の行にある処理でエラーが出たりするので、質問でのエラー文が直接その原因だとは思っていなかったということです。

改めてですが指摘くださりありがとうございます。
以後気をつけて回答させていただきます。

Answer 4

24、25，26で「 too few arguments to function call, expected 4, have 2」と出ます。

はメッセージの通りで、関数の定義では引数が4つになっていますが、呼び出している時に引数を2つしか渡していません。