Разбивает строку на две строки по разделителю

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)); 

Этот код производит правильный выход, когда я Test это с 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

EDIT: Существует еще одна ошибка для

index = indexOf(line, ':'); 

Теперь ошибка Valgrind в подстроки в строке input[endIndex] = '\0';:

Invalid write of size 1 

EDIT: Решение моего кода, что есть две ошибки

в главном

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 символов в строке аргумента. Он возвращает то же значение, что и indexOf(), если символ присутствует (за исключением типа size_t вместо int), и он возвращает длину строки, если это не так, что вам нужно в вашем случае.

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