基于 window.location.href 的条件 URL 附加或重定向的 Javascript

Question

我正在尝试制作一个书签，单击该书签时将检查当前选项卡/窗口的 URL，以查看它是否包含“char1”和/或“char2”（给定字符）。如果两个字符都存在，它将重定向到另一个 URL，对于另外两个，它将分别附加当前 URL。

我相信肯定有一种比以下方式更优雅的方式来说明这一点（到目前为止，这对我来说非常有效），但我对 Javascript 的了解并不多。我的（笨重且重复的）工作代码（道歉）：

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');
}

完全符合我的需要，但是……至少可以说不是很优雅。

有没有更简单的方法来做到这一点，也许使用 vars 或伪对象？还是更好的代码？

Answer 1

一种可扩展的代码。我疯了吗？

var loc = window.location.href;
var arr = [{
  url: "https://website.com/",
  chars: ["char1", "char2"]
}, {
  url: loc + "append1",
  chars: ["char1"]
}, {
  url: loc + "append2",
  chars: ["char2"]
}];

function containsChars(str, chars)
{
  var contains = true;
  for(index in chars) {
    if(str.indexOf(chars[index]) == -1) {
      contains = false;
      break;
    }
  }
  return contains;
}

for(index in arr) {
 var item = arr[index];
 if(containsChars(loc, item.chars)) {
    window.location.href = item.url;
    break;
 }
}

Answer 2

对 dthorpe 建议的（某种）重构：

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;

调用assign 与为window.location 赋值相同，无论如何，您都在方法中使用了加法赋值+= 运算符：

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

这实际上会在调用 assign 方法之前将“append2”分配到 window.location.href 的末尾，使其变得多余。

您还可以通过将 window.location 设置为 var 来减少 DOM 查找。

Answer 3

var location =window.location.href

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

Answer 4

我能看到的唯一减少是将多余的 indexof 调用提取到 vars 中，然后测试 vars。但它不会对性能产生任何明显的影响。

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