cin.get(); doesn't work even with cin.ignore();

Looks good.

A couple extra things to consider if you want. First, I prefer to start with a completely empty file. I don't like that Dev-C++ starts you off with non-standard code, but I also think its better to just understand what you need and put it in when you need it. For example, you don't need <stdlib.h> in that program. You do need <iostream> for cout and cin, and you do need <string> for the string class. If you understand that, then for the next program you write you can decide which you need and only use those.

A second thing is to work on your indenting (unless the forum software messed it up). Each time you add a new scope (basically in between your braces for the main function and the ifs and else ifs) you should indent the code four more spaces than the previous line. When the scope ends at the closing brace, go back to the previous indentation level. For example:

Code:

    if ( age <= 12 && age > 0) {
        cout<<"You are a child!\n";
    }

Originally Posted by calumn

I used to have the && but someone on msn told me to change it to and and it still worked.
I have changed the and back into && but it still gives me the exact same errors

yeah, my mistake, I forgot that worked in C++ too... and don't listen to what people on msn tell you and don't read the msdn... evil lurks there.

To change what Dev-C++ shows:

1. Go to options
2. Click on Environment Options
3. Select the Misc. tab
4. Change the code and press ok

See they keep saying use && because its good programming practice and would sometimes bail half way out and you might get something wrong outputed.

"and" is only there because some people in the remote mountains of Tahiti may not have "&" symbols on their typewriters. IMO you should refrain from using it whenever possible, as that is rather non-standard coding practice.

Ok I wont use "and again.
Do I need

Code:

using namespace std;

at the start of every thing I make because It says to put it there in the tutorials but Dev C++ didn't have it there.
I have added it anyway
The program works fine with and without using namespace std; at the top. I kept it there anyway.
This is my updated code and it tells you if you are an adult or child etc. even if you enter something like -22.

Code:

#include <iostream>
#include <string>

using namespace std;

int main()
{
 cout<<"Hello\n";
cout<<"Please enter your first name and press enter\n";

string name;

cin>> name;
cin.ignore();
cout<<"Now enter your age\n";

int age;

cin>> age;
cin.ignore();
cout<<"So you are called "<< name <<" and you are "<< age <<" years old\n";
if ( age <= 12 && age > 2) {
    cout<<"You are a child\n";
}
else if ( age >= 0 && age <= 2) {
    cout<<"You are a baby\n";
}
else if ( age >= 13 && age <= 19) {
    cout<<"You are a teenager\n";
}
else if ( age > 19 && age <= 55) {
    cout<<"You are an adult\n";
}
else if ( age >= 65 && age < 100) {
    cout<<"You are and OAP\n";
}
else if ( age < 0) {
    cout<<"You haven't been born yet\n";
}
else if (age >= 100) {
    cout<<"You are antique\n";
}
cin.get();
return 0;
}

I also indented it before cout.
I am going to try and make it so that it asks you if the details are right and if you say yes if syas something like OK but if you say no it goes back to asking you your name.

Is it possible to use words inside and if statements
Say I made a string called right and then it asked the user is the information was right or wrong and then there was an if statement that said if the variable was == to yes then it would say ok but if the variable was == no it would say Oh well then. Sfter I got that to work I would add loops so that it would go back to the start if you said no.

Code:

cout<<"Is this right?\n";
string right;
cin>> right;
cin.ignore();

if ( right == yes) {
    cout<<"OK\n";
}
else if ( right == no) {
     cout<<"Thats good then\n";
}
cin.get();

I don't think it will work and I keep on getting errors while compiling:
c:\docume~1\neilson\mydocu~1\progra~2\c__~1\abouty ~1.cpp: In function `int main()':
c:\docume~1\neilson\mydocu~1\progra~2\c__~1\abouty ~1.cpp:48: `yes' undeclared (first use this function)
c:\docume~1\neilson\mydocu~1\progra~2\c__~1\abouty ~1.cpp:48: (Each undeclared identifier is reported only once
c:\docume~1\neilson\mydocu~1\progra~2\c__~1\abouty ~1.cpp:48: for each function it appears in.)
c:\docume~1\neilson\mydocu~1\progra~2\c__~1\abouty ~1.cpp:51: `no' undeclared (first use this function)
I think that it think that no and yes are a function.
Have I just done it wrong and is there a better way of doing it.
Thanks for all your help

You can do this, but you must first understand that there are rules
that the compiler uses to differentiate between variables and
literal text. The compiler errors are telling you that the compiler
thinks that yes and no are variables or a functions.

To make it do the comparison, put those words in quotes, "yes"
for example. Then your program will work because the compiler
knows it is text, refered to as a string literal.

Just an early warning though, this works fine for C++ strings,
but you will (sooner or later) come across C style strings - less
fancy and powerful. To do comparisons on those you need the
function strcmp (). No need to worry about it yet though.

Thanks
I have changed it to that and it works fine.
I have made it also ask you if it is right and it can tell if you are not telling the truth about your age.
This is the updated code
What do you think I should try to add next

Code:

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

using namespace std;

int main()
{
cout<<"Hello\n";
cout<<"Please enter your first name and press enter\n";

string name;

cin>> name;
//Clear screen:

system("cls");
ageenter: /*Makes ageenter*/
cout<<"Now enter your age\n";

int age;

cin>> age;
system("cls"); /*Clears the screen*/
if (age < 1 ) {
    cout << name << ", you are not " << age << "\n";
goto ageenter; /*goes to ageenter*/
}
else if (age > 124 ) {
    cout << name << ", you are not " << age << "\n";
goto ageenter;
}
else
{
cout<<"So you are called "<< name <<" and you are "<< age <<" years old\n";

if ( age < 12 && age > 0 ) {
    cout<<"You are a child!\n";
}
else if ( age >= 13 && age <= 19) {
    cout<<"You are a teenager\n";
}
else if ( age > 19 && age <= 55) {
    cout<<"You are an adult\n";
}
else
{
    cout<<"You are elderly";
}
}

cout<<"Is this right?\n";
string right;
cin>> right;
if (right == "yes") {
    cout<<"OK\n";
    system("pause");
}
else if (right == "no") {
goto ageenter;
}

return 0;
}

I am trying to make it open in full screen.
I don't know how to do this but I asked one of my friends said you have to make it so that the program thinks that the user is pressing alt-enter by doing this:
Add the windows.h library and then add:

Code:

keybd_event(VK_MENU, MapVirtualKey(VK_MENU, 0), 0, 0); 
    keybd_event(VK_RETURN, MapVirtualKey(VK_RETURN, 0), 0, 0); 
    keybd_event(VK_RETURN, MapVirtualKey(VK_RETURN, 0), KEYEVENTF_KEYUP, 0); 
    keybd_event(VK_MENU, MapVirtualKey(VK_MENU, 0), KEYEVENTF_KEYUP, 0);

Is it hard to make them so that they don't run in DOS/command prompt and run in a windows sort of thing if you know what I mean or is that way too hard.

Program works well, just a few tips though

1) The use of goto is "frowned upon" (thats the wording i see a
lot), basically due to the visual and logical effect that it has on
your code - programmers don't like to go looking for labels, and
repeated use leads to code that is difficult to follow and can cause
debugging issues (not from personal experience, I've never used
goto). Nevertheless, it seems to be very natural to beginners,
so that i suggest that the next couple of programs you write,
use goto and get a feel for controlling the flow of your program.
Then learn about loops - the more clearer and definite way to
achieve program flow control. Here's a link about them.

2) You don't need stdio.h for this program - thats a C header
anyway. Also, as for stdlib.h (also C), the C++ equivalent is
cstdlib (no .h). Either will work in this program, but just so you
know.

3) As for opening a console in full screen, yes Virtual Keyboard
(VK) is one way of doing it for you, but there is probably an
actual function in the windows header that allows you to do
this. I don't know any off the top of my head because I haven't
done any "windows programming," but perhaps someone else
can suggest a method. Nevertheless, I'd like to make the point
that since you are a beginner, messing with VK and windows
programming features is ill advised, because you are learning
code that is very system specific. You can do loads of things with
VK but that doesn't mean there isn't a simpler and better way to
do it with standard code. My advice is to hold back on
implementing such a feature.

4) Making it look like a windows dialog box is much much harder,
and with what i said in 3 above, you really should wait a good
while before you mingle with windows API programming.

5) Code indentation has been mentioned earlier on, but I thought
I'd reiterate here.

The next steps your program could make would be to allow the
user to enter yes or no in any case (i.e. YeS). Take a look at
a function called tolower or toupper, or compare to every
permutation of the letters (inefficient but it can be done). I won't
give you a specific way of doing this, so you can do some
investigations and find out for yourself. Nonetheless, if you
decide to try it and run into problems, just call back here and you
can get help.

Also I'm guessing that you haven't learned about function yet
(such as writing your own, passing data to and from them etc.)
Take a look at this as well.

I think that I have over complicted things with my loops.
I am getting 3 errors right now.

Code:

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

using namespace std;

int main()
{
cout<<"Hello\n";
cout<<"Please enter your first name and press enter\n";

string name;

cin>> name;
//Clear screen:

system("cls");

cout<<"Now enter your age\n";

int age;

cin>> age;
system("cls"); /*Clears the screen*/
if (age < 1 ) {
    cout << name << ", you are not " << age << "\n";

}
else if (age > 124 ) {
    cout << name << ", you are not " << age << "\n";

}
else
{
cout<<"So you are called "<< name <<" and you are "<< age <<" years old\n";

if ( age < 12 && age > 0 ) {
    cout<<"You are a child!\n";
}
else if ( age >= 13 && age <= 19) {
    cout<<"You are a teenager\n";
}
else if ( age > 19 && age <= 55) {
    cout<<"You are an adult\n";
}
else
{
    cout<<"You are elderly\n";
}
}

cout<<"Is this right?\n";
string right;
cin>> right;

if (right == "no") {
while ( right == "no" ) {
    cout<<"Enter your age\n";
    cin>> age;
    system("cls"); /*Clears the screen*/
if (age < 1 ) {
    cout << name << ", you are not " << age << "\n";

}
else if (age > 124 ) {
    cout << name << ", you are not " << age << "\n";

}
else
{
    cout<<"So you are called "<< name <<" and you are "<< age <<" years old\n";

if ( age < 12 && age > 0 ) {
    cout<<"You are a child!\n";
}
else if ( age >= 13 && age <= 19) {
    cout<<"You are a teenager\n";
}
else if ( age > 19 && age <= 55) {
    cout<<"You are an adult\n";
}
else
{
    cout<<"You are elderly\n";
}
}
}
else if ( right == "yes") {
    cout<<"Ok, Press enter or Esc to exit\n";
    cin.get()
}
}
return 0;
}

Code:

if (right == "no") { /* Not necessary */
while ( right == "no" ) {

Yep, you've got a lot of syntax errors, I can tell just by running it through a syntax checker. However, I don't think it's the fault of adding a loop. You really need to adopt a cleaner coding style that is easier to read.

Code:

int main (void)
{
    if (condition) 
    {
        do stuff;
        do this too;
        and this;
    }
    else 
    {
        try this instead;
    }
    return 0;
}

In my psuedo-code you can tell by eyeballing where one code block ends and another begins.

You had a lot of code duplication there, no need. Here is your
code tweaked around a little and reformatted to be easier to
read:

Code:

#include <iostream>
#include <string>
#include <cstdlib>

using namespace std;

int main(){

	string right = "no";

	cout<<"Hello\n";

	while (right == "no"){
		cout<<"Please enter your first name and press enter\n";

		string name;
		cin>> name;
		system("cls");

		cout<<"Now enter your age\n";
		int age;
		cin>> age;

		system("cls");

		if (age < 1 ) {
			cout << name << ", you are not " << age << "\n";
		}

		else if (age > 124 ) {
			cout << name << ", you are not " << age << "\n";
		}

		else{
			cout<<"So you are called "<< name <<" and you are "<< age <<" years old\n";

			if ( age < 12 && age > 0 ) {
				cout<<"You are a child!\n";
			}

			else if ( age >= 13 && age <= 19) {
				cout<<"You are a teenager\n";
			}

			else if ( age > 19 && age <= 55) {
				cout<<"You are an adult\n";
			}

			else{
				cout<<"You are elderly\n";
			}
		}

		cout<<"Is this right?\n";
		cin>> right;

		while ((right != "yes") && (right != "no")){

			if ((right != "yes") && (right != "no")){
				cout<<"Please enter yes or no\n";
				cout<<"Is this right?\n";
				cin>> right;
			}
		}
	}

    cout<<"Ok, Press enter or Esc to exit\n";
    cin.get();

	return 0;
}

This should work for you, and I added a check to see whether
yes or no was not entered. The code should be fairly
straightforward to you now.

Thanks for helping me again.
I will try to write the code in a neater style and in all languages everyone knows that i am very messy. Thanks for the help. I see what you have done because at first I was wondering how it knew that the string called right was no but then I saw it being made at the very start.