C++自定义HashTable的用法给出了SegFault。

Question

我必须为一个项目实现一个Linked HashTable。现在我必须用我的HashTable提出一个练习和一个解决方案。一切都很好，除了我得到随机的Segfault错误。

我说的随机是指。是同一行代码引起的，但总是在不同的时间调用。我在Atom、Codeblocks和Visual Studio Code中测试了我的代码。Atom和CB都抛出了SegFault错误，但VS Code运行起来却没有问题。

注意：这不是完整的代码。它是一个头文件的一部分，包含在main.cpp文件中，然后被编译并运行。代码。

#include <iostream>
#include <typeinfo>
#include <string>

using namespace std;

//List:
template<class T>
struct Node
{
  string data;
  Node *next;
};

class List
{
  private:
    Node *head, *tail;
    int length;
    friend class HashTable;
  public:
    List();
    List(const List &L);
    //~List() {delete this;};
    List& operator =(List L);
    int find(string);
    void insert(string value);
    void remove_head();
    void remove_poz(int);
    void remove_tail();
    void clear();
    void display();
};

List::List()
{
   head = NULL;
   tail = NULL;
   length = 0;
}

template<>
string List<string>::findByIndex(int ind)
{
    int i = 0;
    Node<string>* temp = new Node<string>;
    temp = head;
    while (temp != NULL)
    {
        i++;
        if (i == ind) return temp->data;
        temp = temp->next;
    }
    delete temp;
    return "-1";
}

template<class T>
void List<T>::remove_head()
{
  Node<T>* temp = new Node<T>;
  temp = head;
  head = head->next;
  delete temp;
  length--;
}

template<class T>
void List<T>::remove_pos(int pos)
{
  int i;
  Node<T>* curr = new Node<T>;
  Node<T>* prev = new Node<T>;
  curr = head;
  for (i = 1; i < pos; ++i)
  {
      prev = curr;
      curr = curr->next;
  }
  if (curr)
  {
    prev->next = curr->next;
    length--;
  }
  else cout << "Error" << endl;
}

template<class T>
void List<T>::remove_tail()
{
  Node<T>* curr = new Node<T>;
  Node<T>* prev = new Node<T>;
  curr = head;
  while (curr->next != NULL)
  {
      prev = curr;
      curr = curr->next;
  }
  tail = prev;
  prev->next = NULL;
  delete curr;
  length--;
}

//HashTable:
class HashTable
{
  private:
    List *table;
    float load, stored;
    int slots;
    friend class List;
  public:
    HashTable();
    HashTable(int);
    ~HashTable();
    int hashFunc(string key);
    int findTable(string);
    int findList(string);
    HashTable& operator =(const HashTable&);
    void resize();    //I need this one
    void insert(string);
    void remove(string);
    void clear(int);
    void clear();
    void display();
};

HashTable::HashTable()
{
  stored = 0;
  load = 0.00;
  slots = 15;
  table = new List[slots];
}

int HashTable::hashFunc(string key)
{
  int g, h = 0;
    unsigned int i;
    for (i = 0; i < key.size(); ++i)
    {
        h = (h << 4) + (int)(key[i]);
        g = h & 0xF0000000L;
        if (g != 0)
        {
            h = h ^ (g >> 24);
        }
        h = h & ~g;
    }
    return h % slots;
}

template<class T>
void HashTable<T>::remove(T value)
{
  int ind = hashFunc(value);
  int findInd = table[ind].findByValue(value);


  if (findInd == 0)
      table[ind].remove_head();

  else if (findInd < table[ind].length)
      table[ind].remove_pos(findInd);

  else table[ind].remove_tail();

  if (table[ind].isEmpty()) occupied--;
  stored--;
  load = stored / slots;
}

导致segfault的函数:(这个函数会在一个循环中被反复调用，直到我的表中没有更多的元素)

string reakcio(HashTable<string>& HT, int tarolok)
{
    const int anyagszam = rand() % 4 + 2; //Min 2, Max 5 anyag hasznalodik
    int i = 0, j;
    string anyagok[5];
    string eredmeny;

    for(j = 0; j < tarolok && i < anyagszam; ++j) //elemek kivetele
    {
        while(!HT.table[j].isEmpty())
        {
            anyagok[i++] = HT.table[j].findByIndex(1); //This line right here is the culprit :(
            HT.remove(anyagok[i-1]);
        }
    }

    const int siker = rand() % 4 + 0; //75% esely a sikerre
    if (siker)
    {
        eredmeny = anyagok[0];
        for(i = 1; i < anyagszam; ++i)
            eredmeny += " + " + anyagok[i];
    }
    else
        eredmeny = "Sikertelen reakcio";
    return eredmeny;
}

(注：这里只显示可能需要的功能)

在main.cpp的函数调用之前，我的hashtable或者我的list中的每个元素都是一个10个字符长的随机字符串value.srand(time(NULL))。

非常感谢任何帮助或建议，因为我被卡在这里，我真的需要继续进行下一部分的练习，但我不能没有这个。

main.cpp文件。

#include <iostream>
//#include "LinkedHash.h"
#include "functions.cpp"

int main()
{
  HashTable<string> Anyagok;
  int tarolok;

  tarol(Anyagok); //Stores the data from file, no problem here, functions.cpp
  tarolok = Anyagok.getSlots();

  srand(time(NULL));
  int i = 1;
  while (Anyagok.getStored() > 5 )
    cout<<reakcio(Anyagok, tarolok)<<" "<<i++<<endl;

  return 0;
}

LinkedHash.h包含了哈希特表和列表, functions.cpp包含了有问题的函数.

EDIT:根据建议，我改掉了

Node<string>* temp = new Node<string>;
temp = head;

部分

Node<string>* temp = head;

也删除了删除行.但我的问题还是一样。

Answer 1

一切都很好，除了我得到随机的Segfault错误。

那就什么都不能用了。

初次审查显示出对列表类中的角标没有什么关心。你需要为列表类定义一个正确的行为

• 空单操作
• 首尾操作
• 检索不到钥匙

发现明显的错误。

• remove_head, remove_tail在空列表上会出现segfault. head是NULL. head->next是无效的内存访问. 类似的错误在实现中到处都是。
• HashTable<T>::remove(T value) 总是会删除一些东西。即使值参数不在hashtable中。这是有很大缺陷的
• findByIndex返回"-1 "是没有意义的。"-1 "是一个有效的输入。
• Node<T>* temp = new Node<T>;temp = head;. 你刚刚泄露了内存。你需要一个 指针 到 操纵节点地址. 你不应该通过实例化Nodes来获得一个指针。这对于一个小项目来说不是问题（即不明显），但对于真正的实现来说是不可接受的。