I feel stupid having to ask but.......

ok i have just started making a text-adventure and i already ran into a problem. I it so simple it is sad but i can figuer it out.
here is the code:

Code:

---

#include <iostream>
#include <stdlib.h>

int main(int argc, char *argv[])
{

cout<<"\t\t\t\tTHE";
cout<<"\t\t\t\t\t\t\t\t";
cout<<"\t\t  TEMPLE";
cout<<"\t\t\t\t\t\t\t\t";
cout<<"\t  OF";
cout<<"\t\t\t\t\t\t\t\t";
cout<<"\t\t    TYR";
cout<<"\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t1 : Play";
cout<<"\t\t\t\t\t\t\t\t\t2 : Exit";

int playorexit;

cin>> playorexit;
if(playorexit==1);
        {
        cout<<" Thanks for deciding to play!";
        }
if(playorexit==2);
        {
        system("PAUSE");
        return 0;
        }
                                                                                                          system("PAUSE");       
  return 0;
}

---

what happens is when the user selects 1 it says thanks for deciding to play, but when the user selects 2 it displays thanks for deciding to play as well as exiting. Can anybody suggest a solusion?

You don't want the semi-colons at the end of the lines with if().

You might also want to look up things like endl and flush. A damn site easier than all those tabs.

Check out the <iomanip> header as well.

Thanks guys, i know it was kinda a dumb question but i had to ask.

That code won't compile. You have this '#include <iostream>' which implies the newer headers which are inclosed in the namespace std. Yet you don't have a using statement, and you don't use std:: for your cout statements.

Place 'using namespace std;' (w/o the quotes) under the includes.

Hey, im new to c++ my self but i thought maybe i could help. :\
well here goes....

Code:

---

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

int main(int argc, char *argv[])
{
int playGame;

cout<<"\t\t\t\t THE\n";
cout<<"\t\t\t\tTEMPLE\n";
cout<<"\t\t\t\t OF\n";
cout<<"\t\t\t\t TYR\n\n";
cout<<"1: Play \n";
cout<<"2: Exit \n";
cout<<"--> ";
cin>>playGame;

    if (playGame == 1)
    {
        cout<<"Thanks for deciding to play!\n\n";
    }
    if (playGame == 2)
    {
        cout<<"Good bye\n\n";
        system ("PAUSE");
        return 0;
    }
system ("PAUSE");
return 0;
}

---

hope that helped.

That code compiles and runs with no errors. So what ´s the problem???

P.S. Don´t mixed old and new header files. Insteed of

Code:

---

#include <stdlib.h>

---

use

Code:

---

#include <cstdlib>

---

Quote:

---

That code compiles and runs with no errors. So what ´s the problem???

---

Yeah, his code ran and compiled but it didn't do what he intented it to do.

Quote:

---

Yeah, his code ran and compiled but it didn't do what he intented it to do.

---

Ops, i´ve confused your code with Ikurik. Sorry.

Your´s code compiles and runs with no problem then :).
P.S. Edit the post to use new header-files.

What will using the new header files do to the program. After the first or second reply that told me not to use a semicolon at the end of an if statement, everything compiled qnd worked fine

Quote:

---

Originally posted by Ikurik
What will using the new header files do to the program. After the first or second reply that told me not to use a semicolon at the end of an if statement, everything compiled qnd worked fine

---

follows c99 standard. newer compilers should give backwards warnings for deprecated headers.

All of the C++ Standard Library headers are available only 'without the .h' (i.e. <iostream>, <fstream>, <list>, etc).

The C Standard Library headers are available in both the old C form, and the new C++ form (the only difference being the C++ ones are in the std namespace). So, <stdlib.h>=<cstdlib>, <time.h>=<ctime>, etc.

Its preferable to use the new style because of the use of the std namespace. Both are valid, however.

Oh ok sorry about that, i didn't know about #include <cstdlib>
, im also pretty new to c++, are there any big differences? because im using devcpp 5 and it adds

#include <stdlib.h> automatically

Just the namespace. If you go into environment options though, you can specify what it automatically gives you.

Quote:

---

Originally posted by Zach L.
Just the namespace. If you go into environment options though, you can specify what it automatically gives you.

---

Cool, thx