在if语句中将多个字符变量转换为数字

Question

我正在尝试使用“if”语句将用户输入的单词（黑色、红色、绿色、蓝色）转换为整数（1、2、3、4）。编程很新，所以欢迎任何解释。下面的代码告诉我“与字符串文字比较会导致未指定的行为”，并且总是使用“else”因素，给我 4。

变量声明：

int calccol = 0;
char usercol[5];

用户输入颜色选择：

std::cout << "Pick a color: " ;
 std::cout << "\n Black \n Red \n Green \n Blue \nYour color choice: " ;
 std::cin >> usercol ;

if 语句失败：

if (usercol == "black") {
      calccol = calccol + 1;
  } else if (usercol == "red") {
      calccol = calccol + 2;
  } else if (usercol == "green"){
      calccol = calccol + 3;
  } else {
      calccol = calccol + 4;
  } ;

   std::cout << calccol ;

当用户输入单词“green”时，我希望 calccol 的值例如为 3。

Answer 1

你不能比较 char* 这样的平等。您必须使用strcmp 并查看结果是否为0。但既然这是 C++，为什么不改用 std::string 呢？只需更改此行：

char usercol[5];

到这里：

std::string usercol;

它按预期工作。

---

下面是它如何使用 C 风格的字符串来代替： 首先，您需要确保缓冲区足够大，以前没有：

char usercol[10]; // has to be big enough to hold the choice

在本例中，它的大小为10，即输入文本的大小为 9，表示字符串结束的空终止符为 1。一个问题是（可能是恶意的）用户可以输入更大的字符串，导致缓冲区溢出，这是未定义的行为，可能会导致内存损坏。需要更复杂的解决方法来防止这种情况发生，例如，请参阅 this answer。

接下来，您需要修正比较。 usercol == "black" 不起作用，因为它将缓冲区的地址与放在程序可执行文件中的硬编码 "black" 的地址进行比较。他们永远不会匹配。相反，strcmp 必须像这样使用：

if (!strcmp(usercol, "black")) {...

strcmp 如果字符串相等则返回0，否则返回。这些将分别隐式转换为false 和true。通过在它前面添加!，反转它。这给了我们想要的结果：如果usercol 的字符串内容与"black" 的字符串内容相同，则if (!strcmp(usercol, "black")) 是true。