通过分隔符将字符串拆分为两个字符串

Question

我尝试使用此代码通过分隔符将一个字符串分成两个字符串

int indexOf(char *msg, char c) {
    int i;
    for (i = 0; msg[i] != '\0'; i++) {
        if (msg[i] == c)
            return i;
    }
    return -1;
}

char *substring(char *msg, int startIndex, int endIndex) {
    int length = endIndex - startIndex;
    char *input = (char *)malloc(length * sizeof(char) + 1);

    int i;
    for (i = startIndex; i != endIndex; i++) {
        input[i - startIndex] = msg[i];
    }
    input[endIndex] = '\0';

    return input;
}

在main 我有：

index = indexOf(msg, ':');

first = substring(line, 0, index - 1);
second = substring(line, index + 2, strlen(line));

当我使用 valgrind 对其进行测试时，此代码会产生正确的输出。在第二个变量中分配的子字符串会产生错误。

这个函数的问题在哪里？有没有其他方法可以将字符串拆分成两个字符串？

char *msg = readMessage(stdin);
index = indexOf(msg, '\n');
char *line, *first, *second; 

line = substring(msg, 0, index);

结束 valgrind Address 0x5203a52 is 5 bytes after a block of size 13 alloc'd

编辑：还有另一个错误

index = indexOf(line, ':');

现在 valgrind 错误位于 input[endIndex] = '\0'; 行的子字符串中：

Invalid write of size 1

编辑：我的代码有两个错误的解决方案

主要

index = indexOf(msg, ':');

应该是

index = indexOf(line, ':');

在子字符串中

input[endIndex] = '\0';

应该是

input[length] = '\0';

谢谢大家

Answer 1

你的代码有一些问题：

• input[endIndex] = '\0'; 使用了错误的索引。应该是input[length] = '\0';

• 在main() 中，您不应该对indexOf 的返回值做出隐含的假设。如果在line 中找不到:，则发布的代码会调用未定义的行为：

这是一个更安全的版本：

int index = indexOf(line, ':');
if (index >= 0) {
    // found the `:` separator
    char *first = substring(line, 0, index);
    if (line[index + 1] == ' ') {
        index++;  // skip the space after the :
    }
    char *second = substring(line, index + 1, strlen(line));
    ...
}

您可以使用strcspn() 而不是indexOf 来提取测试次数较少的部分：

char *msg = readMessage(stdin);
size_t index = strcspn(msg, "\n");
char *line = substring(msg, 0, index);
...

strcspn() 返回其参数字符串中最多但不包括和的字符数。如果字符存在（size_t 类型而不是 int 除外），它返回与 indexOf() 相同的值，如果不存在，它返回字符串的长度，这是你想要的案子。

Answer 2

index = indexOf(msg, ':');
                ^^^
                line ?

这里

second = substring(line, index+2, strlen(line));

您假设该行比 index+2 长。您应该检查以避免在malloc 调用中出现零。换句话说 - 如果分隔符是最后一个字符，你就会遇到问题。

您应该检查malloc返回的值

if (!input)
{
    // Print error message
    exit(1);
}

顺便说一句 - 看看 strdup 和 memcpy 或 strncpy