创建一个函数来匹配 main 中的字符串

Question

这是一项小组作业，我们的教授将项目延长了 1 周，这变得相当困难。有 50 个阶段/测试，我们只能达到第 11 阶段，然后功能失败。

这个函数在我们的 .cpp 文件中（我们肯定是这个函数导致了问题，因为当我们改变它的一部分时，它会影响我们已经通过的第 11 阶段）。

int segment::match(const char word[]) {
    int i;
    cout << data[0];
    data[0] == "OOP";
    cout << data[0];
    for(i=0;i<NUM_MAX;i++) {
        cout << "word = " << &word[i] << " data[i] = " << data[i];
        if(strstr(&word[i],data[i])!= NULL)
        break;
    }
       return i==NUM_MAX ? 1 : i-1;

从 main.cpp（作为作业提供给我们）这就是我们想要完成的目标

通过了测试 11...

Your match( ) return value ----> -1
Actual match( ) return value --> -1 
Press the ENTER key to continue... 
word = OOP data[i] = 

Failed while testing the match( ) 
function... Failed on test 12... 
Your match( ) return value ----> -1 
Actual match( ) return value --> 1 
Press the ENTER key to continue...

You passed 11/50 tests... 
Your program is 22.00% complete! 
Your program still needs some work! 
Keep at it!

该函数应该做的是检查“oop”，如果不存在则以 -1 退出，如果存在则应以 1 返回 true。

我想我要问的是如何使该函数以正确的顺序返回 -1 和 1？

如果您想访问 main.cpp 和 segement.cpp，我可以将它们作为文件上传到某个地方，因为它们很长，我不想塞满帖子。

感谢您的帮助，谢谢。

编辑* 这是我们拥有的完整代码 http://jsfiddle.net/h5aKN/

“html”部分包含我们构建的 segement.cpp。 jscript 部分有 a2main.cpp，这是我们教授构建的。

Answer 1

data[0] == "OOP"; 可能不是您想要做的。双 = (==) 测试是否相等，所以在这里您要测试 data (data[0]) 的第一个索引处的项目和字符串 "OOP" 是否相等。

在运行测试时，您正在计算：word = OOP data[i] =，这意味着word[i] 可能已正确定义，但data[i] 未正确定义。这可以追溯到上面等价测试的用法。

如果你正确设置了初始化data，（正确的意思是正确分配内存，我不知道data是在哪里实例化的），那么测试可能会返回-1，因为它会得到一个非空指针从strstr() 调用（假设data 是正确的类型），i 在breaking 上将是0，而三元运算符将产生i-1, = -1。

所以修复data变量的初始化/赋值

如果您不限于 c 样式字符串（字符数组），我会使用 std::string 类型及其相关方法（如果您还没有，请参阅 the c++ string reference）。通常使用起来会更好

Answer 2

如果您将单词列表传递给函数：

使用 (strstr(&word[i],data[i])) 表明您正在另一个字符串中寻找一个字符串。因此，您正在遍历字符串（单词）列表。

那么这看起来不对：

int segment::match(const char word[]) {

你在这里传递一个字。
它不可能告诉它应该是什么，但猜测是：

int segment::match(const char* word[]) {
                   //      ^^^^^

但老实说，整个事情都是相当丑陋的 C++。如果您正在编写 C，我会说很好，但是如果您正确地编写 C++，类型系统将使您免于所有这些问题。使用 std::string 来表示单词。