help me

im tierd, and making mistakes, i cant see the error in my header file...please help.


header

Code:

 
#include <stdio.h>
#define labor 0.35
#define tax .085
int main (void);
void  getnum (float *le, float *wi, float *di, float *co);
void calc1(float ln, float wd, float ds, float cs, float *area, float
*carpet, float *labor, float *install, float *discount, float *subt, float *total);
void calc2(float len, float wid, float cos, float *are, float *car, float *lab, float *ins);
void calc3(float carp, float labo, float inst, float dcs, float *disc, float *subto);
void calc4(float subs, float *totalf);

main file

Code:

#include "my.h"

int main (void)

{
// Local Declarations
float l;
float w;
float d;
float c;
float ar;
float ca;
float la;
float in;
float dis;
float su;
float to;
//Statements

//Get the input form the user
getnum(&l,&w,&d,&c);
calc1(l,w,d,c,&ar,&ca,&la,&in,&dis,&su,&to);
print1(l,w,d,c,ar,ca,la,in,dis,su,to);
return 0;
}

error

Code:

-bash-3.00$ gcc -c main.c
In file included from main.c:1:
my.h:7: error: syntax error before numeric constant

its probably obvious to you guys

warthog

Use header guards in your header.

Code:

#ifndef MY_HEADER_H
#define MY_HEADER_H
/* Your code here */
#endif

Your problem is you're defining labor as a numeric constant (0.35) and later in your calc1() function you declare a pointer as labor. When the preprocessor is done with this you now have

Code:

float *0.35

Rename the variable.

thanx man,

stupid mistake on my part

A little tip for debugging manually... when you have a big long line like that full of declarations. Break the error
line down into seperate lines then compile again to get a more clear look at the error.

Code:

void calc1(float ln, float wd, float ds, float cs, float *area, float
*carpet, float *labor, float *install, float *discount, float *subt, float *total);  /*Here is the error */

Code:

void calc1(float ln, float wd, float ds, float cs, float *area, float *carpet, 
float *labor,  /* Here is the error */
float *install, 
float *discount, 
float *subt, 
float *total);

...and/or adopt the convention of using uppercase for macros.

ok, ive compiled the program, and in the assignment, we are not supposed to prompt the user. Instead, we save numbers in a file called "mydata"

then when running the program we type in ./a.out<mydata

and its is supposed to crunch the numbers

instead im getting segmintation fault

warthog

Any number of things could have went wrong, but you need to rewrite main() to accept command line arguments, and then write the file you want to use.

i dont know how to make main accept post line arguments

how do i do this

heres the get data code

[code]

#include "my.h"
void getnum (float* le, float* wi, float* di, float* co)
{

//local declarations
float len;
float wid;
float dis;
float cos;

//Statements

scanf("%f %f %f %f\n",len,wid,dis,cos);

return;
}

[\code]

Your code tag syntax is wrong. It should look like this:

Code:

// first line of code

// last line of code

The outcome would look like this:

Code:

printf("Created in code tag format

[/CODE] not [\CODE].

Originally Posted by swgh

Your code tag syntax is wrong. It should look like this:

Code:

// first line of code

// last line of code

The outcome would look like this:

Code:

printf("Created in code tag format

im still not exactly sure what you are saying.....
do i put that in main.c?