Path: csiph.com!usenet.pasdenom.info!weretis.net!feeder4.news.weretis.net!npeer.de.kpn-eurorings.net!npeer-ng0.de.kpn-eurorings.net!newsfeed.arcor.de!newsspool2.arcor-online.net!news.arcor.de.POSTED!not-for-mail
Content-Type: text/plain; charset="UTF-8"
Message-ID: <1562636.Ll19dNWgsJ@PointedEars.de>
From: Thomas 'PointedEars' Lahn <PointedEars@web.de>
Reply-To: Thomas 'PointedEars' Lahn <cljs@PointedEars.de>
Organization: PointedEars Software (PES)
Date: Sun, 02 Dec 2012 19:39:59 +0100
User-Agent: KNode/4.4.11
Content-Transfer-Encoding: 8Bit
X-Face: %i>XG-yXR'\"2P/C_aO%~;2o~?g0pPKmbOw^=NT`tprDEf++D.m7"}HW6.#=U:?2GGctkL,f89@H46O$ASoW&?s}.k+&.<b';Md8`dH6iqhT)6C^.Px|[=M@7=Ik[_w<%n1Up"LPQNu2m8|L!/3iby{-]A+#YE}Kl{Cw$\U!kD%K}\2jz"QQP6Uqr],./"?;=4v
Face: iVBORw0KGgoAAAANSUhEUgAAADAAAAAwBAMAAAClLOS0AAAAGFBMVEXTxa4RFk5dUWANED8PFEfy7+MGBiW+n3ZNF/QuAAACaElEQVQ4jVXUwVOcMBQG8Dc7Rc4PUntdWV2uxjDpGaGeozOp1woar4jd5t/v9wLstMwsA/ntlxdCAgUc1hjTc9/JCZfGoo3wG3HdmdAWrIJRHe7GM/TmpY5VFefuVcAkkPbLIaN8rmPmjloyZxgyR3GuJ4K0AGtJ2htz8o7yqikm759fldQXaMpbDzjKAG+8v+AugVTOPO5DOjLvGtUYQwh0CPjnVMyGd+8/GfUB5nLKJDD2aLDh5HYyMDJGDwQIo2ZmZcKbowNmAdB/AzyFhrmF2MHRb0QJJfaAnwGB6orZhoykLzJtGwF/xpYxI1dswomiUj3gTuAIqCn/4C7cULwGNBtwMTk3Y4LfKB5YUaOKBKYtpplm7u0vip8tU1NWWyI/7XdcSuIDoMt6rVHMWT0DbjHPGqDqZVSa6zleLcUTcIKLoMv3ueJluALtAo9B302zPPlrtiVScRdCjXvVh3e3JpYa/jjkuC9N+LrBMlz/eAN4eQijX2EdLo6c5tGGHwLyHFtXk89dDGHwCVhG9T0S/j55AhRZgkMCmUQXJ49TnS1wnQDvw0eAh9ICeMmEFbCnPMFzjAvsWoEWEFdYEx+S0MoUZ1gT1wId8+AF3Bl2OoEu906AUHx5VLw/gXYg/x84loOah/2UYNrgiwSwGO7RfUzVBbx/kgpckumGOi6QirtD6gkLTitbnxNol47S2jVc2vsN5kPqaAHT8uUdAJM4v/DanjYOwmUjWznGfwB7sGtAtor5BgofDuzaRj4kSQAqDakTsKORa3Q3xKi3gE1fhl71KRMqrdZ2AWNNg/YOhQyrVBnb+i+nEg4bsDA+egAAAABJRU5ErkJggg==
Subject: Re: Problem w/ creating dynamic TR and its TDs
Newsgroups: comp.lang.javascript
References: <e4c1bc98-b6a5-4142-a7a7-3fff318a7057@l12g2000vbj.googlegroups.com> <k9e45v$d6v$1@speranza.aioe.org> <5462881.fXVs31Ce0e@PointedEars.de> <k9f0rg$ajv$1@speranza.aioe.org> <5551855.fYihAxhPQP@PointedEars.de> <k9g4a1$54q$1@speranza.aioe.org> <2075691.byxZkP05Om@PointedEars.de>
Followup-To: comp.lang.javascript
MIME-Version: 1.0
Lines: 58
NNTP-Posting-Date: 02 Dec 2012 19:40:00 CET
NNTP-Posting-Host: 001e1de2.newsspool1.arcor-online.net
X-Trace: DXC=iaiKWM`8X37d8Nb@@ZG@b=ic==]BZ:af>4Fo<]lROoR1nkgeX?EC@@0IRTKiQcV`14DZm8W4\YJN<R=i2=[N6Y6:^;HD;G0U[d>Ak9?fi>3?6=
X-Complaints-To: usenet-abuse@arcor.de
Xref: csiph.com comp.lang.javascript:17417

Thomas 'PointedEars' Lahn wrote:

> Consider this instead:
> 
>   for (var key in params)
>   {
>     if (params.hasOwnProperty(key))
>     {
>       // …
>     }
>   }
> 
> The least you should do is to cache the value of
> `Object.prototype.hasOwnProperty', maybe even in a bound variable of a
> closure (as can be seen in JSX), if you do not call it through the
> prototype chain.
> 
>   /*
>    * Or `Object.prototype', but `{}' appears to have a shorter
>    * scope chain, which might win over the longer prototype chain
>    */
>   var _hasOwnProperty = {}.hasOwnProperty;
> 
>   for (var key in params)
>   {
>     if (_hasOwnProperty.call(params, key))
>     {
>       // …
>     }
>   }
> 
> It might be useful to combine these:
> 
>   var hasHasOwnProperty =
        ^^^^^^^^^^^^^^^^^
That must _hasOwnProperty.  I wrote the hasHasOwnProperty() approach later 
but it did not work well with boolean expressions, so I reverted it (only 
partially, apparently).

>     (typeof params.hasOwnProperty == "function")
>       ? params.hasOwnProperty
>       : {}.hasOwnProperty;
> 
>   for (var key in params)
>   {
>     if (_hasOwnProperty.call(params, key))
>     {
>       // …
>     }
>   }


PointedEars
-- 
Prototype.js was written by people who don't know javascript for people
who don't know javascript. People who don't know javascript are not
the best source of advice on designing systems that use javascript.
  -- Richard Cornford, cljs, <f806at$ail$1$8300dec7@news.demon.co.uk>