Help with functions!!!!

I have three warnings on this program:

#include <iostream.h>

int get_month ();
int get_day ();
int get_year ();
int display (int,int,int);

void main ()
{
int dia, mes, anos;
dia=get_day ();
mes=get_month ();
anos=get_year ();
display(dia,mes,anos);
return;
}

int get_day ()
{
int i;
cout << "\nPlease enter day of brith ";
cin >> i;
return i;
}
int get_month ()
{
int hate;
cout << "\nPlease enter month of brith ";
cin >> hate;
return hate;
}

int get_year ()
{
int functions;
cout << "\nPlease enter year of birth ";
cin >> functions;
return functions;
}

int display (int dia, int mes, int anos)
{
cout << "";
return 0;
}


I cant find the problem, please help.

the fault is that u don't need a return if u have declared a void!

line 15 has a return - delete it and it will compile!

hope Im right!!

It compiles and runs fine for me as it is.

If you have a void function but don't place an expression after the return then you are essentially saying return void, which is legal in C++. However, void main() is a nonstandard function, the ANSI standard defines that main should be called as an int, like so:

Code:

---

int main(void){
    //code
    return 0;
}

---

The program does run however, once the year is entered the program stops. So I am thinking that there is something wrong with:
int display (int dia, int mes, int anos)
{
cout << "";
return 0;
}

I get the following warning:


Compiling YYYPG6_1.CPP:
Warning YYYPG6_1.CPP 50: Parameter 'dia' is never used in function display(int,int,int)
Warning YYYPG6_1.CPP 50: Parameter 'mes' is never used in function display(int,int,int)
Warning YYYPG6_1.CPP 50: Parameter 'anos' is never used in function display(int,int,int)
this part:

Well yeah...your passing those 3 params, but doing nothing with them...

The compiler issuiong warnings in that way is like it saying "what was the point of passing them?"

The program is working after you have the user enter the year. However, it works so fast that it displays it and closes the window immediately after. To combat this, include<stdlib.h> and type system("PAUSE") at the end of your program.

Code:

---

int display (int dia, int mes, int anos)
{
cout << "";
return 0;
}

---

You're not displaying anything. You want to cout dia, mes and anos, not "".

Thanx guys for the help. its finanlly running!!

Thanx a million!!!!
Taurusborn