expected initializer before 'extern' when using math.h

Question

The error points me to line 36 of the math.h file, which I haven't messed with. SRK.cpp is the only file that needs the header, but it won't be, so it seemed logical to include it in the header file. Clearly something is messed up (personally I expect there's a typo or other simple mistake in there, but since the error is pointing me to the header file included with C++ I don't know where to look).

header.h

#ifndef HEADER_H
#define HEADER_H
#include <math.h>
double reducedP(double P, double Pc);
double reducedT(double T, double Tc);
double SRK(double Tr, double Pr, double acc);
#endif

main.cpp

#include <iostream>
#include "header.h"
using namespace std;

int main()
{
    double T = 0;
    double Tc = 0;
    double Tr = 0;
    double P = 0;
    double Pc = 0;
    double Pr = 0;
    double acc = 0;
    double Z = 0;
    cout << "Enter Temperature: ";
    cin >> T;
    cout << "\n";
    cout << "Enter Pressure: ";
    cin >> P;
    cout << "\n";
    cout << "Enter  Critical Temperature: ";
    cin >> Tc;
    cout << "\n";
    cout << "Enter Critical Pressure: ";
    cin >> Pc;
    cout << "\n";
    Tr = reducedT(T,Tc);
    Pr = reducedP(P,Pc);
    cout << "Reduced T,P\t\t\t" << Tr << "\t\t" << Pr <<"\n";
    cout << "Enter accentric factor: ";
    cin >> acc;
    cout << "\n";
    Z = SRK(Tr, Pr, acc);
    cout << "Z is " << Z << "\n"; 
    return 0;
}

SRK.cpp

double SRK(double Tr, double Pr, double acc)
#include <math.h>
{
    double alpha;
    double phi = 1;
    double epsilon = 0;
    double omega = 0.08664;
    double psi = 0.42748;
    double Zc = 1/3;
    double a = (1+(0.480 + 1.574*acc - .176*acc*acc)*(1-sqrt(Tr)));
    alpha = pow(a,2);
    cout << "Alpha is " << alpha << "\n";
    double beta = omega*(Pr/Tr);
    cout << "beta is " << beta << "\n";
    double q = (psi*alpha)/(omega*Tr);
    cout << "q is " << q << "\n";
    double Z = 0;
    double test = 0;
    double Z_init = 1;
    while(fabs(Z_init-test)>.00001)
    {
        Z = 1 + beta - (q*beta)*((Z_init - beta)/((Z_init)*(Z_init+beta)));
        cout << "\n" << Z_init << "\n";
        test = Z_init;
        Z_init = Z;
    }
    return (Z);
}

Why are you putting #include <math.h> between the function and the body? — Mysticial
– Mysticial, Commented Apr 14, 2012 at 23:42
As a side note, use <cmath> instead of <math.h>. The latter is deprecated as of the former's release. — Qaz
– Qaz, Commented Apr 15, 2012 at 0:05

je4d · Accepted Answer · 2012-04-14 23:44:02Z

5

This:

double SRK(double Tr, double Pr, double acc)
#include <math.h>
{

is invalid. You can't put a #include of a system header file between a function signature and its body.

Change it to:

#include <math.h>
double SRK(double Tr, double Pr, double acc)
{

In general, your #include lines should go at the top of your .cpp file, before any of your code.

answered Apr 14, 2012 at 23:44

je4d

7,87835 silver badges46 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

tjtoml Over a year ago

Thanks, that got it. Sorry for the dumb question, I'm a bit new at this.

Diego Sevilla · Accepted Answer · 2012-04-14 23:42:16Z

2

In SRK.cpp you have to move the #include <math.h> to the beginning of the file. Also, in header.h you strictly don't need math.h, so you don't have to include it.

answered Apr 14, 2012 at 23:42

Diego Sevilla

29.1k4 gold badges62 silver badges91 bronze badges

Collectives™ on Stack Overflow

expected initializer before 'extern' when using math.h

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related