Created
September 27, 2012 00:11
-
-
Save dtx/3791420 to your computer and use it in GitHub Desktop.
A simple HashMap implementation in Javascript, just something I wrote while reading DailyJS.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
var HashMap = function(){ | |
this._size = 0; | |
this._map = {}; | |
} | |
HashMap.prototype = { | |
put: function(key, value){ | |
if(!this.containsKey(key)){ | |
this._size++; | |
} | |
this._map[key] = value; | |
}, | |
remove:function(key){ | |
if(this.containsKey(key)){ | |
this._size--; | |
var value = this._map[key]; | |
delete this._map[key]; | |
return value; | |
} | |
else{ | |
return null; | |
} | |
}, | |
containsKey: function(key){ | |
return this._map.hasOwnProperty(key); | |
}, | |
containsValue: function(value){ | |
for(var key in this._map){ | |
if(this._map.hasOwnProperty(key)){ | |
if(this._map[key] === value){ | |
return true; | |
} | |
} | |
} | |
return false; | |
}, | |
get: function(key){ | |
return this.containsKey(key) ? this._map[key] : null; | |
}, | |
clear: function(key){ | |
this.size = 0; | |
this._map= {}; | |
}, | |
keys: function(){ | |
var keys = []; | |
for(var keys in this._map){ | |
if(this._map.hasOwnProperty(key)){ | |
keys.push(key); | |
} | |
} | |
return keys; | |
}, | |
values: function(){ | |
var values[]; | |
for(var key in this._map){ | |
if(this._map.hasOwnProperty(key)){ | |
values.push(this._map[key]); | |
} | |
} | |
}, | |
size: function(){ | |
return this._size; | |
} | |
}; | |
Hello, thank you for the code.
Just add return inside "values" function.
And in function "keys" the for loop variable is key
for(var key in this._map)
How is this a hash map if there's no hashing/equals checks?
You are writing a HashMap, with a HashMap.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Nice work, thanks for putting this out into the ether!
Unrelated to my compliment, I think line 62 is missing an
=
. :)