C - 如何修改函数内部的指针数组

Question

我正在尝试在C中实现哈希表，几乎拥有它。 我正在通过链接使用每个数组槽中的链表来实现冲突解决，我希望能够在函数调用内部进行链接。

现在，问题是为了使修改成为永久性的，我相信我需要一个额外的间接层。 这是一个问题，当我尝试遍历列表时，前一个元素被下一个元素覆盖（请参阅我在insert（）函数中的注释）。 我试图传递这个数组与额外的间接级别指定几种不同的方式，但我得到编译器警告和seg错误。

这对你们中的一些人来说可能看起来很简单，但是现在我已经抓住了很长一段时间了，这个场景（通过修改指针数组）在我的文本中没有得到处理，我似乎无法找到这个确切的问题问（虽然它可能是我不认识的形式）。 我不一定在寻找代码的“快速修复”，但我想了解完成我想要做的事情的最佳实践。

#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#include <time.h>
//#include "list.c"
struct node{
    int key;
    struct node* next;
};

void createTable(struct node *table[], int numEls, int numSlots);
int hash(int key, int numSlots);
void insert(struct node *table[], int key, int slot, int numSlots);
void display(struct node *table[], int numEls);

int main(void){
    srand(time(NULL));
    int numEls, numSlots;
    printf("Please enter the desired number of slots in the hash table: ");
    scanf("%d", &numSlots);
    struct node *table[numSlots];
    printf("\nPlease enter the desired number of elements: ");
    scanf("%d", &numEls);
    printf("\nYour load factor will be %f", (float)numEls/numSlots);
    createTable(table, numEls, numSlots);
}

void createTable(struct node *table[], int numEls, int numSlots){
    for(int i = 0; i < numSlots; i++)
        table[i] = NULL;
    for(int j = 0; j < numEls; j++){
        for(int k = 0; k < 99999999; k++){}//give the rand function time 
        int el = rand()%100;
        insert(table, el, hash(rand()%100, numSlots), numSlots);
    }
    display(table, numSlots);
}

int hash(int key, int numSlots){
    return((int)(pow(key, 2.819)) % numSlots);
}

void insert(struct node *table[], int key_, int slot, int numSlots){
    printf("\nInserting %d into slot %d", key_, slot);
    fflush(stdout);
    struct node* new = malloc(sizeof(struct node));
    (new)->key = key_;
    (new)->next = NULL;
    struct node** temp = &(table[slot]);
    if((*temp) == NULL){
        printf(" (*temp) == NULL");
        (*temp) = new;
    }
    else{
        printf(" %d", (*temp)->key);
        while((*temp)->next != NULL){
            printf(" %d", (*temp)->next->key);
                (*temp) = (*temp)->next;  //head is overwritten with head->next 
        }
        (*temp)->next = new;
        printf(" %d", (*temp)->next->key);
    }
}

void display(struct node *table[], int numSlots){
    for(int i = 0; i < numSlots; i++){
        printf("\nSlot %d:", i);
        struct node* temp = table[i];
        while(temp != NULL){
            printf(" %d", temp->key);
            temp = temp->next;
        }
    }
}

Answer 1

在注释行中，您将使用链接列表中的下一个元素覆盖指向头部（最外层数组内）的指针 ，这可能不是您的意图。

更正是沿着指针列表向下走，直到找到最后一个指针，而不在步行期间修改主数据结构。

这是insert()的更正版本。

void insert(struct node *table[], int key_, int slot, int numSlots){
    printf("\nInserting %d into slot %d", key_, slot);
    fflush(stdout);
    struct node* new = malloc(sizeof(struct node));
    new->key = key_;
    new->next = NULL;

    // Here we make a copy of the pointer to the head node in the linked list.
    // This way, we never overwrite the original copy which lives in the array itself.
    struct node* head = table[slot];
    if(head == NULL){
        printf(" head == NULL");
        table[slot] = new;
    }
    else{
        while(head->next != NULL) {
            head = head->next;  //head is overwritten with head->next 
        }
        head->next = new;
    }
}

Answer 2

这不是一个答案，但对于评论来说太大了......你能解释一下这段代码应该做什么吗？

while((*temp)->next != NULL)
{
    printf(" %d", (*temp)->next->key);
    (*temp) = (*temp)->next;  //head is overwritten with head->next 
}
(*temp)->next = new;

我希望您希望将新节点附加到此插槽中现有节点的链接列表的末尾。 但是这段代码实际上更新了头部以指向列表中的最后一个节点，就像你的评论所说的那样（泄漏内存 - 列表中的早期节点现在无法访问）。 然后它使最后一个节点（现在是列表中唯一的节点）指向new节点。

因此，您的“列表”只有长度1或2，并且每次尝试输入第三个条目时它会泄漏内存。

在我看来，你传递table （这是链表的头部列表）没有任何问题，但问题是维护列表的代码是不正确的。