Question

#define TABLE_SIZE 100489 // must be a power of 2
typedef map<string,int> MAP_TYPE;
typedef pair<string, int> PAIR_TYPE;

class HashTable
{
public:                                     //public functions
    HashTable();
    ~HashTable();
    int find(string);
    bool insert(string, int);
private:
    int hash( const char* );
    vector< MAP_TYPE > v;
};

//HashTable constructor
HashTable::HashTable()
{
    vector< MAP_TYPE > v;               //initialize vector of maps
    v.reserve(TABLE_SIZE);                  //reserve table size
    MAP_TYPE m;
    for (int i = 0; i < TABLE_SIZE; ++i)    //fill vector with empty maps
    v.push_back(m);
    cout << v.size();
}

int HashTable::find(string key)
{
    cout << "in find" << '\n';
    //String to const char* for hash function
    const char *c = key.c_str();
    //find bucket where key is located
    int hashValue = HashTable::hash(c);
    cout << hashValue << '\n';
    string s = key;
    cout << v.size(); //Prints 0 but should be TABLE_SIZE
    //look for key in map in bucket
    MAP_TYPE::const_iterator iter = v[hashValue].find(s);
    if ( iter != v[hashValue].end()) //check if find exists
        return iter->second; //return value of key
    else
        return -1; //arbitrary value signifying failure to find key
}

int main()
{
    HashTable my_hash;
    string s = "hi";
    int z = my_hash.find(s);
    cout << z; //should return -1
    return 0;
}

我正在测试我的哈希表的查找功能，但它正在返回一个分段错误。即使我在find函数中构造了具有正确大小的向量v，但现在大小为0？我不认为它访问相同的变量。哈希函数很好。怎么了？

Answer 1

在C ++类中，变量最好在初始化列表中初始化：

HashTable::HashTable() : v(TABLE_SIZE, MAP_TYPE()) // **here**
{}

std::vector有一个构造函数，它接受一个大小和一个默认值，所以你可以调用它。实际上，由于要使用的默认值实际上是使用默认构造函数创建的映射，实际上您可以编写以下内容，因为在这种情况下可以省略第二个参数：

HashTable::HashTable() : v(TABLE_SIZE)
{}

Answer 2

您实际上从未在私有成员变量向量v中插入任何内容。您可以在HashTable构造函数中创建一个向量作为本地堆栈变量，您也名称v。然后，您继续将项插入到临时向量中，一旦离开构造函数，该向量就会超出范围。当然，稍后，在HashTable::find函数中，类成员变量v为空。

你的构造函数应该是这样的：

//HashTable constructor
HashTable::HashTable()
{
    this->v.reserve(TABLE_SIZE);                  //reserve table size
    MAP_TYPE m;
    for (int i = 0; i < TABLE_SIZE; ++i)    //fill vector with empty maps
        this->v.push_back(m);
}

上面我使用this关键字来强调我正在访问私有成员变量v，但您不需要它。（尽管通常最好采用私有成员变量的命名约定，例如添加_后缀或m_前缀等。）

初始化向量的更好方法是简单地使用向量的填充构造函数，如R. Martinho Fernandes's回答中所述。

Answer 3

在名为v的函数中有一个局部变量，以及一个私有类成员v。

尝试： this->v

分段错误访问私有类变量

3 个答案: