fixed chuanxshi#56 added note

christianbirg · Jun 24, 2013 · 60482cb · 60482cb
1 parent bd76e07
commit 60482cb
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/function-patterns/memoization.html b/function-patterns/memoization.html
@@ -43,6 +43,11 @@
 
 			// preferred method 2
 			// multiple arguments using JSON stringify
+
+			// this will only work correctly for primitive (and Array) arguments, 
+			// since the order in which properties of Objects are enumerated is undetermined 
+			// and can even change between enumerations. 
+			// (a JSON encoder that sorts the keys won't have this issue).
 
 			var myFunc = function () {
 				var cachekey = JSON.stringify(Array.prototype.slice.call(arguments)),