C add strtok to char *

Question

У меня проблема при использовании strtok, и я не знаю, есть ли проблема на strtok или что-то еще.

У меня есть .txt файл с данными, как это:

sometextdada;othertextdata 
yetmoredata;andmoredata 

Считанные данные должны храниться в struct определенной следующим образом:

typedef struct team{ 
    char *name; 
    char *teamPlace; 
}Team; 

Если я делаю это:

char buffer[100]; 
Team eq; 
/*Read first line*/ 
fgets(buffer, 100, equipas)!= NULL); 
eq.name= strtok(buffer,";\n"); 
eq.teamPlace= strtok (NULL,";\n"); 

printf("%s %s\n", eq.name,eq.teamPlace); 

Я вижу, что strtok работает должным образом и сохраняет sometextdada в eq.name и othertextdata в eq.teamPlace

Теперь я хочу заменить, что printf с функцией, которая добавляет eq в связанный список, который определен следующим образом:

typedef struct nodeTeam{ 
    int numberOfTeams; 
    Team team; 
    struct nodeTeam *next; 
    struct nodeTeam *prev; 
}NodeTeam; 

Так что заменить printf на addNodeTeamsSorted(headEquipas,&tailEquipas,eq);

fgets(buffer, 100, equipas)!= NULL); 
eq.name= strtok(buffer,";\n"); 
eq.teamPlace= strtok (NULL,";\n"); 

addNodeTeamsSorted(headEquipas,&tailEquipas,eq); 

Теперь, если я распечатаю связанный список, я вижу, что мой узел добавлен, но name и teamPlace содержит мусорные символы. Но если я это сделать:

fgets(buffer, 100, equipas)!= NULL); 
eq.name= "test"; 
eq.teamPlace= "test2"; 

addNodeTeamsSorted(headEquipas,&tailEquipas,eq); 

Я могу видеть, что все работает, как ожидалось, так что заставляет меня думать, что проблема, когда Строка char на мой struct

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

Answer 1

strtok действует на буфер, указанный вами в the first call. Вместо того, чтобы хранить возвращаемые указатели напрямую (что указывает на buffer, что вы перезаписываете при обработке каждой строки), вам необходимо сделать копию строки (например, strncpy())

Answer 2

Ваша проблема в том, что ваш буфер создан в стеке, а результаты strtok указывают на буфер.

Когда вы возвращаетесь из этой функции или читаете другую строку из буфера, стек (и, следовательно, ваш буфер) повторно используется для чего-то другого, а данные, на которые указывают указатели, были перезаписаны. Это не происходит с постоянными строками, поскольку они находятся в области статической памяти, а не в стеке.

Попробуйте использовать функцию strdup() на выходе strtok() и сохранить их в своей структуре; это скопирует строки, возвращенные strtok() в кучу, где они не будут перезаписаны неожиданно.