简单的 C++ 程序；似乎无法正确

Question

#include<iostream>
#include<conio>
#include<string>

using namespace std;

class CompanyName
{
public:

    char name[10];

    void display()
    {
        if(name=="emily")
            cout<<"Hired!"<<endl;
    }
};

void main()
{
    CompanyName c;

    cout<<"enter name"<<endl;
    cin>>c.name;

    c.display();

    getch();
}

所以我的朋友刚刚被安置在一家公司，我想通过一个小型 CPP 程序在她的 Facebook 墙上祝贺她（并宣布某某公司雇用了她）。我不想只打印“恭喜艾米丽！”因为我已经这样做了。所以这一次你只要看代码就可以看到我想要做什么。任何更好的建议都非常受欢迎。 但我的问题是，我无法运行此代码。我觉得'if'线不对。有人可以为我更正代码吗？我还是个初学者。

Answer 1

您将name 声明为char name[10];，然后name 将被视为指针。你的指针永远不会等于字符串"emily"。

解决方案：强烈建议简单地使用来自#include <string> 的伟大std::string

这应该可以工作（因为您已经包含了该库）：

class CompanyName
{
public:
    std::string name;

    // ..
        if (name == "emily")
           // cout
    // ..
};

Answer 2

你不能只使用==操作符比较name，因为它是一个char数组，你应该使用strcmp(name, "emily") == 0

更好的是，使用std::string 类来存储和操作字符串，使用std::string，您可以简单地使用== 运算符进行比较。

另一件事：你不应该使用getch()，因为它不是标准的c++。

Answer 3

你知道，如果你使用std::string，这段代码会更容易。请看下面的代码：

#include <iostream>
#include <string>

using namespace std;

class CompanyName {
public:
    string name;

    void display()
    {
        if (name == "emily")
            cout << "Hired!" << endl;
    }
};

int main()
{
    CompanyName c;

    cout << "enter name" << endl;
    cin >> c.name;

    c.display();
    return 0;
}

那么这里对字符串的大小没有限制。顺便说一句，你的代码不起作用，因为它只能包含 9 个字符，+\0，这意味着你的字符串太长了。使用上面的解决方案。 Live Example

Answer 4

在 C++ 中直接比较字符数组和字符串是不安全的。正如其他人建议的那样，尝试使用std::string，它将为您处理所有背景内容。

这是一个使用类的析构函数的有趣示例：

#include<iostream>
#include<string>

using namespace std;

class CompanyName
{
public:
    std::string name;
    ~CompanyName()
    {
        if (name == "emily")
            cout << "Hired!" << endl;
        else cout << "Who's that?" << endl;
    }
};

int main()
{
    CompanyName c;
    cout << "Enter new employee name" << endl;
    cin >> c.name;
    return EXIT_SUCCESS;
}

在线编译：http://cpp.sh/433h7