2

我试过环顾四周并尝试了所有解决方案，但我似乎无法解决我的问题。我知道我在 push_front 线上遇到了分段错误，但我只是迷路了。这是代码-

#include <iostream>
#include <fstream>
#include <sstream>
#include <list>

using namespace std;

typedef std::list<int> hSlots; //the list
typedef hSlots* hTable; //an array of lists

class HashTable
{
private:
int p; //p=number of slots in the hash table
hTable tmpPtr;
hTable *table;

 public:
HashTable(int p1);
int h1(int k);
~HashTable();

void chainedHashInsert(int x);

};

 HashTable::HashTable(int p1)
 {
p=p1;
hTable tTable[p];

//initializing to empty lists
for (int i=0; i<p; i++)
{
    tmpPtr = new hSlots;
    tTable[i] = tmpPtr;
}

table = tTable;
}

//destrcutor
HashTable::~HashTable()
{
delete table;
delete tmpPtr;
}

void HashTable::chainedHashInsert(int x)
{
tmpPtr = table[h1(x)];
cout<<"hashed"<<endl;
tmpPtr->push_front(x); //segmentation fault
}

int HashTable::h1(int k)
{
    int z = k%p;
    return z;
}

我没有使用很多列表，所以我不太确定

4

3 回答 3

2

也许这毕竟是一个正确的答案。

您的问题来自手动执行内存管理（错误），而实际上没有必要，在 C++ 中。

这是我在 C++ 中使用直接自动内存管理的看法：

#include <vector>
#include <list>

using namespace std;

template <typename T, typename hSlots = std::list<T> >
class HashTable
{
private:
    int p; //p=number of slots in the hash table
    std::vector<hSlots> table;
    int getbucket(int k) { return k%p; }

public:
    HashTable(int p1) : p(p1), table(p1) {}

    void chainedHashInsert(int x)
    {
        auto& tmpPtr = table[getbucket(x)];
        tmpPtr.push_front(x);
    }
};

int main()
{
    HashTable<int> table(37);
}
于 2013-03-03T02:01:21.260 回答
0 
table = tTable;

这条线是问题（或至少其中之一）。

您将指向自动对象的指针存储到成员变量中，然后在对象被销毁后取消引用（并删除！）它。

于 2013-03-03T01:48:58.317 回答
0

因为tTable是 的局部变量，所以当方法返回并作为悬空指针离开HashTable时它会消失。因此，要摆脱这种情况，请执行以下操作；使用 . 为表格创建空间。HashTabletablenew

HashTable::HashTable(int p1)
 {
p=p1;
table  = new ttTable[p];

//initializing to empty lists
for (int i=0; i<p; i++)
{
    tmpPtr = new hSlots;
    table[i] = tmpPtr;
}
}
于 2013-03-03T01:47:53.340 回答