Skip to content

Instantly share code, notes, and snippets.

@paulirish
Created December 19, 2010 23:37
Show Gist options
  • Star 9 You must be signed in to star a gist
  • Fork 4 You must be signed in to fork a gist
  • 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

@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