互换 const char* 和 std::string

Question

我正在重构一个旧的 C 库，并且当前正在更改外部 API，以便它使用 std::string 而不是 const char*。

ColumnType Table::getColType(const char *name) const
{
    int id = getColumnId(name) ;
    return getColType(id) ;
}

and

int Table::getColumnId (const char * col_name) const
{
    unsigned int i = 0;

    while ((i < m_table.num_cols) && (strcmp(m_table.cols[i]->name, col_name) != 0) )
        i++;

    if (i < m_table.num_cols)
        return i;
    else
        return -1;
}

致：

ColumnType Table::getColType(const std::string& name_in) const
{
    const char* name = name_in.c_str();
    int id = getColumnId(name) ;
    return getColType(id) ;
}

and 

int Table::getColumnId (const std::string& col_name_in) const
{
    const char* col_name = col_name_in.c_str();
    unsigned int i = 0;

    while ((i < m_table.num_cols) && (strcmp(m_table.cols[i]->name, col_name) != 0) )
        i++;

    if (i < m_table.num_cols)
        return i;
    else
        return -1;
}

在新代码中，我将 const char* 传递给现在需要引用 const std::string 的函数。我知道 std::string 可以从 const char* 初始化，并且代码可以正确编译（没有警告等）。

但我只是想确保我不会做任何以后会困扰我的事情（抛开国际问题不谈）。

简而言之——我所做的事情“安全”吗？

Answer 1

它是安全的，只要正确执行，就不会造成任何伤害。

尽管如此，我不认为我会建议继续这样做，除非你能指出一个实质性的好处。

更安全的方法可能是添加一个接受 const string& 的函数，并直接传递到 const char* 函数。这样，您就可以让客户端保留 std::string& 代码，而无需修改内部结构。

例如：

ColumnType Table::getColType(const std::string& name_in) const
{
    return getColType(name_in.c_str());
}

Answer 2

1) getColType 不需要从 std::string 获取 c_str()，只需传递 std::string&直接进入 getColumnId。

2）您应该使用重写的 相等运算符 或使用 std::string::compare 直接代替 strcmp。看

Answer 3

请注意包含 NULL 的 std::string。 C++ 类对他们来说很好； NULL 并不特殊。但 C 字符串当然将其视为字符串结尾。以下不相同：

if (std_string_a == std_string_b) { /* C++ way */ }

// vs.

const char *cstring_a = std_string_a.c_str(),
           *cstring_b = std_string_b.c_str();
if (0 == strcmp(a, b)) { /* C way */ }

当您混合搭配时，您必须担心当 C++ 方式为 false 而 C 方式为 true 时会出现奇怪的错误。