Comment bien malloc pour le tableau de struct en C

Je vais lire dans les deux char* (ou de chaînes de caractères) à l'aide de strtok, et depuis ces deux caractères sont liés, (address : command\n) j'ai décidé d'utiliser une structure.

struct line* array = (struct line*)malloc(sizeof(file) * sizeof(struct line*));

Cette ligne mallocing espace pour la fonction me donne une erreur de segmentation et je me demandais si vous pouvez me dire une bonne façon de malloc de l'espace. Pour le contexte, voici le reste de mon code:

struct line
{
    char* addr;
    char* inst;
};
while loop{
    x = strtok(line,": ");
    y = strtok(NULL,"\n");
    strcpy(array[i].addr,x); //assume that x and y are always 3characters
    strcpy(array[i].inst,++y);
    i++;
}
Gardez à l'esprit que l'allocation de la mémoire pour un tableau de struct line ne pas allouer de la mémoire pour le addr et inst cordes. Selon la façon dont ces pointeurs à l'intérieur de la struct line sont utilisés, vous devrez peut-être également allouer de la mémoire pour les données de la chaîne.

OriginalL'auteur o0tomato0o | 2013-10-27

  1. 16

    L'allocation de fonctionnement est le même pour tous les types. Si vous avez besoin d'allouer un tableau de line structs, vous n'avez qu'à:

    struct line* array = malloc(number_of_elements * sizeof(struct line));

    Dans votre code, vous avez été de l'allocation d'un tableau qui avait la taille appropriée pour line pointeurs, pas pour line des structures. Notez également qu'il n'y a pas de raison de jeter la valeur de retour de malloc().

    Remarque que c'est mieux de style à utiliser:

    sizeof(*array)

    au lieu de:

    sizeof(struct line)

    La raison pour cela est que l'allocation continuera de fonctionner comme prévu dans le cas où vous modifiez le type de array. Dans ce cas, il est peu probable, mais c'est juste une chose générale, la peine de s'y habituer.

    Également noter qu'il est possible d'éviter d'avoir à répéter le mot struct maintes et maintes fois, par typedefing la structure:

    typedef struct line
{
    char* addr;
    char* inst;
} line;

    Vous pouvez alors tout simplement:

    line* array = malloc(number_of_elements * sizeof(*array));

    Bien sûr, n'oubliez pas d'allouer de la mémoire pour array.addr et array.inst.

    OriginalL'auteur Nikos C.

  2. 3

    Pour ce que vous avez décrit, Vous n'avez pas besoin d'allouer de la mémoire pour votre struct, plutôt, vous avez besoin d'allouer de la mémoire pour les membres char *addr;, et char *inst;. Si vous voulez avoir un seul exemplaire de cette structure, la première section de code montre comment initialiser, et attribuer des valeurs. Si vous voulez un tableau, le deuxième exemple de code illustre les différences.

    Cet exemple illustre comment allouer de la mémoire pour les membres d'une unique struct ligne: 

    typedef struct
{
    char* addr;
    char* inst;
}LINE;

LINE line;  

int main(void)
{   

    strcpy(line.addr, "anystring"); //will fail
    line.addr = malloc(80);
    line.inst = malloc(80);
    strcpy(line.addr, "someString");//success;
    strcpy(line.inst, "someOtherString");//success;

}

    Pour le tableau de struct ligne... 

    typedef struct
{
    char* addr;
    char* inst;
}LINE;  //same struct definition

LINE line[10]; //but create an array of line here.

int main(void)
{   
    int i;

    for(i=0;i<10;i++)
    {
      line[i].addr = malloc(80);
      line[i].inst = malloc(80);
    }

    for(i=0;i<10;i++)
    {
        strcpy(line[i].addr, "someString");
        strcpy(line[i].inst, "someOtherString");
    }
    //when done, free memory
    for(i=0;i<10;i++)
    {
        free(line[i].addr);
        free(line[i].inst);
    }      


}
    Ou utiliser strdup() et utiliser le reste de l'80 caractères pour quelque chose de plus utile. 🙂
    Ne pouvez pas argumenter plus utile.

    OriginalL'auteur ryyker