Добавление новой записи в массив struct отменяет все последние записи

Question

Я пишу программу, которая читает информацию из файла, выделяет память для структуры для каждой строки и заполняет эту структуру информацией. Указатель на структуру затем помещается в глобальный массив.

Каждая строка в файле (кроме первой) находится в формате «Unsigned Char - Char - Unsigned Char - Char []».

Моя проблема в том, что каждый раз, когда я добавляю новую структуру, Char [] во всех остальных структурах перезаписывается.

Скажем, у меня есть следующий файл:

1 10 6 first 
2 12 7 second 
3 15 6 third 

Когда я запускаю мою программу с этим файлом, а затем напечатать глобальный массив это дает что-то вроде этого выхода:

ID: 1 | FLAGG: 10 | STR_LEN: 6 | MODELL: third 
ID: 2 | FLAGG: 12 | STR_LEN: 7 | MODELL: third 
ID: 3 | FLAGG: 15 | STR_LEN: 6 | MODELL: third 

вместо Ожидаемый результат:

ID: 1 | FLAGG: 10 | STR_LEN: 6 | MODELL: first 
ID: 2 | FLAGG: 12 | STR_LEN: 7 | MODELL: second 
ID: 3 | FLAGG: 15 | STR_LEN: 6 | MODELL: third 

Код:

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

#define MAX_LENGTH 256 

struct myStruct* globarlArray[MAX_LENGTH] 

struct mySTruct{ 
    unsigned char ID; 
    char FLAGG; 
    unsigned char str_len; 
    char* modell; 
}; 

int set_variables(struct myStruct* s, unsigned char new_ID, char new_FLAGG, unsigned char new_str_len, char* new_modell){ 
    s->ID = new_ID; 
    s->FLAGG = new_FLAGG; 
    s->str_len = new_str_len; 
    s->modell = new_modell; 
    return 0; 
} 
int main(int argc, char* argv[]){ 
    //Read from file 
    ... 
    //Initialize structs, add variables to struct, and add struct to global array 
    unsigned char ID; 
    int ID_INT; 
    char FLAGG; 
    unsigned char str_len; 
    char modell[253]; 
    char* tempstr; 

    for (i = 1; i < lines; i++){ 
     tempstr = str[i]; //str[1] is first line of into from file, str[2] is second line,... 
     Ruter_ID = tempstr[0]; 
     Ruter_ID_INT = Ruter_ID; 
     FLAGG = tempstr[1]; 
     str_len = tempstr[2]; 
     sprintf(modell, "%s", tempstr+3); 
     struct myStruct *line = (struct myStruct*)malloc(sizeof(struct myStruct)); 
     set_vars(line, ID, FLAGG, str_len, modell); 
     globalArray[ID_INT] = line; 
    } 
    //print array 
    for (i = 0; globalArray[i] != NULL; i++){ 
     printf("ID: %d | FLAGG: %d | str_len: %d | Modell: %s", 
     globalArray[i]->ID, globalArray[i]->FLAGG, globalArray[i]->str_len, 
     globalArray[i]->modell); 
    } 
    //Some more code 
    ... 
} 

Заранее благодарен!

Answer 1

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

Answer 2

Моя проблема заключается в том, что каждый раз, когда я добавить новую-структуру, Чар [] во всех других Структуры перезаписан

это происходит, так как вы используете:.

set_vars(line, ID, FLAGG, str_len, modell); 

modell является массивом . g указатель на первый элемент этого массива в каждом экземпляре myStruct.

Вам необходимо выделить память для каждого экземпляра myStruct и сохранить этот указатель .

char* copy = strdup(modell); 
    set_vars(line, ID, FLAGG, str_len, copy); 

Затем вам необходимо убедиться, что копия освобождена.

---

strdup не является стандартной функцией библиотеки, но она может быть легко реализована.