Comments on: Beginning UI Design http://www.bigdoor.com/blog/beginning-ui-design/ Free gamification platform that helps companies increase their traffic and revenue through social rewards Fri, 03 Feb 2012 21:02:26 +0000 hourly 1 http://wordpress.org/?v=3.0.4 By: Collin http://www.bigdoor.com/blog/beginning-ui-design/#comment-42 Collin Mon, 29 Mar 2010 16:34:18 +0000 http://www.bigdoor.com/?p=742#comment-42 I do think it is a good idea to have a function which iterates an array and processes the entries appropriately. The one thing I'm wary about is adding that function to the prototype. In our case, it can cause major problems if we alter the Array prototype if a site that use our code does as well. Just a thought. When you call that mapping function, I'm confused as to where your e variable is coming from. Did you mean to pass an anonymous function in and call each handler that way? this.events[event].map(function(e) { if(typeof e == 'function') { return e(); } }) Thanks for the input! I do think it is a good idea to have a function which iterates an array and processes the entries appropriately. The one thing I’m wary about is adding that function to the prototype. In our case, it can cause major problems if we alter the Array prototype if a site that use our code does as well. Just a thought.

When you call that mapping function, I’m confused as to where your e variable is coming from. Did you mean to pass an anonymous function in and call each handler that way?

this.events[event].map(function(e) { if(typeof e == ‘function’) { return e(); } })

Thanks for the input!

]]> By: Shea Kauffman http://www.bigdoor.com/blog/beginning-ui-design/#comment-41 Shea Kauffman Tue, 23 Mar 2010 20:06:07 +0000 http://www.bigdoor.com/?p=742#comment-41 Why not add a map aspect to the array object: Array.prototype.map = function(fn) { var arr = []; var len = this.length; for(i=0;i<len;i++) { arr.push(fn(this[i])); } return arr; }; Then: var manager = { events: {}, trigger: function(event) { this.events[event].map( this.events[event][e]) } } Why not add a map aspect to the array object:

Array.prototype.map = function(fn) {
var arr = [];
var len = this.length;
for(i=0;i<len;i++)
{
arr.push(fn(this[i]));
}
return arr;
};

Then:
var manager = {
events: {},
trigger: function(event) {
this.events[event].map( this.events[event][e])
}
}

]]>