if statements and strings

[MisterMan]

I'm fairly new to the C++ language, and I've been having difficulty with user-input in relation to if statements.

#include <iostream>
#include <string>
#include <sstream>
using namespace std;
int main ()
{ cout << "What is your favourite colour? ";
string colour;
cin >> colour;
cout << "That's fantastic! "; cout << colour; cout << " is my favourite colour, as well!";
cout << "Are you an "; int adventurer; cout << "adventurer, "; int politician; cout << "policitian, or "; int dreamer; cout << "dreamer?";
string mystring;
cin >> mystring;
if (mystring == adventurer)
   cout << "That's great!";
return 0;
}

I can't seem to get the if statement to function properly. I get the error message
-- C:\aCodeBlocks\Untitled4.cpp|14|error: could not convert 'mystring' from 'std::string {aka std::basic_string<char>}' to 'bool'| --
and I'm hopelessly stuck. Any help would be appreciated. Thank you!

[pogrady]

You are comparing a string to an integer, but i think you intend to compare it to "adventurer".

if (mystring.compare("adventurer"))
   cout << "That's great!";

[rcast]

Also line 8 and 12 cin >> colour; and cin >> mystring; attempts to read a string with cin. This is ok, until the user enters something like "light blue" - in which case cin will stop reading at the first whitespace between light and blue, only reading light in the variable colour. To avoid this from happening, while accepting input from user that could possibly have a space, use:

getline(cin, colour);

to read the enter string before the carriage or return (enter) symbol is found.

[MisterMan]

Thank you! That worked just fine, however...
...whenever I type in "adventurer" for the input prompt on the second question, I don't get anything.
When I type in anything other than adventurer, I get the intended response of "That's great!".

I appreciate the advice greatly, but do you know how to solve this problem? I would like for the user, when they input either "adventurer", "dreamer", or "politician" to get a response corresponding to their respective input. When I tell the user to use numbers (i.e. "1 for adventurer, 2 for politician, etc.) it works, but I'd prefer the use of non-numerical characters to the numbers. Is there any way around this, or perhaps I'm once again missing something?

[jefw123]

Try this:

Change
if (mystring == adventurer)

to

if (mystring == "adventurer")

Does that work now?

Here is the whole program and it works for me...

#include <iostream>
#include <string>
#include <sstream>
using namespace std;
int main ()
{ cout << "What is your favourite colour? ";
string colour;
cin >> colour;
cout << "That's fantastic! "; cout << colour; cout << " is my favourite colour, as well!";
cout << "Are you an "; int adventurer; cout << "adventurer, "; int politician; cout << "policitian, or "; int dreamer; cout << "dreamer?";
string mystring;
cin >> mystring;
if (mystring == "adventurer")
   cout << "That's great!";
return 0;
}

[Chervil]

That's the way the compare() method works. When the strings are equal, it returns zero. And zero as a boolean value means false.
http://www.cplusplus.com/reference/string/string/compare/

So you have to put this:

if (!mystring.compare("adventurer"))
   cout << "That's great!";

or perhaps easier to read,

if (mystring == "adventurer")
   cout << "That's great!";

[MisterMan]

Thank you very much, Chervil and Jefw! Works like a charm now. I'm an amateur when it comes to this language, so I appreciate the help a whole lot.

[jefw123]

No problem buddy :) Best of luck!