Modifying a program to use a function

[cnoviceirish]

Hey guys and gals, I just started to use functions and I'm stuck on a question, the question is as follows,
Q. Based on program a (guess the word program),modify the program so that it uses a function to wait and read the user's input.

here is the code for a, this program works fine

#include <string.h>
#include <stdio.h>

int main(void)
{
    const char SECRET[]="pink";
    char guess[20];
    int x;
    
    for (x = 0; ((guess != "pink") && (x < 3)); x = x + 1)
    {
    printf("Guess the secret word\n");
    scanf("%s",guess);
    printf("%s\n Is incorrect, please try again\n");
    
    if(strcmp(SECRET,guess)==0)
    {
        printf("Correct guess - well done\n");
        break;
        
    }
    }    
return 0;
}

And here is my attempt

#include <string.h>
#include <stdio.h>

void get_user_input(char*);
int main(void)
{
    const char SECRET[]="pink";
    int x;
    get_user_input(&SECRET);
    return 0;
}
    get_user_input(char* guess)
{    
    
    for (x = 0; ((guess != "pink") && (x < 3)); x = x + 1)
    {
    printf("Guess the secret word\n");
    scanf("%s",guess);
    printf("%s\n Is incorrect, please try again\n");
    
    if(strcmp(SECRET,guess)==0)
    {
        printf("Correct guess - well done\n");
        break;
        
    }
}

This doesn't work for me and I'm not really sure why, can anyone give me some pointers, thanks in advance

[std10093]

Maybe you should start learning to read what the compiler says to you

I guess it will say that x is undeclared in your function. So you have to declare it. The x in the main function is out of scope for your function.
Also the call from the main should be without &, since SECRET is an array

[Malcolm McLean]

You haven't really understood what the question is getting at.

get_user_input() should prompt the user for input, then return it. It shouldn't do anything else. The loop and the test for accuracy should still be in main().