I see you do
var runFilters = function(){...};
Whilst this is not wrong, it would be far cleaner to write
function runFilters() {...}
in my opinion, as its quite clearly a function and not a reference to one.
> 3. Lines 66-81 are painful, and I think you know it. Instead of looping
> through every element, try giving all the elements a unique id, or something
> like that, so that you can hash the results of one of those arrays and create
> an O(n) loop instead of an O(n*n) pattern.
I'd have to agree with Nathan, You could probably manage your data better and
compare a simple ID.
--
To view archived discussions from the original JSMentors Mailman list:
http://www.mail-archive.com/jsmentors@jsmentors.com/
To search via a non-Google archive, visit here:
http://www.mail-archive.com/jsmentors@googlegroups.com/
To unsubscribe from this group, send email to
jsmentors+unsubscr...@googlegroups.com
