如何正确地将包含int数组的结构分配给struct数组?

时间:2016-10-05 03:20:23

标签: c arrays struct flexible-array-member

我想知道如何将包含int数组的结构分配给结构数组。无论我想到什么新的解决方案,我都会得到错误的结果。

我认为问题在于这段代码:

struct Codes *create(int as) {
    struct Codes *c = malloc(sizeof (struct Codes)+as * sizeof (int));
    c->as = as;
    for (int i = 0; i < as; i++) {
        c->a[i] = i;
    }

    return c;
}

整个代码:

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

struct Codes {
    int as;
    int a[];
};

struct Code {
    int as;
    struct Codes *ci[];
};

struct Codes *create(int as) {
    struct Codes *c = malloc(sizeof (struct Codes)+as * sizeof (int));
    c->as = as;
    for (int i = 0; i < as; i++) {
        c->a[i] = i;
    }

    return c;
}

struct Code *and(int as, struct Codes *cd) {
    struct Code *c = malloc(sizeof (struct Code)+as * sizeof (struct Codes));
     for (int i = 0; i < as; i++) {
        c->ci[i] = cd;
    }
    c->as = as;
    return c;
}

int main(int argc, char **argv) {

    struct Codes *cd;
    cd = create(4);

    struct Code *c;
    c = and(2, cd);

    for (int i = 0; i < c->as; i += 1) {
        for (int j=0; j < c->ci[i]->as; j++) {
            printf("%d \n", c->ci[i]->a[j]);
       }
    }

    free(cd);
    free(c);

}//main

实际结果:

0 
1 
2 
3 

预期结果:

0 
1 
2 
3
0
1
2
3 

1 个答案:

答案 0 :(得分:1)

struct Code *c = malloc(sizeof (struct Code)+as * sizeof (struct Codes));不正确。 struct Code的ci是一个指针数组,但是你为一个结构数组分配了空间。

要解决此问题,请更改为sizeof(struct Codes *),或者最好使用取消引用指向您为其分配空间类型的指针的模式:

struct Code *c = malloc( sizeof *c + as * sizeof c->ci[0] );

此外,for (int j;应为for (int j = 0;。您的代码使用未初始化的j值导致未定义的行为,这恰好是您碰巧获得输出的机会。使用gcc标志-Wextra会诊断出此错误。