C strtok и зЬгсру

Question

У меня есть текстовый файл, похожий на следующее:

Name1: ID1 
Name2: ID2 
Name3: ID3 

Я пытаюсь разобрать его, чтобы получить

Name1 
Name2 
Name3 

хранится в переменной.

я написал следующую функцию:

/* 
* filename Name of file to read 
* result  The result will be stored here 
*/ 

void readlist(char* filename, char* result) { 
    FILE *fp; 
    char buffer[2048]; 
    memset((void *)result, '\0', BUFFER_SIZE); 

    fp = fopen(filename, "r"); 

    while (fgets(buffer, sizeof(buffer), fp)) { 
     char *token = NULL; 
     token = strtok(buffer, ":"); 
     strcat(result, token); 
    } 

    fclose(fp); 
} 

Однако, когда я называю его:

char result[2048]; 
readlist("test.txt", result); 
printf("%s", result); 

Я получаю пустой вывод. Кажется, что strtok() испортил данные, но я могу ошибаться.

Что я здесь делаю неправильно?

Спасибо заранее!

Answer 1

Я побежал код (или, по крайней мере, я создал одну программу из фрагментов кода), и она работала хорошо для меня:

#include <stdio.h> 
#include <string.h> 

void readlist(char* filename, char* result); 

int main(void) { 
char result[2048]; 
readlist("test.txt", result); 
printf("%s", result); 
} 

void readlist(char* filename, char* result) { 
    FILE *fp; 
    char buffer[2048]; 

    fp = fopen(filename, "r"); 

    while (fgets(buffer, sizeof(buffer), fp)) { 
     char *token = NULL; 
     token = strtok(buffer, ":"); 
     strcat(result, token); 
    } 


    fclose(fp); 
} 

Когда я запустил его на входной файл, я получил как выход

Name1Name2Name3 

Ровно как ожидалось.

Это использовало gcc-компилятор версии 4.2.1 в Mac OS. Это говорит о том, что ваш код не так далеко, как вы думаете (независимо от того, является ли компилятор инициализацией строки до 0, прежде чем запуск зависит от реализации, очевидно). Чтобы быть в безопасности, вам нужно убедиться, что ваш исходный результат - все нули. Вы можете сделать

char result[2048] = {'\0'}; 

Это гарантирует, что все элементы инициализированы нулем.Другой способ заключается в использовании

static char result[2048]; 

Так как любой переменной, объявленной static будет инициализирован в ноль.

Наконец, такие вещи, как

result[0] = '\0'; 

бы начать строку с нулевой длиной - и то, что следует не имеет значения. Это, наверное, самое чистое.

Answer 2

Убедитесь, что вы инициализируете result пустой строке. Либо

char result[2048] = ""; 

в вызывающем или

result[0] = '\0'; 

в верхней части ReadList().

Answer 3

Вы никогда не инициализировали результат, перед вызовом readlist или после.

просто добавить перед вызовом readlist: strcpy(result, "");

Answer 4

char result[2048]; 

Инициализировать это утверждение в противном случае результат будет содержать значения для мусора, как сво auto переменной.

Таким образом, используется char result[2048] = ""; до readlist() в вашей функции main.