Comments on: The “Last in a Loop” Bug

By: Kent Brewster

Kent Brewster — Tue, 25 Nov 2008 07:28:41 +0000

This is neat, thanks! Some suggestions:

- label the items #0 through #4, so the alerts actually match the item numbers

- explain variable e on line 6. Is it set to currentLoopValue? If so, how? When I delete it, it seems to run fine.

- expand the explanation of where currentLoopValue is coming from. Right, it’s being passed from the value of i in the closure on line 9, but it’s hard to spot the connection unless you know how closures work.

By: Recent Links Tagged With "function" - JabberTags

Recent Links Tagged With "function" - JabberTags — Sat, 08 Nov 2008 12:34:11 +0000

[...] is releasing Digital Cameras with Makeup Function Saved by ShadowOfTheAssylum on Fri 07-11-2008 The “Last in a Loop” Bug Saved by lihlii on Fri 07-11-2008 Isabel De Los Rios Calorie FAQs Saved by baruchin on Fri [...]

By: daaku

daaku — Thu, 28 Aug 2008 16:39:56 +0000

I find the pattern kinda ugly too.. I’ve like to use Array.prototype.forEach (not available in IE, but easy to fix):

var els = document.getElementById(’last-one-example-2′).getElementsByTagName(’LI’);

// convert to array since above returns a NodeList
var arr = Array.prototype.slice.call(els);

arr.forEach(function(el, index) {
el.onclick = function(e) {
alert(’You clicked element #’ + index);
}
});