Question

The main issue I’m thinking about is whether assigning a variable in an if statement is safe and reliable across different browsers. If it is safe, I’d like to use it.

Here it reads the querystring and if the querystring variable SN is either Twitter or Facebook then it enters the if and you can use the variable, if the querystring variable doesn’t exist or is some other value then it goes into the else.

    if(socialNetwork = (window.location.search.indexOf("SN=Twitter") > 0) ? "Twitter" : ((window.location.search.indexOf("SN=Facebook") > 0) ? "Facebook" : null))
    {
        alert(socialNetwork);
    }
    else
    {
        alert("nope");
    }

Answer 1

That’s ugly.

var uselessSocialNetworkingApp = window.location.search.replace(/.*\bSN=(\w+)\b.*/, "$1");
if (uselessSocialNetworkingApp)
  alert("yay!");
else
  alert("no");

It’s kind-of funny that there’d be that hideous construction in the “if” header, but that it’d be an “if” instead of a “? :” expression inside the “alert” argument list 🙂

Also, to be at least slightly sympathetic to the intended style, this is an example of what the “let” statement in ultra-modern Javascript is for.