1

我有以下代码：

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

typedef struct _item {
    int v, w;
} item;

void printItems(item *t, int n) {
int i;
    for(i= 0; i < n; i++)
    printf("%4i",t[i].v);
    printf("\n");
}

item * newItemSet(int n) {
    int i;
    item *t = (item*)malloc(sizeof(item));
    for (i = 0; i < n; i++){
        (t+i)->v = rand()%100;
        (t+i)->w = rand()%100;
    }
    return t;
}

item * mer(item*a, int n, item*b, int m) {
    int size = m+n;
    item*q = (item*)malloc(sizeof(item)*size);
    int i, c;
    c = 0;
    printf("a----\n");
    printItems(a,n);
    printf("b----\n");
    printItems(b,m);
    for(i = 0; i<n; i++) {
        q[c] = a[i];
        c++;
    }
    for(i = 0; i<m; i++) {
        q[c] = b[i];
        c++;
    }
    printItems(q,size);
    return q;
}   


int main(void) {
    srand(time(NULL));
    item * a = newItemSet(6);
    item *b = newItemSet(6);
    item *c = mer(a,6,b,6);
    printItems(c,12);   
    return 0;
}

在每次执行时， 的第一部分c都是正确的，直到 . 的最后两个元素为止b。我错过了什么？

4

2 回答 2

3

的分配newItemSet似乎不正确。它应该是：

item *t = malloc(sizeof(item) * n);
于 2013-11-11T18:58:53.997 回答
1 
item *t = (item*)malloc(sizeof(item));

应该：

item *t = malloc(n * sizeof(item));

您正在为n项目分配空间，而不是一个item。此外，不需要强制转换的返回值，malloc它可以隐藏错误。

于 2013-11-11T18:58:50.047 回答