switch/case & functions

When I run this (trying to use the tutorials) I get a 'parse error before if' for line 30 (see notes in code). I can't figure out why!

Code:

---

#include <iostream.h>       
#include <conio.h>       
int playgame(int a);
int loadgame(int b);
int multiplayer(int c);
int main()
{         
  int input;         
  cout<<"1. Play game"<<endl;
  cout<<"2. Load game"<<endl;
  cout<<"3. Play multiplayer"<<endl;
  cout<<"4. Exit"<<endl;
  cin>>input;       
  switch (input)       
  {         
  case 1: playgame(1);
          break;       
  case 2:
          loadgame(2);
        break;       
  case 3:
        multiplayer(3);
          break;       
  case 4:
        return 0; 
  default:         
        cout<<"Error, bad input, quitting";       
  }
}
  int playgame(if (a==1)  //parse error before 'if'
                  {
                  <<cout"Game loading...";
                  })
  int loadgame(if (b==2)
                  {
                  <<cout"Loading saved game...";
                  })
  int multiplayer(if (c==3)
                  {
                  <<cout"Multiplayer game loading...";
                  })
{
  return 0;
}

---

look at your curly braces. you close up the main function before the if.

I took those out and I'm still getting the same 'parse error before if'

The reason I had closed off the main function was because the example for functions in tutorials had done that.

Thanks for replying so quickly :)

int playgame( <-before the if. Obviously bad.

wait, I'm looking down in that section of code and it is completely wrong. Are you trying to do functions or what?

by doing this:

Code:

---

int playgame(int a);
int loadgame(int b);
int multiplayer(int c);

---

You have set three functions, each taking a single int variable. Since the functions are of type int, they should return an integer as well.

by doing the following:

Code:

---

playgame(1);

---

you are making a euqal to 1 and starting the function. But Your function is not defined anywhere...the whole thing sort of beats the purpose...I would have something like this:

Code:

---

case 1:
    cout << "Game Loading...";
    playgame(); //and this function will start the game and so on

---

You are making it more difficult then it really is,

axon

Code:

---

#include <iostream.h>       
#include <conio.h>       
void playgame(int a);
void loadgame(int b);
void multiplayer(int c);
int main()
{         
  int input;         
  cout<<"1. Play game"<<endl;
  cout<<"2. Load game"<<endl;
  cout<<"3. Play multiplayer"<<endl;
  cout<<"4. Exit"<<endl;
  cin>>input;       
  switch (input)       
  {         
  case 1:
    playgame(1);
          break;       
  case 2:
          loadgame(2);
        break;       
  case 3:
        multiplayer(3);
          break;       
  case 4:
        return 0; 
  default:         
        cout<<"Error, bad input, quitting"<< endl;       
  }
}
void playgame(int a)
      {
      if (a==1)  //parse error before 'if'
            {
            cout << "Game loading..." endl;
            }
      }
void loadgame(int b)
      {
      if (b==2)
            {
            cout << "Loading saved game..." << endl;
            }
      }
void multiplayer(int c)
  {
  if (c==3)
          {
          cout <<"Multiplayer game loading..." << endl;
        }
  }

---

Code:

---

  int playgame(if (a==1)  //parse error before 'if'
                  {
                  <<cout"Game loading...";
                  })
  int loadgame(if (b==2)
                  {
                  <<cout"Loading saved game...";
                  })
  int multiplayer(if (c==3)
                  {
                  <<cout"Multiplayer game loading...";
                  })

---

Code:

---

  int playgame(int a)
  {
                  cout << "Game loading..." << endl;
  }
  int loadgame(int b)
  {
                  cout << "Loading saved game..." << endl;
  }
  int multiplayer(int c)
  {
                  cout<< "Multiplayer game loading..." << endl;
  }

---

Thanks for all the help, I appreciate it :)

Torsin

An if-statement is not an expression. It doesn't evaluate to anything, or return anything.

This means that you can't write: x = if(A==B0);