Error expected ) while passing a pointer member of a structure to a function

Question

I am a beginner to C. In my program i have a structure and a function. I'm trying to pass a pointer which is present in the structure as an argument in my function. But it is showing the error "Expected )" at the dot operator. This is confusing because the other arguments of my function are from the structure as well but this error is not seen in those.

I tried changing the return type of the function to all types and still nothing.

struct signal
{
bool *input;
int previousop;
int n;
}s; //my structure
void noiseremove(bool *input, int n, int count1, int count0, bool 
previousop)//function i declared and defined before main function
{//my function here}
void main()
{
void noiseremove(bool *s.input , int s.n, int s.count1, int s.count0, bool 
s.previousop); //this is where i call the function and facing an error at 
*s.input 
}

I'm not sure where i'm going wrong here or if the syntax is wrong. I expect the function to accept the parameters but it isn't.

"I'm not sure where i'm going wrong here" - `void noiseremove(bool *s.input , int s.n, int s.count1, int s.count0, bool s.previousop);` is wrong. And that isn't a function call; that's an improperly constructed prototype. If the syntax is wrong, it won't compile (and this won't). If the logic is wrong (but the syntax is right, which this isn't) it either won't run correctly, or at worst, will *seem* to run correctly but only because undefined behavior is just that; undefined. Go to your book/tutorial and look up how to call functions. — WhozCraig, Jan 23 '19 at 03:43
while calling the function , it shoud be like - noiseremove(s.input , s.n, s.count1, s.count0, s.previousop); should not define the argument types etc while calling a function. — Balu, Jan 23 '19 at 05:38

Ruks · Answer 1 · 2019-01-23T04:34:41.287

Having functions inside another function is not possible in C...

So, your code should somewhat look like this:

struct signal
{
    bool *input, previousop;
    int n, count0, count1;
} s;

void noiseremove(bool *input, int n, int count1, int count0, bool previousop)
{
    /* Try using multi-line comments since single-line comments can comment out the end
       braces as well...*/
}

void main()
{
    /* Initialize the structure before accessing any of its variables, or it will lead
       to undefined behavior! */
    s = {0};
    /* Don't declare the identifiers again... It is 'syntax error' and 
       your calling convention doesn't in the least look like a calling 
       convention but more like a function declaration with invalid identifiers
       for the parameters... */
    noiseremove(s.input , s.n, s.count1, s.count0, s.previousop);
}

score 1 · Answer 2 · answered Jan 23 '19 at 03:55

struct signal
{
    bool *input, previousop;
    int n, count1, count0;
} s; //my structure

void noiseremove(bool *input, int n, int count1, int count0, bool previousop) //function i declared and defined before main function
{
    // my function here
}
void main()
{
    // this should compile but s has not been initialized
    noiseremove(s.input , s.n, s.count1, s.count0, s.previousop);
    // not sure what you were going for here
    //*s.input 
}

Error expected ) while passing a pointer member of a structure to a function

2 Answers2