字符串使用的小误区

• 2020 年 2 月 13 日
• 筆記

最近代码评审时候遇到的这些小误区。

  咋一看代码貌似没什么问题，简单的字符串比较。可是仔细看了看感觉哪里不对劲，运行结果却是一直是输出"UTF-32"。这里有个误区是，字符串(char *)是不能直接比较的，下列代码比较的是字符串的地址，这样就会导致它们字符串地址永远不会相等就一直输出的是"UTF-32"结果了。

string str("gbk");  if (str.c_str() == "gbk")      cout << "GBK";  else if (str.c_str() == "utf-8")      cout << "UTF-8";  else if (str.c_str() == "utf-16")      cout << "UTF-16";  else      cout << "UTF-32";  

正确做法：

1. 直接使用string对象比较。

string str("gbk");  if (str == string("gbk"))      cout << "GBK";  else if (str == string("utf-8"))      cout << "UTF-8";  else if (str == string("utf-16"))      cout << "UTF-16";  else      cout << "UTF-32";  

1. 使用strcmp函数比较字符串。

string str("gbk");  if (strcmp(str.c_str(), "gbk") == 0)      cout << "GBK";  else if (strcmp(str.c_str(), "utf-8") == 0)      cout << "UTF-8";  else if (strcmp(str.c_str(), "utf-16") == 0)      cout << "UTF-16";  else      cout << "UTF-32";  

  下列代码运行后直接段错误，原因是释放了一个非法的内存地址(静态存储区)。庆幸的是在C++11中，编译器会直接报错。

char *str = (char *)malloc(32);  str = "Hello world!";  free(str);  

正确做法(使用strcpy复制字符串)：

char *str = (char *)malloc(32);  strcpy(str, "Hello world!");  free(str);