X-Git-Url: http://plrg.eecs.uci.edu/git/?p=c11tester.git;a=blobdiff_plain;f=hashtable.h;h=4758999a6a4f23148446a5f0cf6376480fa4910c;hp=8302000b03e9f9946eb064a8c5c61ca1d8f27b44;hb=0d3c4eca7d657d93c5a30681f22d9856847370f9;hpb=9e438bd5e9eec9cc1fb9d158f394f8998fd10cf2

diff --git a/hashtable.h b/hashtable.h
index 8302000b..4758999a 100644
--- a/hashtable.h
+++ b/hashtable.h
@@ -1,53 +1,54 @@
+/*      Copyright (c) 2015 Regents of the University of California
+ *
+ *      Author: Brian Demsky <bdemsky@uci.edu>
+ *
+ *      This program is free software; you can redistribute it and/or
+ *      modify it under the terms of the GNU General Public License
+ *      version 2 as published by the Free Software Foundation.
+ */
+
 /** @file hashtable.h
  *  @brief Hashtable.  Standard chained bucket variety.
  */
 
-#ifndef HASHTABLE_H
-#define HASHTABLE_H
+#ifndef __HASHTABLE_H__
+#define __HASHTABLE_H__
 
 #include <stdlib.h>
 #include <stdio.h>
+#include <string.h>
+#include "mymemory.h"
+#include "common.h"
 
 /**
- * Hashtable linked node class, for chained storage of hash table conflicts. By
- * default it is snapshotting, but you can pass in your own allocation
- * functions.
+ * @brief HashTable node
  *
  * @tparam _Key    Type name for the key
  * @tparam _Val    Type name for the values to be stored
- * @tparam _malloc Provide your own 'malloc' for the table, or default to
- *                 snapshotting.
- * @tparam _calloc Provide your own 'calloc' for the table, or default to
- *                 snapshotting.
- * @tparam _free   Provide your own 'free' for the table, or default to
- *                 snapshotting.
  */
-template<typename _Key, typename _Val, void * (* _malloc)(size_t), void * (* _calloc)(size_t, size_t), void (*_free)(void *)>
+template<typename _Key, typename _Val>
 struct hashlistnode {
 	_Key key;
 	_Val val;
-	struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> *next;
+};
 
-	void * operator new(size_t size) {
-		return _malloc(size);
-	}
+template<typename _Key, int _Shift, typename _KeyInt>
+inline unsigned int default_hash_function(_Key hash) {
+	return (unsigned int)(((_KeyInt)hash) >> _Shift);
+}
 
-	void operator delete(void *p, size_t size) {
-		_free(p);
-	}
-
-	void * operator new[](size_t size) {
-		return _malloc(size);
-	}
-
-	void operator delete[](void *p, size_t size) {
-		_free(p);
-	}
-};
+template<typename _Key>
+inline bool default_equals(_Key key1, _Key key2) {
+	return key1 == key2;
+}
 
 /**
- * Hashtable class. By default it is snapshotting, but you can pass in your own
- * allocation functions.
+ * @brief A simple, custom hash table
+ *
+ * By default it is snapshotting, but you can pass in your own allocation
+ * functions. Note that this table does not support the value 0 (NULL) used as
+ * a key and is designed primarily with pointer-based keys in mind. Other
+ * primitive key types are supported only for non-zero values.
  *
  * @tparam _Key    Type name for the key
  * @tparam _Val    Type name for the values to be stored
@@ -61,37 +62,33 @@ struct hashlistnode {
  * @tparam _free   Provide your own 'free' for the table, or default to
  *                 snapshotting.
  */
-template<typename _Key, typename _Val, typename _KeyInt, int _Shift=0, void * (* _malloc)(size_t)=malloc, void * (* _calloc)(size_t, size_t)=calloc, void (*_free)(void *)=free>
-	class HashTable {
- public:
+template<typename _Key, typename _Val, typename _KeyInt, int _Shift = 0, void * (*_malloc)(size_t) = snapshot_malloc, void * (*_calloc)(size_t, size_t) = snapshot_calloc, void (*_free)(void *) = snapshot_free, unsigned int (*hash_function)(_Key) = default_hash_function<_Key, _Shift, _KeyInt>, bool (*equals)(_Key, _Key) = default_equals<_Key> >
+class HashTable {
+public:
 	/**
-	 * Constructor
+	 * @brief Hash table constructor
 	 * @param initialcapacity Sets the initial capacity of the hash table.
 	 * Default size 1024.
 	 * @param factor Sets the percentage full before the hashtable is
 	 * resized. Default ratio 0.5.
 	 */
-	HashTable(unsigned int initialcapacity=1024, double factor=0.5) {
+	HashTable(unsigned int initialcapacity = 1024, double factor = 0.5) {
 		// Allocate space for the hash table
-		table = (struct hashlistnode<_Key,_Val, _malloc, _calloc,_free> **) _calloc(initialcapacity, sizeof(struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> *));
+		table = (struct hashlistnode<_Key, _Val> *)_calloc(initialcapacity, sizeof(struct hashlistnode<_Key, _Val>));
+		zero = NULL;
 		loadfactor = factor;
 		capacity = initialcapacity;
-		threshold = (unsigned int) (initialcapacity*loadfactor);
-		mask = (capacity << _Shift)-1;
-		size = 0; // Initial number of elements in the hash
+		capacitymask = initialcapacity - 1;
+
+		threshold = (unsigned int)(initialcapacity * loadfactor);
+		size = 0;	// Initial number of elements in the hash
 	}
 
-	/** Destructor */
+	/** @brief Hash table destructor */
 	~HashTable() {
-		for(unsigned int i=0;i<capacity;i++) {
-			struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> * bin = table[i];
-			while(bin!=NULL) {
-				struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> * next=bin->next;
-				delete bin;
-				bin=next;
-			}
-		}
 		_free(table);
+		if (zero)
+			_free(zero);
 	}
 
 	/** Override: new operator */
@@ -114,155 +111,263 @@ template<typename _Key, typename _Val, typename _KeyInt, int _Shift=0, void * (*
 		_free(p);
 	}
 
-	/** Reset the table to its initial state. */
+	/** @brief Reset the table to its initial state. */
 	void reset() {
+		memset(table, 0, capacity * sizeof(struct hashlistnode<_Key, _Val>));
+		if (zero) {
+			_free(zero);
+			zero = NULL;
+		}
+		size = 0;
+	}
+
+	void resetanddelete() {
 		for(unsigned int i=0;i<capacity;i++) {
-			struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> * bin = table[i];
-			while(bin!=NULL) {
-				struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> * next=bin->next;
-				delete bin;
-				bin=next;
+			struct hashlistnode<_Key, _Val> *bin = &table[i];
+			if (bin->key != NULL) {
+				bin->key = NULL;
+				if (bin->val != NULL) {
+					delete bin->val;
+					bin->val = NULL;
+				}
 			}
 		}
-		memset(table, 0, capacity*sizeof(struct hashlistnode<_Key, _Val, _malloc, _calloc, _free> *));
-		size=0;
+		if (zero) {
+			if (zero->val != NULL)
+				delete zero->val;
+			_free(zero);
+			zero = NULL;
+		}
+		size = 0;
 	}
 
-	/** Put a key value pair into the table. */
+	void resetandfree() {
+		for(unsigned int i=0;i<capacity;i++) {
+			struct hashlistnode<_Key, _Val> *bin = &table[i];
+			if (bin->key != NULL) {
+				bin->key = NULL;
+				if (bin->val != NULL) {
+					_free(bin->val);
+					bin->val = NULL;
+				}
+			}
+		}
+		if (zero) {
+			if (zero->val != NULL)
+				_free(zero->val);
+			_free(zero);
+			zero = NULL;
+		}
+		size = 0;
+	}
+
+	/**
+	 * @brief Put a key/value pair into the table
+	 * @param key The key for the new value; must not be 0 or NULL
+	 * @param val The value to store in the table
+	 */
 	void put(_Key key, _Val val) {
+		/* HashTable cannot handle 0 as a key */
+		if (!key) {
+			if (!zero) {
+				zero=(struct hashlistnode<_Key, _Val> *)_malloc(sizeof(struct hashlistnode<_Key, _Val>));
+				size++;
+			}
+			zero->key=key;
+			zero->val=val;
+			return;
+		}
+
 		if (size > threshold)
 			resize(capacity << 1);
 
-		struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> *ptr = table[(((_KeyInt)key) & mask)>>_Shift];
-		struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> *search = ptr;
+		struct hashlistnode<_Key, _Val> *search;
 
-		while(search!=NULL) {
-			if (search->key==key) {
-				search->val=val;
+		unsigned int index = hash_function(key);
+		do {
+			index &= capacitymask;
+			search = &table[index];
+			if (!search->key) {
+				//key is null, probably done
+				break;
+			}
+			if (equals(search->key, key)) {
+				search->val = val;
 				return;
 			}
-			search=search->next;
-		}
+			index++;
+		} while (true);
 
-		struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> *newptr=(struct hashlistnode<_Key,_Val,_malloc,_calloc,_free> *)new struct hashlistnode<_Key,_Val, _malloc, _calloc, _free>;
-		newptr->key=key;
-		newptr->val=val;
-		newptr->next=ptr;
-		table[(((_KeyInt)key)&mask)>>_Shift]=newptr;
+		search->key = key;
+		search->val = val;
 		size++;
 	}
 
 	/**
-	 * @brief Get a valid pointer to a value corresponding to a given key
-	 *
-	 * Ensure that key is present in the hash table, then return a pointer
-	 * to its value bin. This may require either creating a new bin for
-	 * this key (with a default-constructed value) or simply locating and
-	 * returning a pointer to an existing value.
-	 * @param key The key to check
-	 * @return A pointer to the value in the table
+	 * @brief Lookup the corresponding value for the given key
+	 * @param key The key for finding the value; must not be 0 or NULL
+	 * @return The value in the table, if the key is found; otherwise 0
 	 */
-	_Val * get_safe_ptr(_Key key) {
-		if (size > threshold)
-			resize(capacity << 1);
+	_Val get(_Key key) const {
+		struct hashlistnode<_Key, _Val> *search;
+
+		/* HashTable cannot handle 0 as a key */
+		if (!key) {
+			if (zero)
+				return zero->val;
+			else
+				return (_Val) 0;
+		}
 
-		struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> *ptr = table[(((_KeyInt)key) & mask)>>_Shift];
-		struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> *search = ptr;
+		unsigned int oindex = hash_function(key) & capacitymask;
+		unsigned int index=oindex;
+		do {
+			search = &table[index];
+			if (!search->key) {
+				if (!search->val)
+					break;
+			} else
+			if (equals(search->key, key))
+				return search->val;
+			index++;
+			index &= capacitymask;
+			if (index==oindex)
+				break;
+		} while (true);
+		return (_Val)0;
+	}
 
-		while(search!=NULL) {
-			if (search->key==key) {
-				return &search->val;
+	/**
+	 * @brief Remove the given key and return the corresponding value
+	 * @param key The key for finding the value; must not be 0 or NULL
+	 * @return The value in the table, if the key is found; otherwise 0
+	 */
+	_Val remove(_Key key) {
+		struct hashlistnode<_Key, _Val> *search;
+
+		/* HashTable cannot handle 0 as a key */
+		if (!key) {
+			if (!zero) {
+				return (_Val)0;
+			} else {
+				_Val v=zero->val;
+				_free(zero);
+				zero=NULL;
+				size--;
+				return v;
 			}
-			search=search->next;
 		}
 
-		struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> *newptr=(struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> *)new struct hashlistnode<_Key,_Val, _malloc, _calloc, _free>;
-		newptr->key=key;
-		newptr->next=ptr;
-		table[(((_KeyInt)key)&mask)>>_Shift]=newptr;
-		size++;
-		return &newptr->val;
-	}
-
-	/** Lookup the corresponding value for the given key. */
-	_Val get(_Key key) {
-		struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> *search = table[(((_KeyInt)key) & mask)>>_Shift];
 
-		while(search!=NULL) {
-			if (search->key==key) {
-				return search->val;
+		unsigned int index = hash_function(key);
+		do {
+			index &= capacitymask;
+			search = &table[index];
+			if (!search->key) {
+				if (!search->val)
+					break;
+			} else
+			if (equals(search->key, key)) {
+				_Val v=search->val;
+				//empty out this bin
+				search->val=(_Val) 1;
+				search->key=0;
+				size--;
+				return v;
 			}
-			search=search->next;
-		}
+			index++;
+		} while (true);
 		return (_Val)0;
 	}
 
-	/** Lookup the corresponding value for the given key. */
-	_Val * getptr(_Key key) {
-		struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> *search = table[(((_KeyInt)key) & mask)>>_Shift];
+	unsigned int getSize() const {
+		return size;
+	}
 
-		while(search!=NULL) {
-			if (search->key==key) {
-				return & search->val;
-			}
-			search=search->next;
-		}
-		return (_Val *) NULL;
+	bool isEmpty() {
+		return size == 0;
 	}
 
-	/** Check whether the table contains a value for the given key. */
-	bool contains(_Key key) {
-		struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> *search = table[(((_KeyInt)key) & mask)>>_Shift];
+	/**
+	 * @brief Check whether the table contains a value for the given key
+	 * @param key The key for finding the value; must not be 0 or NULL
+	 * @return True, if the key is found; false otherwise
+	 */
+	bool contains(_Key key) const {
+		struct hashlistnode<_Key, _Val> *search;
 
-		while(search!=NULL) {
-			if (search->key==key) {
-				return true;
-			}
-			search=search->next;
+		/* HashTable cannot handle 0 as a key */
+		if (!key) {
+			return zero!=NULL;
 		}
+
+		unsigned int index = hash_function(key);
+		do {
+			index &= capacitymask;
+			search = &table[index];
+			if (!search->key) {
+				if (!search->val)
+					break;
+			} else
+			if (equals(search->key, key))
+				return true;
+			index++;
+		} while (true);
 		return false;
 	}
 
-	/** Resize the table. */
+	/**
+	 * @brief Resize the table
+	 * @param newsize The new size of the table
+	 */
 	void resize(unsigned int newsize) {
-		struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> ** oldtable = table;
-		struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> ** newtable;
+		struct hashlistnode<_Key, _Val> *oldtable = table;
+		struct hashlistnode<_Key, _Val> *newtable;
 		unsigned int oldcapacity = capacity;
 
-		if((newtable = (struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> **) _calloc(newsize, sizeof(struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> *))) == NULL) {
-			printf("Calloc error %s %d\n", __FILE__, __LINE__);
-			exit(-1);
+		if ((newtable = (struct hashlistnode<_Key, _Val> *)_calloc(newsize, sizeof(struct hashlistnode<_Key, _Val>))) == NULL) {
+			model_print("calloc error %s %d\n", __FILE__, __LINE__);
+			exit(EXIT_FAILURE);
 		}
 
-		table = newtable;          //Update the global hashtable upon resize()
+		table = newtable;	// Update the global hashtable upon resize()
 		capacity = newsize;
-		threshold = (unsigned int) (newsize * loadfactor);
-		mask = (newsize << _Shift)-1;
+		capacitymask = newsize - 1;
 
-		for(unsigned int i = 0; i < oldcapacity; i++) {
-			struct hashlistnode<_Key, _Val, _malloc, _calloc, _free> * bin = oldtable[i];
+		threshold = (unsigned int)(newsize * loadfactor);
 
-			while(bin!=NULL) {
-				_Key key=bin->key;
-				struct hashlistnode<_Key, _Val, _malloc, _calloc, _free> * next=bin->next;
+		struct hashlistnode<_Key, _Val> *bin = &oldtable[0];
+		struct hashlistnode<_Key, _Val> *lastbin = &oldtable[oldcapacity];
+		for (;bin < lastbin;bin++) {
+			_Key key = bin->key;
 
-				unsigned int index = (((_KeyInt)key) & mask) >>_Shift;
-				struct hashlistnode<_Key, _Val, _malloc, _calloc, _free> * tmp=newtable[index];
-				bin->next=tmp;
-				newtable[index]=bin;
-				bin = next;
-			}
+			struct hashlistnode<_Key, _Val> *search;
+			if (!key)
+				continue;
+
+			unsigned int index = hash_function(key);
+			do {
+				index &= capacitymask;
+				search = &table[index];
+				index++;
+			} while (search->key);
+
+			search->key = key;
+			search->val = bin->val;
 		}
 
-		_free(oldtable);            //Free the memory of the old hash table
+		_free(oldtable);	// Free the memory of the old hash table
 	}
-
- private:
-	struct hashlistnode<_Key,_Val, _malloc, _calloc, _free> **table;
+	double getLoadFactor() {return loadfactor;}
+	unsigned int getCapacity() {return capacity;}
+	struct hashlistnode<_Key, _Val> *table;
+	struct hashlistnode<_Key, _Val> *zero;
 	unsigned int capacity;
-	_KeyInt mask;
 	unsigned int size;
+private:
+	unsigned int capacitymask;
 	unsigned int threshold;
 	double loadfactor;
 };
-#endif
+
+#endif	/* __HASHTABLE_H__ */