Question

I am trying to make a bookmarklet that when clicked will check the URL of the current tab/window to see if it contains ‘char1’ and/or ‘char2’ (a given character). If both chars are present it redirects to another URL, for the other two it will append the current URL respectively.

I believe there must be a more elegant way of stating this than the following (which has so far worked perfectly for me) but I don’t have great knowledge of Javascript. My (unwieldy & repetitive) working code (apologies):

if (window.location.href.indexOf('char1') != -1 &&
    window.location.href.indexOf('char2') != -1)
{
    window.location="https://website.com/";
}
else if (window.location.href.indexOf('char1') != -1)
{
    window.location.assign(window.location.href += 'append1');
}
else if (window.location.href.indexOf('char2') != -1)
{
    window.location.assign(window.location.href += 'append2');
}

Does exactly what I need it to but, well… not very graceful to say the least.

Is there a simpler way to do this, perhaps with vars or a pseudo-object? Or better code?

Answer 1

A (sort-of) refactoring of dthorpe’s suggestion:

var hasC1  = window.location.href.indexOf('char1')!=-1
var hasC2  = window.location.href.indexOf('char2')!=-1
var newLoc = hasC1 
               ? hasC2 ? "https://website.com/" : window.location.href+'append1'
               : hasC2 ? window.location.href+'append1' : '';

if (newLoc)
    window.location = newLoc;

Calling assign is the same as assigning a value to window.location, you were doing both with the addition assignment += operator in the method anyway:

window.location.assign(window.location.href+='append2')

This would actually assign “append2” to the end of window.location.href before calling the assign method, making it redundant.

You could also reduce DOM lookups by setting window.location to a var.