C++ problems

It's a little unclear exactly what you want it to do. Perhaps you could write it in plain english, then we can see how the code differs from your intention. Also, read the forum FAQ for how to use src tags - makes it easier to read for people wanting to help.

To be more effective in your plea for help, you should:

1. Post only the relevant code.
2. Post the exact error messages or describe the specific problem very clearly and succinctly.
3. Show exactly where the errors or problems occur.
4. Use [ code ] [ / code ] or [ source ] [ / source ] tags.

Quote:

Original post by JohnBolton To be more effective in your plea for help, you should: Post only the relevant code. Post the exact error messages. Show exactly where the errors occur.

I think it's a logic problem, not a syntax one. I think he's still learning flow control.

There is so much wrong with this I really dont know where to start...

Include only takes a single file. If you want to include more than one, use multiple include's



Your code needs to be in a function. Presumably you want it all in your main(). Why do you have _tmain()? If thats really your entry point then substitute that for main in the following.

The general structure should be something like this:



If you make those changes I think you still have logic errors, but you should have code that compiles. Start with just the first menu, get that working then add the second menu. Also you might want to think about making seperate functions to display each of the menus. It will make your code a lot clearer.

Alan

#include "stdafx.h" <strike>int _tmain(int argc, _TCHAR* argv[]){   return 0;}</strike><font color=gold>You can't have both main() and _tmain()._tmain is a macro that is used for Unicode-aware programs.Don't bother with it for now.</font>#include <iostream>	<strike><windows.h></strike><font color=lime>#include <windows.h></font><font color=gold>You can't include several files with a single #include.Your current code probably doesn't need <windows.h> anyway.</font>using namespace std;int main() <strike>;</strike><font color=gold>If you put a semicolon there, you are declaring main, notdefining it. It means you are telling the compiler "there is afunction called 'main'." instead of telling it what the functioncontains.</font><font color=lime>{</font>   do    {      int input = <strike>' '</strike> <font color=lime>0</font>;      <font color=gold>' ' is a character, which corresponds to an integer value of 32.      Initializing your variable is laudable, but your choice of initial      value is strange.</font>      cout<<"1.Play Game\n";      cout<<"2.Play Multiplayer\n";      cout<<"3.Exit\n";      cin>> input;      cin.ignore(); <font color=gold>skip one character? Why?</font>      do       {         system("cls");         int two_input = <strike>' '</strike> <font color=lime>0</font>;         cout<<"Select!\n";         cout<<"1.New Game\n";         cout<<"2.Load Game\n";         cout<<"3.Return to Start\n";	 cin>> two_input;         cin.ignore();      } while ( <font color=lime>two_</font>input == 1 );      <font color=gold>You've got to make sure the loop's code can affect the loop      condition. If you read into two_input but check input, nothing      good will happen.</font>   } while ( <strike>two_</strike>input == 3);<font color=lime>}</font> <font color=gold>end of main</font>