Functions, what am I doing wrong here?

Code:

---

#include <iostream.h>
#include <stdio.h>
int main()
{
int name()
{
int nam;
cout<<"Enter your name: ";
cin>>nam;
}
int phn()
{
int phone;
cout<<"Enter the Phone number";
cin>>phone;
}
cout<<nam;
cout<<phone;
}

---

I just want it to compile and go into the other functions

Firt off you are declaring your functions in main. Which is no good. Fucntion declarations must be made outside of main. Secondly you are not calling your fucntions.

You can't define functions within another function, i.e. a local function. You will also need return the value, or use a global variable since the local variable you are declaring inside the finctions goes out of scope when the function exits.

Sorry that was a bit too fast paced for me could you simple it out?

Code:

---

#include <iostream.h>
#include <stdio.h>
int main()
{
int name()
{
int nam;
cout<<"Enter your name: ";
cin>>nam;
}
int phn()
{
int phone;
cout<<"Enter the Phone number";
cin>>phone;
}
cout<<nam;
cout<<phone;
}

---

should be.

Code:

---


#include <iostream>
#include <stdio>
using namespace std;


int name()
{
int nam;
cout<<"Enter your name: ";
cin>>nam;

return nam;

}
int phn()
{
int phone;
cout<<"Enter the Phone number";
cin>>phone;

return phone;

}

int main()

{
cout<<nam;
cout<<phone;
}

---

Also you declared name and phone as being intergers and they should be chars. :)

Quote:

---

Originally posted by mattbbx
Sorry that was a bit too fast paced for me could you simple it out?

---

I used to do that same thing. You can't define function within other functions.

Code:

---

int main()
{
void myFunction() // Can't do this.
{
cout<<"Hello";
}
return 0;
}

---

is an example of the improper way to do it. Instead:

Code:

---

void myFunction() {
cout<<"Hello";
}

int main()
{
myFunction();
return 0;
}

---

would be okay, though.

ahhhhhhhhhh i see, even a small thing matters
Thank you

RealityFusion, thanks but for some reason it wont compile it complains about the variables in main()

Ok i changed int to char but now the window flashes up real quick and I dont see anything

I know. I basically just plotted out the problems you are going to have even after you do what the other guy suggested though. :).
You also forgot to put

Code:

---

return 0;

---

at the end of our main fucntion.

"Ok i changed int to char but now the window flashes up real quick and I dont see anything"

That means your program ran, finished, and the dos window closed. To get the dos window to pause so you can see the output read this:

http://faq.cprogramming.com/cgi-bin/...&id=1043284385

My favorite way to get a pause is to use:

Code:

---

#include <iostream>

using namespace std;

int main()
{
    cin.ignore();        //this waits for a newline char
    return 0;
}

---

This is nice because it completely ignores whatever gets inputted.

Edit: Oh yeah. Code is sooo hard to read when it's not indented.

Quote:

---

Originally posted by RealityFusion
I know. I basically just plotted out the problems you are going to have even after you do what the other guy suggested though. :).
You also forgot to put

Code:

---

return 0;

---

at the end of our main fucntion.

---

Any ANSI-C++ 98 compiler does not require a return 0; at the end of main. In '98, not explicitly returning anything is an implied "return 0;". If your compiler complains, it's not following the C++ standard.

Wow! I didn't know that. Thanks for the info!