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循环中尝试比较时，我得到以下错误：

错误C2678：二进制'=='：找不到运算符，它接受类型为'std :: string'的左手操作数（或者没有可接受的转换） 32取代; c：\ program files \ microsoft sdks \ windows \ v6.0a \ include \ guiddef.h（192）：尝试匹配参数列表时可能是'int operator ==（const GUID＆amp;，const GUID＆amp;）' '（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或对{1}的引用时，targetName[i]会调用字符串上的operator[]，该字符串会对字符串进行索引并实际为您提供一个字符。

Answer 2

将targetName[i]的每次出现更改为(*targetName)[i]。或者，正如R. Martinho Fernandes所建议并传递参考而不是指针一样。

Answer 3

当targetName是指针时，此代码

if(targetName[i] == ineligibleChars[j])
    targetName[i] = '_';

尝试从指针索引，就像它曾经（指向一个数组）一样。要访问实际指向的字符串，需要在索引到字符串

之前取消引用指针

if((*targetName)[i] == ineligibleChars[j])
    (*targetName)[i] = '_';

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

3 个答案: