Can't fix errors

Hey all, this is probably something everyone will spot straight away but i've started writing my code and get some errors I can't get past...

error C2198: 'f_menu' : too few arguments for call
error C2198: 'f_display' : too few arguments for call

Heres my code

Code:

---

#include <stdio.h>
#include <stdlib.h>

typedef struct
{
        char id[5];
        char first_name[20];
        char surname[20];
        int mem_type[5];
        int expiry[5];
} DATABASE;

int f_menu (DATABASE);
DATABASE f_add();
void f_display(DATABASE);


void main()
{
        DATABASE members;

        int menu = 0;

        menu = f_menu ();

        do
        {

        switch (menu)
        {
        case 1 :
                f_add ();
                break;
        case 2 :
                f_display ();
                break;
        case 3 :
                printf("Exiting Program\n\n");
                break;
        }
} while (menu != 3);

}

int f_menu ()
{
        int menu = 0;

        printf("\n\nLeisure Center Menu\n");
        printf("  1. Add Members\n  2. Display Members\n  3. Exit\n ");
        printf("\nEnter choice: ");

        do
        {
                scanf("%d", &menu);
                fflush(stdin);

                if((menu < 1 ) || (menu > menu))
                {
                        printf("Invalid choice. Try again: ");
                }

        } while ((menu < 1 ) || (menu > menu));

        return (menu);
}

DATABASE f_add()
{
        DATABASE members;

        printf("Enter member ID : ");
        gets(members.id);

        printf("Enter first name : ");
        gets(members.first_name);

        printf("Enter surname : ");
        gets(members.surname);

        printf("Enter membership type : ");
        scanf(members.mem_type);
        fflush(stdin);

        printf("Enter memership expiry month (1-12) : ");
        scanf(members.expiry);
        fflush(stdin);

        return members;
}

void f_display(DATABASE members)
{

}

---

I'm a begginner as i'm sure you can all tell... so please keep it simple :confused:

Code:

---

menu = f_menu ();

---

you have not put any argument in the function call, but you declare the function as having DATABASE struct as an argument

Code:

---

int f_menu (DATABASE);

---

so you need to pass the struct in the call

same goes for f_display

Another problem is that your function definitions really do take no arguments at all. You need to make up your mind: should they take arguments or should they not?
After that, you have a lot more stuff to fix:
SourceForge.net: Do not remove parameter names - cpwiki
SourceForge.net: Gets - cpwiki
SourceForge.net: Fflush - cpwiki
You have also gotten the scanf wrong. It must take a string with a formatting that tells it how to to read the data. And you are also only passed along a variable to store the data. And you are also passing along integer arrays. I am thinking you might want to store the data inside ONE element inside the arrays, not the entire arrays.

I did what you guys said and it didnt change anything, I get the same errors...

So, what is your current code?

I been playing around with passing the struct in the function calls, but nothing I do fixes the errors, just makes more...

Code:

---

#include <stdio.h>
#include <stdlib.h>

typedef struct
{
        char id[5];
        char first_name[20];
        char surname[20];
        int mem_type[5];
        int expiry[5];
} DATABASE;

int f_menu (DATABASE);
DATABASE f_add(DATABASE);
void f_display(DATABASE);

void main()
{
       
        DATABASE members;

        int menu = 0;

        menu = f_menu (DATABASE);

        do
        {

        switch (menu)
        {
        case 1 :
                f_add (DATABASE members);
                break;
        case 2 :
                f_display ();
                break;
        case 3 :
                printf("Exiting Program\n\n");
                break;
        }
} while (menu != 3);

}

int f_menu (DATABASE)
{
        int menu = 0;

        printf("\n\nLeisure Center Menu\n");
        printf("  1. Add Members\n  2. Display Members\n  3. Exit\n ");
        printf("\nEnter choice: ");

        do
        {
                scanf("%d", &menu);
                fflush(stdin);

                if((menu < 1 ) || (menu > menu))
                {
                        printf("Invalid choice. Try again: ");
                }

        } while ((menu < 1 ) || (menu > menu));

        return (menu);
}

DATABASE f_add(DATABASE)
{
        printf("Enter member ID : ");
        gets(members.id);

        printf("Enter first name : ");
        gets(members.first_name);

        printf("Enter surname : ");
        gets(members.surname);

        printf("Enter membership type : ");
        scanf(members.mem_type);
        fflush(stdin);

        printf("Enter memership expiry month (1-12) : ");
        scanf(members.expiry);
        fflush(stdin);

        return members;
}

void f_display(DATABASE)
{

}

---

Let me take a peek at it. Back in a bit.

I took it all out and put it all back in and got this with 2 different errors but fixed the previous ones...

Code:

---

#include <stdio.h>
#include <stdlib.h>

typedef struct
{
        char id[5];
        char first_name[20];
        char surname[20];
        int mem_type[5];
        int expiry[5];
} DATABASE;

int f_menu ();
DATABASE f_add();
void f_display();

void main()
{
       
        DATABASE members;

        int menu = 0;

        menu = f_menu ();

        do
        {

        switch (menu)
        {
        case 1 :
                f_add (DATABASE members);
                break;
        case 2 :
                f_display ();
                break;
        case 3 :
                printf("Exiting Program\n\n");
                break;
        }
} while (menu != 3);

}

int f_menu ()
{
        int menu = 0;

        printf("\n\nLeisure Center Menu\n");
        printf("  1. Add Members\n  2. Display Members\n  3. Exit\n ");
        printf("\nEnter choice: ");

        do
        {
                scanf("%d", &menu);
                fflush(stdin);

                if((menu < 1 ) || (menu > menu))
                {
                        printf("Invalid choice. Try again: ");
                }

        } while ((menu < 1 ) || (menu > menu));

        return (menu);
}

DATABASE f_add( DATABASE members)
{
        printf("Enter member ID : ");
        gets(members.id);

        printf("Enter first name : ");
        gets(members.first_name);

        printf("Enter surname : ");
        gets(members.surname);

        printf("Enter membership type : ");
        scanf(members.mem_type);
        fflush(stdin);

        printf("Enter memership expiry month (1-12) : ");
        scanf(members.expiry);
        fflush(stdin);

        return members;
}

void f_display()
{

}

---

now I get:
error C2143: syntax error : missing ')' before 'type'
error C2059: syntax error : ')'

You have yet to fix what was pointed out to you.
Furthermore, the problem is not any type of logic, but rather syntax. You have a book nearby? You should look at function call syntax, or failing the having a book, look at a tutorial.

You're going about making this program, the wrong way.

What you want to do is start with the essential thing - your struct, and get it set up. Will one struct be enough to hold several members data?

No. So you need an array of these structs. Get that first.

Then start with ONE function, and get it working, then add another function, and get it working, etc.

Big mistake to start on several functions, get a lot of code done, and then find out OOPS! 90% of it is not right, and has to be thrown out, and re-done.

Here's your program with several suggestions in it. No, it does not work, these are suggestions to get rid of obvious errors, only.

Code:

---

#include <stdio.h>
#include <stdlib.h>

typedef struct
{
        char id[5];
        char first_name[20];
        char surname[20];
        int mem_type[5];
        int expiry[5];
} DATABASE;

int f_menu (void);
//DATABASE f_add(DATABASE);
void f_add(DATABASE members);
void f_display(DATABASE);

//void main()  //int main and with a return, please
int main()
{
       
        DATABASE members;

        int menu = 0;

        //menu = f_menu (DATABASE);
  //DATABASE is an alias for a struct, only.
  menu = f_menu();

        do
        {

        switch (menu)
        {
        case 1 :
                f_add (members);
                break;
        case 2 :
                f_display (members);
                break;
        case 3 :
                printf("Exiting Program\n\n");
                break;
        }
} while (menu != 3);

  menu = getchar();
  return 0;
}

//int f_menu (DATABASE members) You don't need members in f_menu, at all.
int f_menu()
{
        int menu = 0;

        printf("\n\nLeisure Center Menu\n");
        printf("  1. Add Members\n  2. Display Members\n  3. Exit\n ");
        printf("\nEnter choice: ");

        do
        {
                scanf("%d", &menu);
                //fflush(stdin); <==NO!
    /* You can't flush your kitchen sink, just your toilet */

                if((menu < 1 ) || (menu > menu))
                {
                        printf("Invalid choice. Try again: ");
                }

        } while ((menu < 1 ) || (menu > menu));

        return (menu);
}

//DATABASE f_add(DATABASE)
/*DATABASE is just an alias for a struct name. It's not "something"
  to be passed to any function. You want "members", instead.
*/
void f_add(DATABASE members)
{
        printf("Enter member ID : ");
        gets(members.id);

        printf("Enter first name : ");
        gets(members.first_name);

        printf("Enter surname : ");
        gets(members.surname);

        printf("Enter membership type : ");

  //NOW c'mon! :)
//        scanf(members.mem_type);
//        fflush(stdin); //quit trying to flush the kitchen sink!

        printf("Enter memership expiry month (1-12) : ");

  //Again!
//        scanf(members.expiry);
//        fflush(stdin);
  //no return needed
}

void f_display(DATABASE members)
{

}

---

Why don't you start with adding a very small members[] of say 4, and put in three names, initially. Then in the functions that change the members data, you need to pass the address to the members array.

The above gives no errors, but doesn't work either - it's just a step up.

First, read this.
http://cboard.cprogramming.com/c-pro...t-process.html
It's about not trying to write the whole thing in one go, then hoping the compiler will figure out what your 100+ line ramble actually means.

Second, read the FAQ on why gets() is bad, and fflush(stdin) is a tool of the devil.