Local Function Definitions are illegal...very evil problem

Hi. This code is a portion of a whole program, and I get the error message stated in the thread title...but I can't seem to fix it, no matter what I do.

Here is the Code:

Code:

---

char User_Choice;
 void UserThree(char User_Choice){
        std::cout << "Do you like the number 3? (Yes=Y, No=N)\n";
                std::cin >> User_Choice;
                if {(User_Choice == y)
                        std::cout << "You are cool\n";
                }
                else {
                        if  (User_Choice == n)
                                std::cout << "You are Un-cool\n";
                }
                else {
                        if (User_Choice != y, n)
                                std::cout << "Invalid Decision" << endl;
                }


 }

---

Can someone out there point out what I should do?

Are you trying to define your function inside a function or inside main? That's a nono.

Ahh...this function is inside the main() function...should I move it out?

Yes, you cannot have a function definition within another.

yes -- move it out

first put ' around your characters

Code:

---

if  (User_Choice == 'n')

---

don't use else { if, use else if

Code:

---

else if  (User_Choice == 'n')

---

thirdly use && (AND) to make sure it's not one thing and the other.

Code:

---

else if (User_Choice != 'y' && User_Choice != 'n')
                std::cout << "Invalid Decision" << endl;

---

or you could just use else

Code:

---

else
 std::cout << "Invalid Decision" << endl;

---

edit also use endl instead of \n

Code:

---

std::cout << "Do you like the number 3? (Yes=Y, No=N)" << endl;

---

Code:

---

void UserThree(char User_Choice){
        char User_Choice;
        std::cout << "Do you like the number 3? (Yes=Y, No=N)" << endl;
        std::cin >> User_Choice;

        if (User_Choice == 'y') {
            std::cout << "You are cool\n";
        } else
        if  (User_Choice == 'n') {
                std::cout << "You are Un-cool\n";
        } else {
                std::cout << "Invalid Decision" << endl;
        }
}

---

Yep. Move it outside main. Then you can call it from inside main. If the function is defined after main, then be sure to use a prototype before main (or the first use of the function call).

When I move it outside of main() I get all kinds of errors, though.

post some of them please

Nevermind...It's working now, thanks all.

Actually, I am having a typedef error..."warning C4091: 'typedef ' : ignored on left of 'my_soundtrack' when no variable is declared"

My guess is that you're using typedef somewhere incorrectly. Can't say until you post more code.

I will post it later...after trying to use this new advice I messed up the whole file...

Try writing less code between pressing "compile", then you have a much better idea of what caused the new set of error messages to appear.