Question

可能重复：
Case insensitive string comparison in C++

我为c ++编写了一些代码来比较两个字符串的相等性。我想要的是校对。我计划将来使用它来制作更多的程序，所以这个功能很好地完成它的工作很重要。这个功能看起来像可重复使用，便携式等吗？是否有更“最新”的方法来做到这一点？我使用了一个c库，但这是一个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;
}

Answer 1

您应该将函数重命名为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);
                   });
    }

Answer 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());

因为您可以访问std :: string字母，如c string：

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

另请注意，我从+1 ...

中删除了i<str1.length() + 1

对于其他方法 - 请参阅：Case insensitive string comparison in C++

C ++，检查两个字符串的大小写不敏感

2 个答案: