如何解决以下函数中的内存分配问题? (分配)
How to fix memmory allocation problem in the function below? (malloc)
我需要读取一个 .txt 文件并将文件中的每个单词分配到一个从结构向量指向的结构中。我会在下面更好地解释。
感谢您的帮助。
我的程序只分配文件的第一个字...
我知道问题出在函数 insere_termo() 中,因为我已经测试了 fscanf whitout 调用该函数并且它运行正常。
结构
typedef struct _item
{
int conta; //contador
char *termo; //palavra
} Item;
typedef struct _mapa
{
int total; // número de itens no mapa
int blocos; // número de blocos de itens alocados
Item **lista; // vetor de ponteiros para itens
} Mapa;
主要
int main()
{
Mapa mp;
FILE *arq;
int i, result, numPalavras;
float x;
int valor, max, min, mincar;
char caminho[20];
char termo[40];
int tam;
inicia_mapa(&mp);
valor = menu();
这是我的代码的重要部分。我在用户提供的路径中读取文件并调用函数 insere_termo();
负责分配 struct
.
中的每个单词
switch (valor)
{
case 1:
printf ("*Ler Arquivo* \n");
printf("Digite caminho: ");
scanf("%s", caminho);
arq = fopen(caminho,"r"); // abrir arquivo no caminho especificado
if (arq == NULL)
{
printf("Erro, nao foi possivel abrir o arquivo\n");
}
while(fscanf(arq, "%s", termo) != EOF)
{
//Adiciona cada palavra no vetor
insere_termo(&mp, termo);
}
break;
函数
INICIA_MAPA() 是负责启动映射的函数,分配一个包含 10 个指针的块到结构 "ITEM".
INSERE_TERMO() 是负责分配文本文件给出的单词的函数。内存分配存在一些问题,但我无法解决。欢迎任何帮助。
void inicia_mapa (Mapa *mp) //inicia um mapa vazio
{
mp->lista = (Item**)malloc(BLOCK*sizeof(Item*));
mp->total = 0;
mp->blocos = 1;
}
void insere_termo (Mapa *mp, char *s) // insere um item com termo s e conta = 1
{
if ((mp->total != 0)&&((mp->total%BLOCK) == 0))
{
mp->lista = (Item**)realloc(mp->lista,(mp->blocos+1)*BLOCK*(sizeof(Item*)));
mp->blocos++;
}
mp->lista[mp->total] = (Item*) malloc(sizeof(Item)); // alocação dinâmica de item no mapa
if (mp->lista[mp->total] == NULL)
{
printf("Erro na alocacao de memoria\n");
exit(0);
}
strcpy(mp->lista[mp->total]->termo, s); // Insere termo s
printf("%s\n", mp->lista[mp->total]->termo);
mp->lista[mp->total]->conta++;
mp->total++; // contagem do total
}
strcpy(mp->lista[mp->total]->termo, s);
失败,因为 mp->lista[mp->total]->termo
尚未分配值。将未初始化的指针 p
p 传递给 strcpy(p, ...)
是 未定义的行为 (UB)。
而是分配内存,然后使用 strcpy()
进行复制
// strcpy(mp->lista[mp->total]->termo, s);
p->lista[mp->total]->termo = malloc(strlen(s) + 1);
strcpy(p->lista[mp->total]->termo, s);
// or
p->lista[mp->total]->termo = strdup(s);
完成后一定要释放内存。
可能存在其他问题。
@chux 非常感谢!成功了。
功能更新:
void insere_termo (Mapa *mp, char *s) // insere um item com termo s e conta = 1
{
if ((mp->total != 0)&&((mp->total%BLOCK) == 0))
{
mp->lista = (Item**)realloc(mp->lista,(mp->blocos+1)*BLOCK*(sizeof(Item*)));
mp->blocos++;
}
mp->lista[mp->total] = (Item*) malloc(sizeof(Item)); // alocação dinâmica de item no mapa
if (mp->lista[mp->total] == NULL)
{
printf("Erro na alocacao de memoria\n");
exit(0);
}
mp->lista[mp->total]->termo = strdup(s); // Insere termo s
printf("%s\n", mp->lista[mp->total]->termo);
mp->lista[mp->total]->conta++;
mp->total++; // contagem do total
}
我需要读取一个 .txt 文件并将文件中的每个单词分配到一个从结构向量指向的结构中。我会在下面更好地解释。
感谢您的帮助。
我的程序只分配文件的第一个字...
我知道问题出在函数 insere_termo() 中,因为我已经测试了 fscanf whitout 调用该函数并且它运行正常。
结构
typedef struct _item
{
int conta; //contador
char *termo; //palavra
} Item;
typedef struct _mapa
{
int total; // número de itens no mapa
int blocos; // número de blocos de itens alocados
Item **lista; // vetor de ponteiros para itens
} Mapa;
主要
int main()
{
Mapa mp;
FILE *arq;
int i, result, numPalavras;
float x;
int valor, max, min, mincar;
char caminho[20];
char termo[40];
int tam;
inicia_mapa(&mp);
valor = menu();
这是我的代码的重要部分。我在用户提供的路径中读取文件并调用函数 insere_termo();
负责分配 struct
.
switch (valor)
{
case 1:
printf ("*Ler Arquivo* \n");
printf("Digite caminho: ");
scanf("%s", caminho);
arq = fopen(caminho,"r"); // abrir arquivo no caminho especificado
if (arq == NULL)
{
printf("Erro, nao foi possivel abrir o arquivo\n");
}
while(fscanf(arq, "%s", termo) != EOF)
{
//Adiciona cada palavra no vetor
insere_termo(&mp, termo);
}
break;
函数
INICIA_MAPA() 是负责启动映射的函数,分配一个包含 10 个指针的块到结构 "ITEM".
INSERE_TERMO() 是负责分配文本文件给出的单词的函数。内存分配存在一些问题,但我无法解决。欢迎任何帮助。
void inicia_mapa (Mapa *mp) //inicia um mapa vazio
{
mp->lista = (Item**)malloc(BLOCK*sizeof(Item*));
mp->total = 0;
mp->blocos = 1;
}
void insere_termo (Mapa *mp, char *s) // insere um item com termo s e conta = 1
{
if ((mp->total != 0)&&((mp->total%BLOCK) == 0))
{
mp->lista = (Item**)realloc(mp->lista,(mp->blocos+1)*BLOCK*(sizeof(Item*)));
mp->blocos++;
}
mp->lista[mp->total] = (Item*) malloc(sizeof(Item)); // alocação dinâmica de item no mapa
if (mp->lista[mp->total] == NULL)
{
printf("Erro na alocacao de memoria\n");
exit(0);
}
strcpy(mp->lista[mp->total]->termo, s); // Insere termo s
printf("%s\n", mp->lista[mp->total]->termo);
mp->lista[mp->total]->conta++;
mp->total++; // contagem do total
}
strcpy(mp->lista[mp->total]->termo, s);
失败,因为 mp->lista[mp->total]->termo
尚未分配值。将未初始化的指针 p
p 传递给 strcpy(p, ...)
是 未定义的行为 (UB)。
而是分配内存,然后使用 strcpy()
// strcpy(mp->lista[mp->total]->termo, s);
p->lista[mp->total]->termo = malloc(strlen(s) + 1);
strcpy(p->lista[mp->total]->termo, s);
// or
p->lista[mp->total]->termo = strdup(s);
完成后一定要释放内存。
可能存在其他问题。
@chux 非常感谢!成功了。
功能更新:
void insere_termo (Mapa *mp, char *s) // insere um item com termo s e conta = 1 {
if ((mp->total != 0)&&((mp->total%BLOCK) == 0))
{
mp->lista = (Item**)realloc(mp->lista,(mp->blocos+1)*BLOCK*(sizeof(Item*)));
mp->blocos++;
}
mp->lista[mp->total] = (Item*) malloc(sizeof(Item)); // alocação dinâmica de item no mapa
if (mp->lista[mp->total] == NULL)
{
printf("Erro na alocacao de memoria\n");
exit(0);
}
mp->lista[mp->total]->termo = strdup(s); // Insere termo s
printf("%s\n", mp->lista[mp->total]->termo);
mp->lista[mp->total]->conta++;
mp->total++; // contagem do total
}