删除item-hashtable

时间:2018-05-03 22:19:15

标签: c data-structures hashtable

我正在开发一个项目来放入哈希表(解决与单独链接的冲突)由28 * 28数组形成的图像的ID,这些数据仅存储在1D数组中。 Hashatable是指向Node的指针数组,其中包含图像及其ID。 get函数返回ID并且它完美地工作,除非我删除特定图像然后尝试获取它...生成无限循环并且旧ID仍然存在!!

int get(image img)
{
    int i = hashCode(img);
    if(hashtable[i])
    {
        Node* temp = hashtable[i];
        //prints the ID which is suposed to be deleted
        printf("%d\n\n",temp->info.key);
        int found ;
        while(temp)
        {` `//infinite loop
            found = 1;
            for(i =0; i<sizeOfFile; i++)
            {
                if(temp->info.data.img_arr[i]!=img.img_arr[i])
                {
                    found = 0;
                    break;
                }
            }` 

            if(found==0)
            {
                temp = temp->next;
            }
            else
            {
                return temp->info.key;
            }
        }
    }

    return -1;
}

返回功能

  int removi(image img)
{    Node*prev=NULL;
    int i = hashCode(img);
    if(hashtable[i])
    {  Node* temp = hashtable[i];
        int found ;
        while(temp)
        {    found = 1;
            for(i =0; i<sizeOfFile; i++)
            {    if(temp->info.data.img_arr[i]!=img.img_arr[i])
                {  found = 0;
                    break;
                }
            }
            if(found==0)
            {   prev = temp;
                temp = temp->next;
            }else{
                int value = temp->info.key;
                if(prev){
                    prev->next = temp->next;
                }else{
                    hashtable[i]=temp->next;
                    //prints correctly the ID of next Node
                    printf("%d in remove\n",hashtable[i]->info.key);
                }
                free(temp);
                return value;
            }
        }
    }
    return -1;
}

1 个答案:

答案 0 :(得分:2)

removi功能中,您正在重复使用索引值i。稍后您的for循环修改了值int i = hashCode(img);for(i =0; i<sizeOfFile; i++);

因此,要移除元素的行(当它是列表中的head元素时):hashtable[i]=temp->next; 并没有实际删除它 - 相反,如果i超出界限,它会破坏其他一些列表或调用未定义的行为。