字符串使用的小误区

最近代码评审时候遇到的这些坑。

  咋一看代码貌似没什么问题，简单的字符串比较。可是仔细看了看感觉哪里不对劲，运行结果却是一直是输出"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";

2. 使用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);