我正在编写一个程序,它从文件中读取信息,为每一行的结构分配内存,并用信息填充该结构。然后将指向结构的指针放在全局数组中。
文件中的每一行(第一行除外)的格式为'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
...
}
提前致谢!
答案 0 :(得分:1)
我的问题是,每次添加新结构时,所有其他结构中的Char []都会被覆盖。
这是因为您正在使用:
set_vars(line, ID, FLAGG, str_len, modell);
modell
是一个数组。您将指针存储在myStruct
的每个实例中的该数组的第一个元素。
您需要为myStruct
的每个实例分配内存并存储该指针 1 。
char* copy = strdup(modell);
set_vars(line, ID, FLAGG, str_len, copy);
然后,您需要确保已取消分配副本。
1 strdup
不是标准库函数,但可以轻松实现。
答案 1 :(得分:0)
你的modell
数组被反复使用 - 你的每个结构都指向同一块内存。您应该为每个结构提供字符串的副本(可能使用strcpy()
或相关函数。(当您释放结构时,您也必须担心释放该副本的内存)。