c++ - 函数不允许比较字符串中的单个字符 - C++

Question

我有一个名为 removeInelligibleCharsFromTargetName 的新函数。

void removeInelligibleCharsFromTargetName(string *targetName)
{
    for(int i = 0; i < targetName->length(); i++)
    {
        for(int j = 0; j < ineligibleChars.length(); j++)
        {
            if(targetName[i] == ineligibleChars[j])
                targetName[i] = '_';
        }
    }
}

问题是当我尝试在 if 循环中进行比较时出现以下错误:

error C2678: binary '==' : no operator found which takes a left-hand operand of type 'std::string' (or there is no acceptable conversion) 32> c:\program files\microsoft sdks\windows\v6.0a\include\guiddef.h(192): could be 'int operator ==(const GUID &,const GUID &)' while trying to match the argument list '(std::string, char)'

但是把完全相同的嵌套循环放回我调用它的地方而不是调用它工作正常的函数。

有人能告诉我它在函数中不起作用但在函数外工作正常吗？毫无疑问，它是一个指针，但我不知道是什么。

Answer 1

为什么要传递指向 string 的指针？这真是个坏主意。传递引用。

void removeInelligibleCharsFromTargetName(string& targetName)
{
    for(int i = 0; i < targetName.length(); i++)
    {
        for(int j = 0; j < ineligibleChars.length(); j++)
        {
            if(targetName[i] == ineligibleChars[j])
                targetName[i] = '_';
        }
    }
}

问题是当你有一个指针时，targetName[i] 与 *(targetName+i) 相同。 这相当于对字符串数组进行索引。如果您没有字符串数组，这只会导致未定义的行为。您很幸运，代码无法编译(无法将字符串与字符进行比较)并且编译器捕获了错误。如果它碰巧编译，当你运行它时，你可能会观察到一些非常奇怪的行为。

当你有一个string，或者对一个字符串的引用时，targetName[i] 调用字符串上的operator[]，它索引到字符串中，实际上给你一个字符。