Merge pull request #103 from ihcsim/issue101

chuanxshi · web-flow · commit 10da63a0a0c9 · 2018-06-09T23:54:20.000+08:00
Fix Issue 101 - Better jQuery Pattern Append
diff --git a/jquery-patterns/append.html b/jquery-patterns/append.html
@@ -7,7 +7,8 @@
 	<body>
 		<script>
 			/* Title: append
-			 * Description: use string concatenate and set innerHTML
+			 * Description: use string concatenate and set innerHTML. Note that it is recommended to use the 
+			 * array.join technique to concatenate long strings.
 			 */
 
 			// antipattern
@@ -34,9 +35,18 @@
 			});
 			$('#ballers').html(myhtml);
 
+			// concatenate very long strings
+			var myHtmlBuffer = [];
+			$.each(reallyLongArray, function(count, item) {
+				myHtmlBuffer.push('<li>' + item + '</li>');
+			});
+			$('#ballers').html(myHtmlBuffer.join(''));
 
 			// References
 			// http://paulirish.com/2009/perf/
+			// http://www.sitepen.com/blog/2008/05/09/string-performance-an-analysis/
+			// http://stackoverflow.com/questions/112158/javascript-string-concatenation
+			// http://stackoverflow.com/questions/153381/javascript-string-concatenation-faster-than-this-example
 		</script>
 	</body>
-</html>
+</html>
