3

可能重复:
C++ 中不区分大小写的字符串比较

我已经为 C++ 编写了一些代码来比较两个字符串是否相等。我想要的是校对。我计划在未来将它用于更多的程序,所以这个功能做好它的工作很重要。这个功能是否看起来像一个可重复使用、便携等的功能?有没有更“最新”的方法来做到这一点?我使用了 ac 库,但这是一个 c++ 程序,这是禁忌吗?

谢谢,JH。

//function to compare two strings regardless of case 
//-- returns true if the two strings are equal
//-- returns false if
//  --the strings are unequal
//  --one of the strings is longer than 255 chars
bool isEqual(string str1, string str2){

  if(str1.length()!=str2.length())  //the strings are different lengths,
    return false;               //they can't be equal
  if((str1.length()>255) || (str2.length()>255))
    return false;

  char * cstr1 = new char [str1.length()+1];
  strcpy (cstr1, str1.c_str());
  char * cstr2 = new char [str2.length()+1];
  strcpy (cstr2, str2.c_str());

  for(int i=0; i<str1.length()+1; i++){
    if(toupper(cstr1[i]) != toupper(cstr2[i]))
      return false;
  }

  return true;
}
4

2 回答 2

20

您应该将函数重命名为 isEqual_CaseInsensitive 或其他名称,使其名称与函数的作用相匹配。您应该通过引用传递字符串以避免复制您不需要创建字符串的副本来比较它们

    bool isEqual_CaseInsensitive(const string& a, const string& b)
    {
            return a.size() == b.size() &&
                std::equal(a.begin(), a.end(), b.begin(), [](char cA, char cB) {
                        return toupper(cA) == toupper(cB);
                   });
    }
于 2012-09-24T15:47:11.607 回答
2

该功能看起来相当不错,除了:

这些到 c 字符串的转换不是必需的:

  char * cstr1 = new char [str1.length()+1];
  strcpy (cstr1, str1.c_str());
  char * cstr2 = new char [str2.length()+1];
  strcpy (cstr2, str2.c_str());

因为您可以像在 c 字符串中一样访问 std::string 字母:

  for(int i=0; i<str1.length(); i++){
    if(toupper(str1[i]) != toupper(str2[i]))
      return false;
  }

另请注意,我+1i<str1.length() + 1...中删除了

对于其他方法 - 参见:C++ 中不区分大小写的字符串比较

于 2012-09-24T15:49:03.527 回答