Borland string::find bug

Question

我支持使用 Borland C++ Builder 5.02（从 1997 年开始）编写的 C++ 应用程序。 Borland 字符串类的 find() 方法的行为与我预期的不同：

#include <cstring>
#include <iostream>

int main (int argc, char *argv[])
{
   string needle = "length == eighteen";
   string haystack = "<" + needle + ">";
   if (haystack.find(needle) != NPOS)
      cout << "Found it!" << endl;
   else
      cout << "Not found" << endl;

   return 0;
}

这个程序输出Not found。如果我把针改短一些，它会输出Found it!。如果我将尖括号换成其他字符，它会找到它。空格有效，但括号也无效。

请注意，我在这里使用的是 Borland 字符串库：如果我使用 #include <string> 并改用 std::string，那么它会完全按照我的预期工作。遗憾的是，将整个应用程序更改为使用 STL 字符串并不是一个可行的答案！

从文档看来，Borland 使用基于哈希的算法进行字符串搜索。我找不到更多关于这个的细节，我已经完成了反汇编，但并不聪明。

我很难相信这真的是字符串库中的一个错误，特别是因为如果是这样的话，我希望能够找到一篇文章或关于它的东西。我找不到任何此类信息。

但是，我的想法已经用完了！这是一个已知的错误？有解决办法吗？

编辑：再次查看反汇编后，我认为它正在尝试执行类似 Rabin-Karp 算法的操作，其中哈希函数的计算方式为 mod 33554393（最大素数

Answer 1

您没有使用 Borland 字符串库。 String（大写 S）是 Borland 字符串类。 string（小写 s）与 std::string 完全相同，是 STL 字符串类，它不是 Borland 实现（BCB5 中的 STL 是 RogueWave STL）。您对#include <cstring> 的使用可能会将std::string 带入全局命名空间，这就是您的代码编译的原因。但是您确实应该改用#include <string> 和std::string。至于NPOS，您应该改用string::npos，因为这是string::find() 实际返回的内容。

#include <cstring>
#include <iostream>

int main (int argc, char *argv[])
{
   string needle = "length == eighteen";
   string haystack = "<" + needle + ">";
   if (haystack.find(needle) != string::npos)
      cout << "Found it!" << endl;
   else
      cout << "Not found" << endl;

   return 0;
}

或者：

#include <string>
#include <iostream>

int main (int argc, char *argv[])
{
   std::string needle = "length == eighteen";
   std::string haystack = "<" + needle + ">";
   if (haystack.find(needle) != std::string::npos)
      std::cout << "Found it!" << std::endl;
   else
      std::cout << "Not found" << std::endl;

   return 0;
}

Answer 2

我发现 1998 年的参考资料表明 Borland 搜索字符串的实现存在错误：

https://groups.google.com/forum/?fromgroups=#!searchin/borland.public.cpp.language/cstring$20bug/borland.public.cpp.language/XBzjaJmCYpk/gtMPm-j8jugJ

此外，在历史上的某个时刻，C++ 委员会似乎决定字符串类将成为标准 C++ 的一部分，而 cstring 的字符串类是这个的残余：

https://groups.google.com/forum/?fromgroups=#!searchin/borland.public.cpp.language/borland$20cstring/borland.public.cpp.language/2psY2seRmS4/ywVrqwU1C2wJ

Answer 3

如果你有原始的BC++ 5.02安装盘，字符串类源在BC5\SOURCE\RTL\SOURCE\STRING下。

这里是 string::find_case_index() 函数（由 string::find() 调用）的代码摘录：

const long q = 33554393L;
const long q32 = q<<5;

size_t testlength = length() - startindex;
size_t patternlength = patl = strlen(cp);
if( testlength < patternlength )
    return NPOS;
if( patternlength == 0 )
    return 0;

long patternHash = 0;
long testHash = 0;

const char _FAR *testP = c_str()+startindex;
const char _FAR *patP = cp;
long x = 1;
size_t i = patternlength-1;

while( i-- )
    x = (x<<5)%q;

for( i=0; i<patternlength; i++ )
    {
    patternHash = ( (patternHash<<5) + *patP++  ) % q;
    testHash    = ( (testHash   <<5) + *testP++ ) % q;
    }

testP = c_str()+startindex;
const char _FAR *end = testP + testlength - patternlength;

while (1)
    {

    if(testHash == patternHash)
        if( !get_paranoid_check_flag() ||
            !strncmp( testP, cp, patternlength) )
          return (size_t)(testP-c_str());

    if( testP >= end )
        break;

    // Advance & calculate the new hash value:
    testHash = ( testHash + q32 - *testP * x                 ) % q;
    testHash = ( (testHash<<5)  + *(patternlength + testP++) ) % q;
    }
return NPOS;          // Not found.