使用 strcmp 进行比较

Question

使用 gcc C99 进行编译

我正在尝试使用字符串比较来比较 2 个字符串。 但是，我似乎在 strcmp 行上得到了堆栈转储。

**属性将包含这些，所以我正在寻找框架类型。

[name] [time] [type] [time]
[name] [callref] [type] [string]
[name] [port] [type] [int16]
[name] [frametype] [type] [int16]

这是正确的比较方法吗？

非常感谢您的任何建议，

void g_start_element(void *data, const char *element, const char **attribute)
{
    for(i = 0; attribute[i]; i++)
    {
    /* Only interested in the frametype */
        if(strcmp(attribute[i], "frametype") == 0)
        {
            /* do some work here */
        }

     }
}

Answer 1

您需要有一个空字符串才能终止 for 循环：

[name] [time] [type] [time]
[name] [callref] [type] [string]
[name] [port] [type] [int16]
[name] [frametype] [type] [int16]
null

如果没有这个，for 循环将不会终止，并且当您调用 strcmp 时，您将得到指向垃圾的 attribute[i]。

Answer 2

1. 属性数组是如何初始化的？ 可能已插入 NULL 元素。
2. 此外，数组元素必须以 NULL 结尾。
3. 您可以考虑使用 strncmp() 作为 strcmp( 的更安全替代方案) ）。

Answer 3

此代码的上下文是 expat 解析 - 请参阅这篇文章。 属性数组是交替的名称和值，以单个 0 结尾。

除非您正在寻找名称或值等于您的测试字符串的任何属性（这有点不寻常），否则您的代码应该将 i 增加 2 而不是 1 - 以便它同时跳过名称和值。

您应该比较 attribute[i] 来匹配名称，或者比较 attribute[i+1] 来匹配值。

不要假设属性将按任何特定顺序排列。 目前您只查看属性1，这是值返回的第一个属性。 如果有多个属性，则可以按任意顺序返回它们。

Answer 4

您的输入数组（属性）是否以 NULL 结尾？ 您没有将其列为存在，但代码需要它，否则它将进入随机内存，这是导致段错误的一个很好的原因。

插入要比较的索引和/或属性的打印，以查看它的终止行为是否符合您的预期。

不太确定问题的字符串比较部分...如果输入包含这些方括号等，那么您将永远找不到任何内容，因为您正在查看每个字符串的开头。 尝试 strstr() 如果是这种情况，它会找到子字符串。

Answer 5

添加日志记录并转储所有属性和索引器值。 这将有助于确定出了什么问题。

Answer 6

除了不以 null 终止之外，为什么不使用 strncmp() 呢？ :)

Answer 7

我会添加对 NULL 的检查，这样

if(attribute[1] != NULL && strcmp("frametype", attribute[1]) == 0)

你就不会取消引用 NULL 指针。