C의 구조체에 대한 포인터 배열에서 정보 얻기

게시 한 코드에는 다음과 같은 문제가 있습니다.

라인

amonster->name = (char*)malloc(sizeof(char)*sizeof(name));

말도 안 돼. sizeof(name)플랫폼에 따라 32 비트 또는 64 비트 인 포인터의 크기입니다. 그러나 그 이상을 할당해야 할 수도 있습니다. strlen(name) + 1함수에 전달되는 문자열의 길이 (널 종료 문자 포함)이므로 바이트 를 할당해야합니다 . 다음 줄에도 동일하게 적용됩니다.

amonster->element = (char*)malloc(sizeof(char)*sizeof(element));

또한 라인

amonster->population = (int)malloc(sizeof(int)*sizeof(population));

는 amonster->population포인터가 아니기 때문에 의미 가 없습니다. 동적으로 할당 된 메모리에 대한 포인터를 저장하려는 이유는 무엇입니까? amonster->population는 문자열이 아니라 고정 길이 변수 이기 때문에 여기에서 동적 메모리 할당이 필요 하지 않습니다 struct.. 따라서이 줄을 삭제할 수 있습니다. amonster->population = population;이미 가지고 있는 라인 만 있으면됩니다.

또한 라인

amonster->name = name;

당신이 원하는 것을하지 않습니다. 문자열을 복사하지 않고 포인터, 즉 메모리 주소 만 복사합니다. 그런 식으로 함수로 돌아올 때까지 매달린 포인터 가 될 포인터를 복사 합니다 main. 실제 문자열을 복사하려면 strcpy( amonster->name, name );. 다음 행에도 동일하게 적용됩니다.

amonster->element = element;

라인

a_array[i] = malloc(sizeof(monster));

불필요하고 메모리 누수 만 만듭니다 . 함수의 모든 구조체에 대해 이미 공간을 할당했습니다 createMonster.

선

monster ***m = &a_array;
return *m;

불필요하게 번거롭고 다음과 같이 단순화 할 수 있습니다.

return a_array;

이러한 수정 사항을 모두 적용한 후 코드는 다음과 같아야합니다.

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

typedef struct monster {
    int id;
    char *name;
    char *element;
    int population;
} monster;

monster* createMonster(char *name, char *element, int population)
{
    monster *amonster = (monster*)malloc(sizeof(monster));
    amonster->name = (char*)malloc(sizeof(char)*(strlen(name)+1)); //fixed
    amonster->element = (char*)malloc(sizeof(char)*(strlen(element)+1)); //fixed
    //amonster->population = (int)malloc(sizeof(int)*sizeof(population)); //redundant
    strcpy( amonster->name, name ); //fixed
    strcpy( amonster->element, element ); //fixed
    amonster->population = population;
    return amonster;
}

monster** readMonsters(FILE* infile, int *monsterCount)
{
    char n[15], e[15];
    int p;
    monster **a_array = malloc(*monsterCount * sizeof(struct a*)); //what is struct a???
    for (int i = 0; i < *monsterCount; i++) {
        
        //a_array[i] = malloc(sizeof(monster)); //redundant
        
        fscanf(infile,"%s %s %d",n, e, &p);
        printf("%s %s %d\n", n, e, p);
        a_array[i] = createMonster(n,e,p);
       
    }
    //monster ***m = &a_array; //removed
    //return *m; //removed
    return a_array;
}

int main(){
    int monsterCount;
    char name[15];
    
    FILE *fp = fopen ( "in.txt", "r" );
    fscanf(fp,"%d %s",&monsterCount,name);
    //printf("test %d\n",monsterCount);
    monster **mptr = readMonsters(fp,&monsterCount);
    
    printf("%s %s %d\n", (mptr)[3]->name,(mptr)[3]->element,(mptr)[3]->population);
    
    fclose(fp);
    return 0;
}

여기 에 설명 된대로 이제 프로그램이 올바른 출력을 제공합니다.