Skip to content

Instantly share code, notes, and snippets.

@paulirish
Created December 19, 2010 23:37
Show Gist options
  • Save paulirish/747835 to your computer and use it in GitHub Desktop.
Save paulirish/747835 to your computer and use it in GitHub Desktop.
`x || (x = y)` pattern for using an incoming value otherwise using some default
// from https://twitter.com/#!/LeaVerou/status/16613276313980929
// this pattern below is common for using an incoming value otherwise using some default.
/* Bad: */ x = x ? x : y; // simple ternary.
// if x is truthy lets use it.
/* Better: */ x = x || y; // logical OR.
// you could string a bunch of these easily to grab the first non-falsy value:
// x = x || y || z || a;
/* Best: */ x || (x = y); // right-hand assignment!
// done without the parens this would throw a syntax error.
// if x is truthy we don't even need to do the assigment,
// so the logical OR will bypass it.
// jslint especially hates that last one. big surprise.
// and of course, if x is intentionally supposed to be 0, false, or ''
// then you're fucked and you'll have to do something nasty like:
x = (x !== undefined) ? x : y; // yuck. (parens optional)
@mathiasbynens
Copy link

For benchmarks, please refer to the good old http://jsperf.com/conditional-assignment. (Someone else made a different jsPerf test case with inaccurate tests in it, so don’t use that!)

@irae
Copy link

irae commented Dec 22, 2010

@mathiasbynens Great to see this performance tests!
For me, the best scenario would be that my JavaScript compressor would convert this kind of stuff always to the most performant way.
In fact, could the minification tools out there be changing the performance of the code? They do change some if/else statements to ternarys, right?

@mathiasbynens
Copy link

@irae Closure Compiler is known to tweak the code, for example by switching between for and while loops. YUI Compressor doesn’t seem to alter the code in any way (except for removing redundant whitespace and semicolons, of course). I’m not sure about the other JS minifiers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment