使用动态数组的空指针错误重新分配

Question

我正在尝试基于动态数组创建动态集抽象数据类型。但是，当一个数组已满并且我调用realloc时，我得到一个编译器错误，它是：realloc（）：下一个大小无效我的代码如下：

struct arraystack{
    int top;
    int capacity;
    void **array;
    size_t data_size; //size of my data type
};

struct arraystack * createStack(int capacity,size_t size_data_type){
    struct arraystack *stack= (struct arraystack*) malloc(sizeof(struct arraystack));
    stack->capacity=capacity;
    stack->top=-1;
    stack->data_size=size_data_type;
    stack->array= malloc(stack->capacity*size_data_type);
    return stack;  
}

void push(struct arraystack *stack, void* data){
     if(isFull(stack)){
         printf("Invoco realloc\n");
         stack->capacity=(stack->capacity*2)+1;
         void **tmp=realloc(stack->array,(stack->capacity*stack->data_size)); //this line gives me an error
         if(!tmp){
             printf("Attenzione memoria insufficente!\n");
             exit(1);
         }
         stack->array=tmp;
     }
     stack->array[++stack->top]=data;
     printf("%d pushed to stack\n", data); 
}

int isFull(struct arraystack *stack){
    return stack->top == stack->capacity - 1; 
}

int main(){
    struct arraystack *tmp=createStack(5,sizeof(int));

    int i;
    for(i=0;i<=20;i++){
        push(tmp,i);
        printf("top-> %d memoria allocata->%d\n",tmp->top,tmp->capacity);
    }

}

Answer 1

目前尚不清楚你是否需要void**或void*。

这条线有问题：stack->array= malloc(stack->capacity*size_data_type);

您永远不会正确初始化此内存区域。它应该为你有物品的void*分配空间。这与指向项的数据大小无关。每个这样的指针都需要初始化。您需要跟踪指针的总数以及每个指向项的大小。

或者你应该放弃void**而不是void*，如果这是为了指向一维数组或单个对象。

stack->array[++stack->top]=data;看起来很腥，这使得指数0未初始化。

一定要包括stdlib.h，否则你会在石器时代C90编译器上遇到严重的错误，你的演员隐藏的错误。