Path: csiph.com!v102.xanadu-bbs.net!xanadu-bbs.net!usenet.blueworldhosting.com!nntp-feed.chiark.greenend.org.uk!ewrotcd!news.etla.org!poup.poupinou.org!fdn.fr!feeder1-2.proxad.net!proxad.net!feeder2-2.proxad.net!newsfeed.arcor.de!newsspool4.arcor-online.net!news.arcor.de.POSTED!not-for-mail
Content-Type: text/plain; charset="UTF-8"
Message-ID: <6079239.H4DsMt3hnF@PointedEars.de>
From: Thomas 'PointedEars' Lahn <PointedEars@web.de>
Reply-To: Thomas 'PointedEars' Lahn <usenet@PointedEars.de>
Organization: PointedEars Software (PES)
Date: Mon, 20 May 2013 13:48:28 +0200
User-Agent: KNode/4.10.2
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: jslint and typeof
Newsgroups: comp.lang.javascript
References: <kr2dnRlKLtGEaQTMnZ2dnUVZ_g6dnZ2d@westnet.com.au>
MIME-Version: 1.0
Lines: 46
NNTP-Posting-Date: 20 May 2013 13:50:17 CEST
NNTP-Posting-Host: 4ef6fb71.newsspool2.arcor-online.net
X-Trace: DXC=D\CV<bM5_OGI7\_^6>c20JA9EHlD;3YcB4Fo<]lROoRA8kF<OcfhCOKIAON^PBnRUGDZm8W4\YJNLb@mF9jNikdE7a5\<`>]gfN65mC4\:DmnB
X-Complaints-To: usenet-abuse@arcor.de
Xref: csiph.com comp.lang.javascript:19840

Andrew Poulos wrote:

> A fragment of some code I've been given has this
> 
> if (typeof options.actor !== "undefined") {
>    actor = options.actor;
> } else if (this.actor !== null) {
>    actor = this.actor;
> }
> 
> and jslint complains of the first line that
> 
> "Unexpected 'typeof'. Use '===' to compare directly with undefined."
> 
> but the complaint makes little sense to me.
> 
> Why is typeof unexpected?

Probably because Douglas Crockford is wrong.  Prior to ECMAScript Ed. 5, 
when jslint was released, the “undefined” property of the Global Object was 
already writable, and it was safer to compare against the result of the 
“typeof” operation.  Nowadays, and especially with host objects, there are 
insufficient good reasons to choose a different approach.

> Why is it asking compare the string return with a primitive?

Crockford suggests that you write

  if (options.actor !== undefined) {

instead.  But this can only be recommended if you can assume that 
“undefined” is read-only.

Ignore jslint.

That said, this should probably be written

  var actor = options.actor || this.actor;

See also jsx.array.createComparator().

-- 
PointedEars

Twitter: @PointedEars2
Please do not Cc: me. / Bitte keine Kopien per E-Mail.