Comments on: Three handy javascript techniques

By: Trinithis

Trinithis — Mon, 14 Jan 2008 22:30:22 +0000

arguments.calllee is NOT depricated. In fact, I use it over the function’s name due to refractoring purposes.

You might be confused with Function.arguments.callee because that is depricated.

By: serkanyersen

serkanyersen — Fri, 11 Jan 2008 08:10:55 +0000

use alisasing isteadof with
‘el2 = el.arg1.arg2.arg3.looong.use alert(el2.someVal);

This faster safer and usable.

Great blog by the way.

By: digitarald

digitarald — Sun, 23 Sep 2007 07:41:11 +0000

Just as quick note: arguments.callee is not deprecated, Function.arguments including its properties (callee …) is deprecated, in favor of the functions local variable arguments.

http://developer.mozilla.org/en/docs/Core_JavaScript_1.5_Reference:Functions:arguments:callee

By: Bas

Bas — Wed, 22 Aug 2007 10:20:37 +0000

I’ll fix it for you :), thanks for sharing!

By: simon

simon — Wed, 22 Aug 2007 09:56:36 +0000

hmmm, apparently I suck at using that funky markup business :)

By: simon

simon — Wed, 22 Aug 2007 09:54:14 +0000

If I remember rightly, the with keyword only reliably works with predefined namespaces in that if you were working with an object that looks like:

var ob =  {
   x: 12,
   y: "hello"
}

then

with(ob) {
   x += 1;
}

would work no worries, however you couldn't do:

with(ob) {
  z = 12;
}

and for this reason it might be more versatile to use a simple object assignation, like so (this time using a more contextual / useful / complete example):

var p = Array.prototype;
p.toJSON = function{
   ...
}

thus gaining the benefits of smaller code (albeit slightly less small with bigger operations) using a much more flexible method.

By: Bas Wenneker

Bas Wenneker — Tue, 21 Aug 2007 15:26:16 +0000

Oh, sorry, now I see what you mean. Yeah you're right, the 'a' is in the word 'span', so this condition returns true. To have that problem fixed I'd use something like this:

if( '|p|span|'.indexOf( '|' + tagName + '|' ) ){
     // Some action
}

Thanks for pointing me on this.

By: los

los — Tue, 21 Aug 2007 15:20:59 +0000

The reason why I would use another solution is in the first line of my first comment: The “simpler” condition which uses indexOf is NOT EQUAL to the original condition!

By: Bas

Bas — Tue, 21 Aug 2007 14:49:35 +0000

@Ios: That’s also a solution, but like you wrote, you need to implement Array.contains().
@Tommy: Regular expressions are computationally heavier than String functions (most of the times), but if that doesn’t matter, you’re completely right.

By: Tommy Valand

Tommy Valand — Tue, 21 Aug 2007 13:57:47 +0000

Regarding tip 2, if you have som familiarity with Regular Expression, you can save even more code:

/*
parenthesis around regexp not needed
but easier to read
i for case-insensitive
^ start of string
$ end of string
*/
if( (/^(div|input|p|span|blockquote)$/i).test( document.getElementById('id').tagName ) ){
	//dostuff
}