Can't get this to work.

Code:

---

#include <iostream>
using namespace std;

int main()
{
char personality[20];

cout <<"Enter personality: " <<endl;
cin.getline(personality,20);
if (personality[20] == 'mad')
    {
    cout <<"$$$$ the world!" <<endl;
    }
    else
        {
        cout <<"Lovely" <<endl;
        }
       
system("pause");
}

---

I know I could get this to work by doing a different way but that is not what I want. I want the user to input something wether it be, mad, sad, happy and then depending on which they TYPE display a different text. I don't want an options menu! Not looking for something like this: press M for mad or S for sad. I want to be able to feed off of the whole word.(If its possible)

Code:

---

#include <iostream>
using namespace std;

int main()
{
char personality[20];

cout <<"Enter personality: " <<endl;
cin.getline(personality,20);
if (personality[20]  == 'mad')
    {
    cout <<"$$$$ the world!" <<endl;
    }
        else
        {
        cout <<"Lovely" <<endl;
        }
       
system("pause");
}

---

should be:

Code:

---

#include <iostream>
using namespace std;

//#include <string>  //for alternate if statement below

int main()
{
    char personality[20];

    cout <<"Enter personality: " <<endl;
    cin.getline(personality,20,'\n');
    if (personality == "mad")  //or if(!strcmpi(personality,"mad");
        cout <<"$$$$ the world!" <<endl;
    else
        cout <<"Lovely" <<endl;
       
    system("pause");
    return 0;
}

---

I think you need to use strcmp instead of == in your comparisons.

Also I noticed you don't explicitly have a return, so you may want to add that though I don't think it's necessary from what I have read on this board.

forgot about the return... sometimes it is necessary, depending on what compiler your using... and I think the == will work, but strcmpi is better... or strcmp if you want it to be case-sensitive

Quote:

---

Originally posted by major_small

Code:

---

#include <iostream>
using namespace std;

int main()
{
char personality[20];

cout <<"Enter personality: " <<endl;
cin.getline(personality,20);
if (personality[20]  == 'mad')
    {
    cout <<"$$$$ the world!" <<endl;
    }
        else
        {
        cout <<"Lovely" <<endl;
        }
       
system("pause");
}

---

should be:

Code:

---

#include <iostream>
using namespace std;

//#include <string>  //for alternate if statement below

int main()
{
    char personality[20];

    cout <<"Enter personality: " <<endl;
    cin.getline(personality,20,'\n');
    if (personality == "mad")  //or if(!strcmpi(personality,"mad");
        cout <<"$$$$ the world!" <<endl;
    else
        cout <<"Lovely" <<endl;
       
    system("pause");
    return 0;
}

---

---

The above didn't work. :(

Code:

---

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

int main()
{
char personality[20];

cout <<"Enter personality: " <<endl;
cin.getline(personality,20,'\n'); //don't think '\n' is necessary
if(!strcmpi(personality,"mad"))
{
    cout <<"$$$$ the world!" <<endl;
}
else
{
    cout <<"Lovely" <<endl;
}

cout<<endl<<"You are "<<personality<<", in case you forgot."<<endl;

return 0;
}

---

Read major_small's comments.

I was about to say something about that look at my code:

Code:

---

#include <iostream>
using namespace std;

int main()
{
char personality[20];
char m;
cout <<"Enter" <<endl;
cin.getline(personality,20,'\n');
if (strcmp("mad",personality))
    {
    cout <<"Ruck off!" <<endl;
    }
    else
        {
        cout <<"Lovely" <<endl;
        }
       
system("pause");
}

---

I forgot the ! sign and was about to ask why it only would type Ruck off! for anything i typed besides mad. LOL.
Thanks funkydude!

Quote:

---

I forgot the ! sign

---

Copy & past is your friend. ;)

Quote:

---

Originally posted by major_small
forgot about the return... sometimes it is necessary, depending on what compiler your using... and I think the == will work, but strcmpi is better... or strcmp if you want it to be case-sensitive

---

Nope, the == operator only works on strings of the stl string datatype.

thanks for clearing that up for me golfinguy4

When you use == as you did, it compares the pointers. For example, 'personality == "mad" ' compares the base address of 'personality' with the base address of the string literal "mad".