Thread: Question about compilicated If Then's

In the code that you included above, you declared the variable 'directions' as an integer. When the compiler reaches the expression (directions == north), it's probably going to be looking for a variable called north (which doesn't exist).
For text such as "north" you need to use strings. For selecting from a list of options, I'd suggest that you use a single character for each option e.g. 'N' for north. Also, have you considered using a switch vice nested if...then statements?

Originally posted by drdroid
you could also use this function....

Code:

#include <iostream.h>
#include <string.h>
char dir[30];
cin >> dir;
if(strcmp(dir,"North"))
{
cout << "Follow Up";
}
else
{
cout << "Wrong!";
}

that would probably be the most effictive or if you just want to change your current code try this

Code:

#include <iostream.h>
int main();
{
    char directions;  //can't you use a char* for a string of unknown length?  I'm so out of practice....

    cout<<"What do you want to do?";

    cin>>directions;                                //takes only 1 char, "N" for ex.

***here you will want to clear the buffer, I don't remember how to do that***

if ( [directions=="N"] || [directions == "n")
{
   whatever
}

Code:

#include <iostream>
#include <conio.h>

using namespace std;

char ToLower(char);

int main()
{
	char directions;					//The character which will hold the direction
 	cout<<"What do you want to do?";	
 	cin>>directions;					//Get the user's direction (N) for north, etc
  	cin.ignore(1,'\n');					//Flush the input buffer
  	switch (ToLower(directions))
  	{
  	case 'n':
		cout<<"You pressed north.";
		break;
	case 's':
		cout<<"You pressed south.";
		break;
	case 'w':
		cout<<"You pressed west.";
		break;
	case 'e':
		cout<<"You pressed east.";
		break;
	}
	getch();
	return 0;
}

char ToLower(char InChar)
{
	if((InChar >= 'A') && (InChar <= 'Z'))
	{
		InChar += ('a' - 'A');
	}
	return InChar;
}