正在将对象插入哈希表...出于某种原因不让我?

Inserting object into hash table... won't let me for some reason?

本文关键字:某种原因 对象 插入 哈希表      更新时间:2023-10-16

我正在设计一个简单的程序,它接受我定义的Symbol类的对象并插入到HashTable中。我得到了来自我们的教科书的HashTable.h文件,正如你将看到的,它被设计为能够处理任何对象。

我正在尝试通过以下方式插入我的对象:

hashtable.insert(&temp) //where temp is the object 

但是,我收到以下错误:

    Driver.cpp: In function 'int main()':
    Driver.cpp:127:27: error: no matching function for call to 'HashTable<Symbol>::insert(Symbol*)'
    Driver.cpp:127:27: note: candidates are:
    In file included from Driver.cpp:12:0:
    SeperateChaining.h:50:10: note: bool HashTable<HashedObj>::insert(HashedObj&) [with HashedObj = Symbol]
    SeperateChaining.h:50:10: note:   no known conversion for argument 1 from 'Symbol*' to 'Symbol&'
    SeperateChaining.h:72:10: note: bool HashTable<HashedObj>::insert(HashedObj&&) [with HashedObj = Symbol]
    SeperateChaining.h:72:10: note:   no known conversion for argument 1 from 'Symbol*' to 'Symbol&&'

你们介意看看吗?

编辑:当我尝试按照你们许多人的建议按值插入时,我得到了大量的垃圾,但解析出错误有两个:

opt/local/include/gcc47/c++/bits/stl_algo.h:135:7: error: no match for 'operator==' in '__first.std::_List_iterator<_Tp>::operator*<Symbol>() == __val'

opt/local/include/gcc47/c++/bits/stl_algo.h:135:7: error: no match for 'operator==' in '__first.std::_List_iterator<_Tp>::operator*<Symbol>() == __val' – 

这是我的驱动程序文件和哈希表.h文件:

驱动程序.cpp:

    #include <iostream>
    #include <iomanip>
    #include <cassert>
    #include <fstream>
    #include <string>
    #include <vector>
    #include <time.h>
    #include <unistd.h>
    #include <map>
    #include <cstdlib>
    #include <cmath>
    #include "SeperateChaining.h"
    //#include "hash_chn.h"
    using namespace std;
    int TABLE_SIZE; //I know it's a global, but it allows the Table Size to be taken in within main() and used in hash()
    size_t hash(const string & key);
    class Symbol
    {
    private:
        int key;
        int type;
        string data;
    public:
        const string & getData() const
        {
            return data;
        }
        int getType() 
        {
            return type;
        }
        int getKey()
        {
            return labs(key);
        }
        void setType(int Type)
        {   
            type = Type;
        }
        void setData(string Data)
        {
            data = Data;
        }
        void setKey(int Key)
        {
            key = Key;
        }
    };
    int main()
    {
        HashTable<Symbol> hashtable(TABLE_SIZE);
        Symbol temp;
        vector<Symbol> symbols;
        string s;
        int t;
        int hash_key_array[TABLE_SIZE]; //array to hold hash key values
        ifstream file;
        file.open("symbols.txt");
        if(!file)
        {
            cout << "System failed to open file.";
        }
        else
        {
            cout << "File successfully opened" << endl;
        }
        //for loop to read in the string name and the integer that follows the string name from symbols.txt
        while(file >> s)
        {
            temp.setData(s);
            file >> t;
            temp.setType(t);
            symbols.push_back(temp);
        }
        for(int i = 0; i < symbols.size(); i++)
        {
            cout << symbols[i].getData() << "n";
            cout << symbols[i].getType() << "n";
        }
        cout << "What would you like the table size to be?" << endl;
        cout << "Note: If the table size is greater than the number of objects" <<
        " in the symbols.txt file, it will inevitably throw a segmentation fault" << endl;
        cin >> TABLE_SIZE;
        for(int j = 0; j < TABLE_SIZE; j++)
        {
            temp.setData(symbols[j].getData());
            cout << temp.getData() << endl;
            temp.setType(symbols[j].getType());
            cout << temp.getType() << endl;
            temp.setKey(::hash(symbols[j].getData()));
            cout << "The key is: " << temp.getKey() << endl;
            cout << endl;
            hash_key_array[j] = temp.getKey();

            for (int i = 0; i < TABLE_SIZE; i++)
            {
                if (i != j)
                {
                    if (hash_key_array[i] == hash_key_array[j])
                    {
                        cout << endl;
                        cout << "Collision occurred at " << hash_key_array[i] << endl;
                        //rehash();
                        //cout << "The new key is: " << temp.getKey() << endl;
                        break;
                    }
                }
            }
        hashtable.insert(&temp); //problem is here

        }
    }
    size_t hash(const string & key)
    {
        size_t hashVal = 0;
        for(char ch : key)
        {
            hashVal = 37 * hashVal + ch;
        }
        return labs(hashVal);
    }

SeperateChaining.h:

#ifndef SEPARATE_CHAINING_H
#define SEPARATE_CHAINING_H
#include <vector>
#include <list>
#include <string>
#include <algorithm>
#include <functional>
//#include "Hash.h"
using namespace std;
// SeparateChaining Hash table class
//
// CONSTRUCTION: an approximate initial size or default of 101
//
// ******************PUBLIC OPERATIONS*********************
// bool insert( x )       --> Insert x
// bool remove( x )       --> Remove x
// bool contains( x )     --> Return true if x is present
// void makeEmpty( )      --> Remove all items
template <typename HashedObj>
class HashTable
{
  public:
    //Uses the whatever value table_size has
    //Otherwise, it will make a hash table of size 101
    explicit HashTable( int TABLE_SIZE )
    { 
        currentSize = 0;
        theLists.resize(TABLE_SIZE); 
    }
    bool contains( const HashedObj & x ) const
    {
        //represents the correct list in the hash table vector to start looking through
        auto & whichList = theLists[ myhash( x ) ];
        //returns whatever you wanted to search for in the table provided it is there
        return find( begin( whichList ), end( whichList ), x ) != end( whichList );
    }
    void makeEmpty( )
    {
        for( auto & thisList : theLists )
            thisList.clear( );
    }
    bool insert(HashedObj & temp )
    {
         //represents the correct list in the hash table vector to start looking through
        auto & whichList = theLists[myhash( temp )];
        //goes through the beginning and end of the list, and if it
        //doesn't get to the end, then it found the object you wanted to insert in the hash table already
        //prevents duplicate insertions
        if( find( begin( whichList ), end( whichList ), temp ) != end( whichList) )
            return false;
        //otherwise, it has gotten to the end of the list without finding a duplicate
        //and puts what you want to insert in the list
        whichList.push_back( temp );
        // Rehash; see Section 5.5
        if( ++currentSize > theLists.size( ) )
            rehash( );
        return true;
    }
    bool insert( HashedObj && x )
    {
        auto & whichList = theLists[ myhash( x ) ];      
        if( find( begin( whichList ), end( whichList ), x ) != end( whichList ) )
            return false;
        whichList.push_back( std::move( x ) );
            // Rehash; see Section 5.5
        if( ++currentSize > theLists.size( ) )
            rehash( );
        return true;
    }
    bool remove( const HashedObj & x )
    {
        //represents the correct list in the hash table vector to start looking through
        auto & whichList = theLists[ myhash( x ) ];
        //trying to find x within the list
        //the iterator points to the slot in the list that contains x
        auto itr = find( begin( whichList ), end( whichList ), x );
        //if it gets to the end of the list without finding what you want to remove, then it returns false
        if( itr == end( whichList ) )
        {
            return false;
        }

        //if it finds x, it removes it from the list
        whichList.erase( itr );
        --currentSize;
        return true;
    }
    /*
    void printTable()
    {
        for(int i=0; i < symbols.size(); i++)
        {
            cout << "The hash table contains: " << symbols[i] << endl;
        }
    }
    */
  private: 
     vector<list<HashedObj>> theLists;   // The array of Lists
     int  currentSize;
     void rehash( )
     {
         vector<list<HashedObj>> oldLists = theLists;
         // Creates new double-sized, empty table
         theLists.resize( nextPrime( 2 * theLists.size( ) ) );
         for( auto & thisList : theLists )
             thisList.clear( );
         // Copies the old table into the new table
         currentSize = 0;
         for( auto & thisList : oldLists )
             for( auto & x : thisList )
                 insert( std::move( x ) );
     }
     size_t myhash( const HashedObj & x ) const
     {
         static hash<HashedObj> hf;
         return hf( x ) % theLists.size( );
        }
  };
#endif

hashtable.insert(&temp(

您必须按值插入,而不是按指针插入。删除 & 运算符。

hashtable.insert(&temp);
                 ^

您正在尝试插入临时地址。这就是此错误所说的:

注意:bool HashTable::insert(HashedObj&( [with HashedObj =

符号] 注意:参数 1 没有从"符号*"到 "符号&">

有一个insert版本将引用作为其参数。因此,而不是指针按值插入对象:

hashtable.insert(temp);

看看你得到的错误:

SeperateChaining.h:50:10: note: bool HashTable<HashedObj>::insert(HashedObj&) [with HashedObj = Symbol]
SeperateChaining.h:50:10: note:   no known conversion for argument 1 from 'Symbol*' to 'Symbol&'
SeperateChaining.h:72:10: note: bool HashTable<HashedObj>::insert(HashedObj&&) [with HashedObj = Symbol]
SeperateChaining.h:72:10: note:   no known conversion for argument 1 from 'Symbol*' to 'Symbol&&'

这告诉您正在使用Symbol*调用insert,并且函数insert采用Symbol&Symbol&&

因此,让我们看一下您调用insert的代码:

hashtable.insert(&temp); //problem is here

果然,您正在使用& 这是address-of operator,它将采用 temp 的地址,该地址为 Symbol ,返回指向它的指针(当然,类型为 Symbol*(。所以你用 Symbol* 调用函数,这不是函数所期望的。

旁注:您的实现还有其他一些问题需要解决。问问自己什么是tempHashTable类如何处理它接受的Symbol的 l 值和 r 值引用?