2012-11-18 59 views
4

我创建了一个名为“mcguffins”的结构数组,我得到了一个非常奇怪的错误。C指针没有正确分配

//prints the info in a mcguffin 
    void printInfo(int i,struct mcguffin * new) { 
     printf("%d \tNum: %d\t Word: %s\n", i, new->num, new->word); 
    } 

    //creates a new mcguffin 
    struct mcguffin * addMG(int n, char * w) { 
     printf("Expected output:\n\tNum: %d\tWord: %s\n", n, w); 
     struct mcguffin * new; 
     new = malloc(sizeof *new); 
     new->num = n; 
     strncpy(new->word, w, sizeof(char[20])); 
     printf("Actual output: \n\t"); 
     printInfo(1, new); 
     return new; 
    } 

//creates a list of mcguffin pointers, and sets these pointers to new mcguffins 
struct mcguffin ** writeList() { 
    struct mcguffin ** list = malloc(10 * sizeof(*list)); 
    list[0] = addMG(2, "Jeter"); 
    list[1] = addMG(14, "Granderson"); 
    list[2] = addMG(25, "Teixeira"); 
    list[3] = addMG(13, "Rodriguez"); 
    list[4] = addMG(24, "Cano"); 
    list[5] = addMG(33, "Swisher"); 
    list[6] = addMG(55, "Martin"); 
    list[7] = addMG(20, "Posada"); 
    list[8] = addMG(11, "Gardner"); 
    list[9] = addMG(42, "Mo"); 
    return list; 
} 

出于某种原因,列表[0]和列表[1]没有被分配给所创建的结构体,但是列表[2]通过表[9]是。 addMG工作正常,并且为列表[0]和列表[1]创建结构,但由于某些原因,当我尝试在它们上使用printInfo时,而不是在结构上打印信息时,它会打印出一个内存地址, >数字应该去,并打印出什么新的 - >单词。

0 Num: 30519472 Word: 
1 Num: 30519600 Word: 
2 Num: 25 Word: Teixeira 
3 Num: 13 Word: Rodriguez 
4 Num: 24 Word: Cano 
5 Num: 33 Word: Swisher 
6 Num: 55 Word: Martin 
7 Num: 20 Word: Posada 
8 Num: 11 Word: Gardner 
9 Num: 42 Word: Mo 

这可能是一些愚蠢的错误,因为我是新的C,但任何帮助,将不胜感激。

编辑:为了澄清,mcguffins声明在一个单独的头文件,像这样:

struct mcguffin { 

    int num; 
    char word[20]; 
}; 

回答

2
new = (struct mcguffin *)malloc(sizeof(struct mcguffin *)); 
                 ^^ 

你的指针麦高菲分配足够的空间。丢掉*。更好的是,将其更改为:

new = malloc(sizeof *new); 

您的list分配同样是错误的。您应该分配:

struct mcguffin **list = malloc(10 * sizeof *list); 
+0

我已经相应改变的代码,但我仍然得到真正奇怪的错误还有什么地方都被赋予不同的列表[0]和list [1]。 – user1834200

+0

@ user1834200如何声明'mcguffin'? – cnicutar

+0

它被声明在一个单独的头文件中,如下所示: struct mcguffin { int num; char word [20]; }; 这是我必须使用作为我给予的任务的一部分,所以我不能改变任何事情。 – user1834200