C++：将 std::string 传递给想要更改字符串的 C 函数的正确实现？

Question

我在用 C 编写的第三方库中有一个函数：char* fix_filename_slashes(char* path)。此函数需要传递给它的可变 C 字符串，以便它可以根据操作系统将路径中的所有斜杠更改为正确的使用方式。我在 Facade 中使用的所有字符串都声明为 std::string。我尝试简单地使用 foo.c_str() ，因为每个其他需要 C 字符串的函数都不会更改它，并且需要 const char * ，但是这个函数会导致错误：错误：“const char *”类型的参数与“char *”类型的参数不兼容

我得出的结果是：

char* tempf = const_cast<char*>(filename.c_str());
filename = std::string(fix_filename_slashes(tempf));
tempf = NULL;

被认为是“正确”还是还有其他（更正确？）完成任务的方法？

编辑

哎呀。显然该函数返回字符串的副本。尽管如此，已经给出了一些很好的答案。

Answer 1

如果字符串长度不变，可以使用指向字符串第一个字符的指针。这是 C++03 标准中未定义的行为，但所有已知的实现都可以正常工作，并且在 C++11 标准下明确允许。

fix_filename_slashes(&filename[0]);

如果字符串的大小可能发生变化，您将需要做更多的工作。

filename.resize(max_size, 0);
append_filename_suffix(&filename[0]);
filename.resize(strlen(filename.c_str()));

Answer 2

将其转换为存储在 std::vector 中的以 null 结尾的字符序列：

template <typename Character>
std::vector<Character> to_vector(std::basic_string<Character> const& s)
{
    std::vector<Character> v;
    v.reserve(s.size() + 1);
    v.insert(v.end(), s.begin(), s.end());
    v.push_back(0);
    return v;
}

用法示例：

std::string filename = get_filename();
std::vector<char> filename_cstr = to_vector(filename);
filename = std::string(fix_filename_slashes(&filename_cstr[0]));

Answer 3

由于您会遇到所有麻烦，因此您只需遵守 C 函数的要求并将字符串复制到 char 数组，然后在函数之后从 char 数组创建一个字符串或强制对原始字符串进行复制分配。

    char* temp = new char[str.size() + 1]
    // Force a copy of the result into another string
    str = (const char*)fix_filename_slashes(strncpy(temp, str.c_str(), str.size() + 1));
    delete [] temp;

Answer 4

如果 string 使用单独的缓冲区来存储 c_str 字符串，则不会修改原始字符串。

更好的方法是在堆栈或堆上创建一个 char 缓冲区，将字符复制到其中（以 null 终止），调用修复函数，然后将缓冲区分配回字符串。

Answer 5

这是另一种方法，需要进行一些设置，但之后会自动工作。它依赖于一个临时对象，该对象获取原始字符串的副本并将修改后的字符串复制回析构函数中。显然，所有这些复制的效率都不会太高，但在大多数情况下，效率并不重要。

class mutable_string
{
public:
    mutable_string(std::string & str, int maxlen = 0) : m_str(str)
    {
        m_buffer.resize(max(maxlen, str.length()) + 1);
        memcpy(&m_buffer[0], str.c_str(), str.length()+1);
    }
    ~mutable_string()
    {
        m_str = m_buffer;
    }
    operator char* ()
    {
        return &m_buffer[0];
    }
private:
    std::string &     m_str;
    std::vector<char> m_buffer;
};

fix_filename_slashes(mutable_string(filename));