字符串分割到两个字符串由分隔符

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, ':'); 

另一个bug现在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()返回高达但不包括其参数字符串中的字符d。如果字符存在（size_t类型，而不是int），它将返回与indexOf()相同的值，如果不是，则返回字符串的长度，这是您希望的情况。

Answer 2

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

这里

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

您认为该行比指数+ 2更长。在malloc呼叫中应该检查一些内容以避免零。换句话说 - 如果分隔符是最后一个字符，则会遇到问题。

您应该添加由malloc

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

BTW返回值的检查 - 看看strdup和memcpy或strncpy