Ho implementato la ricerca binaria, la ricerca lineare e una tabella hash per confrontare ogni complessità temporale. Il problema è che in qualche modo, la mia tabella hash è molto più lenta della ricerca binaria quando misuro il tempo per trovare i numeri primi. Qui di seguito è il mio codice:La mia tabella hash è più lenta della ricerca binaria
// Make the hash table 20 times the number of prime numbers
HashTable::HashTable(std::vector<int> primes)
{
int tablesize = primes.size() * 20;
table = new std::list<int>[tablesize];
size = tablesize;
for (auto &prime : primes)
this->insert(prime);
}
// Hash function
int HashTable::hash(int key)
{
return key % size;
}
// Finds element
int HashTable::find(int key)
{
// Get index from hash
int index = hash(key);
// Find element
std::list<int>::iterator foundelement = std::find(table[index].begin(), table[index].end(), key);
// If element has been found return index
// If not, return -1
if (foundelement != table[index].end())
return index;
else
return -1;
}
// Adds element to hashtable
void HashTable::insert(int element)
{
// Get index from hash and insert the element
int index = hash(element);
table[index].push_back(element);
}
HashTable.h
#ifndef HASHTABLE_H
#define HASHTABLE_H
#include <list>
#include <iostream>
#include <vector>
class HashTable
{
private:
// Each position in Hashtable has an array of lists to store elements in case of collision
std::list<int>* table;
// Size of hashtable
int size;
// Hashfunction that returns the array location for the given key
int hash(int key);
public:
HashTable(int tablesize);
HashTable(std::vector<int> primes);
// Adds element to hashtable
void insert(int element);
// Deletes an element by key
void remove(int key);
// Returns an element from hashtable for a given key
int find(int key);
// Displays the hashtable
void printTable();
// Display histogram to illustrate elements distribution
void printHistogram();
// Returns the number of lists in hash table
int getSize();
// Returns the total number of elements in hash table
int getNumberOfItems();
// De-allocates all memory used for the Hash Table.
~HashTable();
};
#endif
Ho già provato a superare la dimensione della dimensione della tabella per eliminare le collisioni, ma non ho notato alcuna differenza.
Questo è un grafico molto bello. Sembra che ci si aspetterebbe: la ricerca di hash ha una complessità a tempo costante e il binario ha uno logaritmico. È solo che la costante del tavolo hash è piuttosto grande. I vettori funzionano molto bene con le cache. – PSkocik
Probabilmente non correlato al benchmark; ma il costruttore dovrebbe accettare i 'primes' per riferimento –
Cosa succede ai tuoi tempi se cambi il tipo di' table' in 'std :: vector *'? –
msandiford